[PATCH] Add more tests for memrchr

H.J. Lu hjl.tools@gmail.com
Sun Jun 4 16:51:00 GMT 2017


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


-- 
H.J.



More information about the Libc-alpha mailing list