Re: [PATCH] compat: add memrchr()

From: Ludvig Strigeus <ludde@strigeus.com>
Date: Oct 15, 2010, 08:56 UTC
Message-ID: <loom.20101015T105350-205@post.gmane.org>
In-Reply-To: <20101015051750.GA21830@burratino>

Jonathan Nieder <jrnieder <at> gmail.com> writes:

Show 5 quoted lines

> 
> Reimplement another handy convenience function from glibc.  memrchr()
> searches from the end of a memory area for a particular character.  It
> is similar to strrchr() but takes a length argument and is
> binary-safe.

System memrchr uses a byte comparison rather than an int comparison.

"The memchr() function scans the first n bytes of the memory area pointed to by s for the character c. The first byte to match c (interpreted as an unsigned character) stops the operation. "

See: http://linux.die.net/man/3/memrchr

It's a good idea to do the same thing for compatibility reasons.

/Ludde