Re: [PATCH v2] bindresvport.3: ATTRIBUTES: Note function that is thread-safe

[Peng Haitao <penght-BthXqXjhjHXQFUHtdCDX3A@public.gmane.org>]

On 05/19/2013 02:24 AM, Michael Kerrisk wrote:
>> diff --git a/man3/bindresvport.3 b/man3/bindresvport.3
>> index d2e73a4..2c7152d 100644
>> --- a/man3/bindresvport.3
>> +++ b/man3/bindresvport.3
>> @@ -89,6 +89,16 @@ is not NULL and
>>  .I sin->sin_family
>>  is not
>>  .BR AF_INET .
>> +.SH ATTRIBUTES
>> +.SS Multithreading (see pthreads(7))
>> +Before glibc 2.17, the
>> +.BR bindresvport ()
>> +uses a static variable that is not protected, so it is not thread-safe.
> 
> Missing word "function" here (function uses..."). This was also the
> case in a number of the other patches you sent recently. (I fixed them all.)
> 

Thanks.

>> +Since glibc 2.17, the
>> +.BR bindresvport ()
> 
> (Missing word "function".)
>> +uses a lock to protect static variable, so it is thread-safe.
> 
> "uses a lock to protect *a* static variable"
> (I fixed this, and also in a few other patches.)
> 

Thanks.

>> +.\" Modified in commit f6da27e53695ad1cc0e2a9490358decbbfdff5e5
> 
> Here, its is enough just to write
> 
> +.\" commit f6da27e53695ad1cc0e2a9490358decbbfdff5e5
> 
> And I'd rearrange things a little, so that the patch is better as:
> 
> ==
> +Since glibc 2.17,
> +.\" commit f6da27e53695ad1cc0e2a9490358decbbfdff5e5
> +the
> +.BR bindresvport ()
> ...
> ==
> 
> I made that fix (and also in one other patch.)
> 

Thanks.

-- 
Best Regards,
Peng

> Cheers,
> 
> Michael
> 
> 
>>  .SH CONFORMING TO
>>  Not in POSIX.1-2001.
>>  Present on the BSDs, Solaris, and many other systems.
> 
> Thank you, Peng.
> 
> Cheers,
> 
> Michael
> 
> 
> --
> To unsubscribe from this list: send the line "unsubscribe linux-man" in
> the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
> 

--
To unsubscribe from this list: send the line "unsubscribe linux-man" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html