This is the mail archive of the
libc-alpha@sourceware.org
mailing list for the glibc project.
Re: [PATCH 1/6] Consolidate Linux sigprocmask implementation (BZ #22391)
- From: Zack Weinberg <zackw at panix dot com>
- To: Rical Jasan <ricaljasan at pacific dot net>
- Cc: Adhemerval Zanella <adhemerval dot zanella at linaro dot org>, GNU C Library <libc-alpha at sourceware dot org>
- Date: Mon, 6 Nov 2017 09:00:10 -0500
- Subject: Re: [PATCH 1/6] Consolidate Linux sigprocmask implementation (BZ #22391)
- Authentication-results: sourceware.org; auth=none
- References: <1509745249-11404-1-git-send-email-adhemerval.zanella@linaro.org> <CAKCAbMiS8shh4vSDRc_Gr2=RjL06VFOUX=yb1RyeHL+14cgCvA@mail.gmail.com> <0e369631-e59a-0c67-db48-4290ffbbefff@pacific.net>
On Sun, Nov 5, 2017 at 9:59 PM, Rical Jasan <ricaljasan@pacific.net> wrote:
> On 11/05/2017 02:46 PM, Zack Weinberg wrote:
>> +@deftypevr Macro int SIGRTMIN
>> +@deftypevrx Macro int SIGRTMAX
>> +@standards{POSIX.1, signal.h}
>> +@cindex real-time signals
>> +The range of signal numbers @code{SIGRTMIN}, @code{SIGRTMIN+1},
>> +@dots{}, @code{SIGRTMAX} is also set aside for you to use any way you
>> +want. In addition, these signals (and no others) are guaranteed to
>> +support @dfn{real-time} signal semantics, which unfortunately this
>> +manual does not yet document.
>
> I don't think we should say that at the end. It will be wrong someday
> (hopefully), without having to do anything to it. It really just needs
> a @pxref we don't have yet, so instead, say nothing, which will continue
> to be correct both before and after real-time signal semantics are
> documented, and after they are, the reference can be added to improve
> the manual.
I don't want to say nothing, because if we say nothing, it's an
undefined term and readers will be frustrated. If we admit that this
is a gap in the manual, they'll still be frustrated but at least they
won't waste time searching for the missing piece.
> Maybe a bug report that real-time signal semantics aren't documented
> would be a good place to both track the issue and keep a reminder to
> update this paragraph.
Agree with this part.
zw