Paul Eggert <[EMAIL PROTECTED]> wrote: > Jim Meyering <[EMAIL PROTECTED]> writes: >> + enum Fts_stat need_stat = p->fts_statp->st_size; >> + switch (need_stat) > > Dunno if this matters, but on typical 64-bit platforms converting to > enum Fts_stat and then switching won't abort in the rare cases when > p->fts_statp->st_size equals 2**32 + 1 or 2**32 + 2, whereas the > previous code would correctly abort in those two cases.
Thanks, but that doesn't matter. When st_size is used that way (when ->fts_info == FTS_NSOK), its value must be FTS_STAT_REQUIRED (1) or FTS_NO_STAT_REQUIRED (2). If there is a way to make it have any other value, it's a bug.
