Re: Small improvement in man page readdir(3)

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hello Ricardo

On Sat, Sep 27, 2008 at 4:04 PM, Ricardo Catalinas Jiménez
<jimenezrick@xxxxxxxxx> wrote:
> In NOTES section says Linux use the d_type field of the struct dirent
> which is only true with certain filesystem like ext2 and ext3. With
> XFS and reiserfs d_type is always equal to 0.
> I think would be a good idea to add just a small comment like "Under
> certain filesystem like ext2 and ext3..." to avoid confussion.

Thanks.  I've added some text covering that point to readdir.3, and
also to getdents.2 (which is used to implement readdir(3)).

The fixes will be in man-pages-3.11, and are available on git now.

> Also there is a small typo in the SYNOPSIS section where there is an
> extra line break between the two includes:
> #include <sys/types.h>
>     //extra line break here
> #include <dirent.h>

An earlier commit makes this point redundant:

commit 15dd4fb58b96d3cf9cc88eab85bce8e7baf4f8c0
Author: Michael Kerrisk <mtk.manpages@xxxxxxxxx>
Date:   Fri Jul 4 22:38:01 2008 +0000

    Add documentation of readdir_r().
    Remove <sys/types.h> from SYNOPSIS; POSIX.1-2001 does not require it.
    Some minor rewordings.

> I have the readdir(3) from 2007-07-30 which comes with the ubuntu 8.04
> in the manpages-dev package version 2.77-1.

Thanks for the details.

> Thanks in advance for your attention.

And thanks very much for your report.

Cheers,

Michael

-- 
Michael Kerrisk
Linux man-pages maintainer; http://www.kernel.org/doc/man-pages/
git://git.kernel.org/pub/scm/docs/man-pages/man-pages.git
man-pages online: http://www.kernel.org/doc/man-pages/online_pages.html
Found a bug? http://www.kernel.org/doc/man-pages/reporting_bugs.html
--
To unsubscribe from this list: send the line "unsubscribe linux-man" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [Kernel Documentation]     [Netdev]     [Linux Ethernet Bridging]     [Linux Wireless]     [Kernel Newbies]     [Security]     [Linux for Hams]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux RAID]     [Linux Admin]     [Samba]

  Powered by Linux