This is the mail archive of the
newlib@sourceware.org
mailing list for the newlib project.
Re: [PATCH] Add rawmemchr
- From: Peter Rosin <peda at lysator dot liu dot se>
- To: newlib at sourceware dot org
- Date: Mon, 24 Jun 2013 13:01:33 +0200
- Subject: Re: [PATCH] Add rawmemchr
- References: <51C7D08E dot 5090706 at users dot sourceforge dot net> <20130624094246 dot GD14427 at calimero dot vinschen dot de>
On 2013-06-24 11:42, Corinna Vinschen wrote:
> On Jun 23 23:52, Yaakov (Cygwin/X) wrote:
>> Code is based on memchr.c; patch for newlib attached.
>
> In how far is rawmemchr different from strchr? Why don't we just alias
> rawmemchr to strchr? Our strchr also uses an optimized search
> algorithm, so, as far as I'm concerned, there's no reason to duplicate
> this functionality. Am I missing something?
See this description, which was part of the patch:
+ This function searches memory starting at <<*<[src]>>> for the
+ character <[c]>. The search only ends with the first occurrence
+ of <[c]>; in particular, <<NUL>> does not terminate the search.
+ No bounds checking is performed, so this function should only
+ be used when it is certain that the character <[c]> will be found.
Cheers,
Peter