diff options
author | Tobias Klauser <tklauser@distanz.ch> | 2007-06-12 18:05:22 +0200 |
---|---|---|
committer | Tobias Klauser <tklauser@xenon.tklauser.home> | 2007-06-12 18:05:22 +0200 |
commit | 4b51d764955cf09604fffadad9b9b88c72ab45db (patch) | |
tree | e032e6c2c6dc1b8e97acfcce1603cb7bc8349fe1 | |
parent | 5015c1eb9a07b2f8351249eac62ab1fe55f03395 (diff) |
inotail.c: Cleanup
Save us some lines
-rw-r--r-- | inotail.c | 18 |
1 files changed, 7 insertions, 11 deletions
@@ -81,10 +81,9 @@ static void usage(const int status) static inline void setup_file(struct file_struct *f) { - f->fd = -1; + f->fd = f->i_watch = -1; f->st_size = 0; f->ignore = 0; - f->i_watch = -1; } static void ignore_file(struct file_struct *f) @@ -209,10 +208,8 @@ static off_t bytes_to_offset(struct file_struct *f, unsigned long n_bytes) if (from_begin) { if (n_bytes > 0) offset = (off_t) n_bytes - 1; - } else { - if ((off_t) n_bytes < f->st_size) - offset = f->st_size - (off_t) n_bytes; - } + } else if ((off_t) n_bytes < f->st_size) + offset = f->st_size - (off_t) n_bytes; return offset; } @@ -382,19 +379,18 @@ static int watch_files(struct file_struct *files, int n_files) ssize_t len; int ev_idx = 0; - /* Keep trying in the case of EINTR (see below) */ + /* Keep trying in the case of EINTR */ for (;;) { len = read(ifd, buf, (n_files * INOTIFY_BUFLEN)); if (unlikely(len < 0)) { - if (errno == EINTR) { - /* Some form of signal, likely ^Z/fg's STOP and CONT interrupted the inotify read, retry */ + /* Some form of signal, likely ^Z/fg's STOP and CONT interrupted the inotify read, retry */ + if (errno == EINTR) continue; - } else { + else { fprintf(stderr, "Error: Could not read inotify events (%s)\n", strerror(errno)); exit(EXIT_FAILURE); } } - /* The read did succeed */ break; } |