Jim Meyering <[EMAIL PROTECTED]> writes:

> 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.

Yes, but the point of the switch is that it attempts to detect these
bugs, and to abort if there is a bug by falling into the default case.
Formerly the code detected the bug when st_size was 2**32 + 2.
Currently the code does not detect the bug in this case; it treats
2**32 + 2 like FTS_STAT_REQUIRED.

It's a very minor point, and not worth much worry.  (I'm just trying
to explain the point at this point. :-)


Reply via email to