Skip site navigation (1)Skip section navigation (2)
Date:      Sun, 29 Jul 2018 00:45:23 +0300
From:      Konstantin Belousov <kostikbel@gmail.com>
To:        Conrad Meyer <cem@freebsd.org>
Cc:        src-committers <src-committers@freebsd.org>, svn-src-all@freebsd.org, svn-src-head@freebsd.org
Subject:   Re: svn commit: r336835 - head/lib/libc/gen
Message-ID:  <20180728214523.GC40119@kib.kiev.ua>
In-Reply-To: <CAG6CVpV78tghDqgE-nGp0WeGZAxOpVp_y%2BpyZLob0DuRuHq_ug@mail.gmail.com>
References:  <201807281908.w6SJ80qo089859@repo.freebsd.org> <20180728200611.GA40119@kib.kiev.ua> <CAG6CVpVEhyutDe7DL7ECVbKi=JpbD%2BAD8d633=eLX6-VbD4i9g@mail.gmail.com> <20180728203253.GB40119@kib.kiev.ua> <CAG6CVpV78tghDqgE-nGp0WeGZAxOpVp_y%2BpyZLob0DuRuHq_ug@mail.gmail.com>

next in thread | previous in thread | raw e-mail | index | archive | help
On Sat, Jul 28, 2018 at 01:59:16PM -0700, Conrad Meyer wrote:
> On Sat, Jul 28, 2018 at 1:32 PM, Konstantin Belousov
> <kostikbel@gmail.com> wrote:
> > Our existing practice is to not provide an advise in the man pages
> > for the userspace code which would harm the portability or the standard
> > conformance.
> 
> Perhaps, but you do not have to look beyond this manual page to find
> that such a practice, if it is even nominally applied, is not followed
> consistently.   The remainder of the document spells the terminating
> value as plain "NULL."  FreeBSD manual pages consistently refer to
> NULL as a pointer value.
> 
> The Linux manual page spells the terminating value (char *)NULL and
> explicitly documents why.  That is perhaps both pedantically correct
> and clear.  I am amenable to such a change.
I do not see much difference between (char *)0 and (char *)NULL.  Both
are better there, because both are correct.

If you prefer (char *)NULL, please use it.

> 
> > I am not so sure about CheriBSD, for instance.
> 
> I observe simply that CheriBSD is not FreeBSD and that CHERI is an
> exotic architecture.
We should not give an advise which breaks other architectures, even
if not included into the mainline repo.



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20180728214523.GC40119>