Re: [bug-gnulib] Re: [patch #3741] Added support for Interix within mountlist.c

[Paul Eggert]

Jim Meyering <address@hidden> writes:

> +     /* In the unlikely event that opendir and each readdir
> +        succeed, but all statvfs calls fail, ensure that we
> +        fail with a valid errno value.  */

If I'm understanding the code correctly, the comment should read "a
statvfs call fails", not "all statvfs calls fail".

> +     char file_name[9 + NAME_MAX]; /* 8 for /dev/fs/ + 1 for NUL */

A minor point.  The above can be replaced by something like this:

    static char const DEV_FS[] = "/dev/fs/";
    char file_name[sizeof DEV_FS + NAME_MAX];

and then you can replace later occurrences of "/dev/fs/" with DEV_FS.