This is the mail archive of the libc-alpha@sourceware.org mailing list for the glibc project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [PATCH] Clean up semaphore EINTR handling after Linux futex docs clarification.


> +	     EINTR is returned if we are interrupted by a signal and we

I'd s/and we/; we/ here.  As written it feels ambiguous syntactically and
requires real understanding of the context just to see how to parse the
sentence.

> +	     forward this to the caller.  (See futex_wait and related
> +	     documentation.  Before Linux 2.6.22, EINTR was also returned on
> +	     spurious wake-ups;  we only support more recent Linux versions,

Only single space after ; here.

Otherwise seems good to me.


Thanks,
Roland


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]