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

["Alejandro Colomar (man-pages)" <alx.manpages@gmail.com>]

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.

Thanks,

Alex

> 
> Thanks,
> 
> Michael
> 
>