On Mon, Sep 30, 2019 at 08:58:19PM +0200, Klemens Nanni wrote:
> segflg is singular, had to read that sentence thrice to get it right.
> 
> OK?
> 
> 
> Index: man/man9/namei.9
> ===================================================================
> RCS file: /cvs/src/share/man/man9/namei.9,v
> retrieving revision 1.21
> diff -u -p -r1.21 namei.9
> --- man/man9/namei.9  13 Aug 2018 23:13:02 -0000      1.21
> +++ man/man9/namei.9  30 Sep 2019 18:55:51 -0000
> @@ -272,7 +272,7 @@ These are the values to which
>  and
>  .Em ndp-\*[Gt]ni_cnd.cn_flags
>  are respectively set.
> -The segment flags which defines whether the pathname is in kernel
> +The segment flag which defines whether the pathname is in kernel
>  address space or user address space is specified by
>  .Fa segflg .
>  The directory from which relative pathnames will be looked up is
> 

i think this is probably correct. but i also think that text should be
in commas:

        The segment flag, which defines ..., is specified by segflg.

there's another text segment that i suspect should be singular:

    FUNCTIONS
         namei(ndp)
                  Convert a pathname into a pointer to a vnode.  The
                  pathname is specified by ndp->ni_dirp and is of
                  length ndp->ni_pathlen.  The ndp->segflg flags
                  defines whether the name in ndp->ni_dirp is an
                  address in kernel space (UIO_SYSSPACE) or an
                  address in user space (UIO_USERSPACE).

i suspect it should read "The ndp->segflg flag defines".

jmc

Reply via email to