Re: [PATCH 2/2] Various pages: Normalize NOTES about non-existing glibc wrappers

["Michael Kerrisk (man-pages)" <mtk.manpages@gmail.com>]

On 12/30/20 1:52 PM, Alejandro Colomar (man-pages) wrote:
> 
> 
> On 12/30/20 10:38 AM, Michael Kerrisk (man-pages) wrote:
>> Hi Alex,
>>
>> On Tue, 29 Dec 2020 at 21:53, Alejandro Colomar (man-pages)
>> <alx.manpages@gmail.com> wrote:
>>>
>>> Hi Michael,
>>>
>>> Please, see a comment below.
>>>
>>> Thanks,
>>>
>>> Alex
>>>
>>> On 12/29/20 9:32 PM, Michael Kerrisk (man-pages) wrote:
>>>> Hi Alex, Ganimedes,
>>>>
>>>> Again, in the subject line, s/
>>>>
>>>> non-existing/nonexistent/
>>>>
>>>> [...]
>>>>
>>>>> --- a/man2/getdents.2
>>>>> +++ b/man2/getdents.2
>>>>> @@ -211,13 +211,13 @@ SVr4.
>>>>>  Library support for
>>>>>  .BR getdents64 ()
>>>>>  was added in glibc 2.30;
>>>>> -there is no glibc wrapper for
>>>>> -.BR getdents ().
>>>>> -Calling
>>>>> +Glibc does not provide a wrapper for
>>>>> +.BR getdents ();
>>>>> +call
>>>>
>>>> s/call/calling/
>>>
>>> See below
>>>
>>>>
>>>>>  .BR getdents ()
>>>>>  (or
>>>>>  .BR getdents64 ()
>>>>> -on earlier glibc versions) requires the use of
>>>>> +on earlier glibc versions) using
>>>
>>> The new version is 'Call XXX using syscall(2).'.
>>
>> I need more context here please. What do you mean with "The new version"?
> 
> Hi Michael,
> 
> I mean after applying this patch.
> 
> This patch changes ". Calling XXX requires the use of syscall(2)"
> to "; call XXX using syscall(2)".
> 
> "; calling XXX using syscall(2)" wouldn't make sense.

Sorry -- got it now. I misread the patch.

Cheers,

Michael



-- 
Michael Kerrisk
Linux man-pages maintainer; http://www.kernel.org/doc/man-pages/
Linux/UNIX System Programming Training: http://man7.org/training/