This is the mail archive of the
newlib@sourceware.org
mailing list for the newlib project.
Re: [PATCH] add memrchr(3)
On Wed, May 09, 2012 at 10:17:21AM -0500, Yaakov (Cygwin/X) wrote:
>On Wed, 2012-05-09 at 10:48 -0400, Christopher Faylor wrote:
>> Why not just grab the source from FreeBSD?
>>
>> http://svnweb.freebsd.org/base/head/lib/libc/string/memrchr.c?revision=178051&view=markup
>>
>> (or actually from OpenBSD since that's where the above originated)
>
>That would have been easy, but then everyone would be complaining that
>its not optimized for speed. :-)
Yeah, that's usually the response whenever I make this suggestion. I'll
give my usual response to that response:
1) This code is good enough for an OS used by thousands of people.
2) We've already seen that it's possible to do a flawed implementation
(no offense intended) so we're relying on catching bugs with code
inspection rather than using something with a known track record. Given
that this is not a function which is used a whole lot, I don't see why
we'd have to worry about extreme optimization.
Also, there is at least one assembly version of this function floating
around. It has a binary documentation requirement but there may be
other optimized versions around too.