[PATCH] Add more tests for memrchr

Adhemerval Zanella adhemerval.zanella@linaro.org
Tue Jun 6 19:46:00 GMT 2017



On 04/06/2017 13:51, H.J. Lu wrote:
> On Mon, May 29, 2017 at 6:51 AM, H.J. Lu <hjl.tools@gmail.com> wrote:
>> On Mon, May 29, 2017 at 1:36 AM, Rafal Luzynski
>> <digitalfreak@lingonborough.com> wrote:
>>> If the subject line of this message is an actual commit comment
>>> then shouldn't it say "memrchr" instead of "memchr"?
>>>
>>> Otherwise I think I'm unable to tell for sure if the patch is
>>> correct or not.
>>
>> Oops.  Fixed:
>>
>> This patch adds some memrchr tests for len == 0 and for positions close
>> to the beginning, which are equivalent to positions close to the end for
>> memchr.
>>
>> * string/test-memrchr.c (test_main): Add tests for len == 0 and
>> for positions close to the beginning, which are equivalent to
>> positions close to the end for memchr.
>>
> 
> Any comments?
> 
> The patch is at
> 
> https://sourceware.org/ml/libc-alpha/2017-05/msg00833.html
> 
> 

LGTM, thanks.



More information about the Libc-alpha mailing list