diff options
author | Jim Meyering <jim@meyering.net> | 1999-07-11 09:05:31 +0000 |
---|---|---|
committer | Jim Meyering <jim@meyering.net> | 1999-07-11 09:05:31 +0000 |
commit | 3f622f08e5a72fa1adcddb84dad6e0dd65b6edb0 (patch) | |
tree | 200e9ba65bb0cafcbecdd81e300b00c188630ed8 /src/tail.c | |
parent | 8287d3c4c4ab038a6b7300aec2302535363cbad9 (diff) | |
download | coreutils-3f622f08e5a72fa1adcddb84dad6e0dd65b6edb0.tar.xz |
(recheck): Use assert instead of unnecessary close_fd.
Remove a couple unnecessary FIXME comments.
Diffstat (limited to 'src/tail.c')
-rw-r--r-- | src/tail.c | 7 |
1 files changed, 3 insertions, 4 deletions
diff --git a/src/tail.c b/src/tail.c index 567893046..f680f84f2 100644 --- a/src/tail.c +++ b/src/tail.c @@ -729,7 +729,7 @@ recheck (struct File_spec *f) && !S_ISFIFO (new_stats.st_mode)) { fail = 1; - f->errnum = -1; /* FIXME */ + f->errnum = -1; error (0, 0, _("`%s' has been replaced with a non-regular file; \ cannot follow end of non-regular file"), @@ -750,8 +750,7 @@ cannot follow end of non-regular file"), else if (prev_errnum && prev_errnum != ENOENT) { new_file = 1; - /* FIXME-now: is this close ever necessary? */ - close_fd (f->fd, pretty_name (f)); /* close the old handle */ + assert (f->fd == -1); error (0, 0, _("`%s' has become accessible"), pretty_name (f)); } else if (f->ino != new_stats.st_ino || f->dev != new_stats.st_dev) @@ -1102,7 +1101,7 @@ tail_file (struct File_spec *f, off_t n_units) error (0, 0, _("%s: cannot follow end of non-regular file"), pretty_name (f)); errors = 1; - f->errnum = -1; /* FIXME */ + f->errnum = -1; } if (errors) { |