Re: [PATCH v4 0/4] mem: extend with a few additional helpers similar to existing string-only functions

David Gibson <[email protected]> Mon, 7 Sep 2015 00:24:48 +1000

Newsgroups	org.ozlabs.lists.ccan
Message-ID	<[email protected]>

On Sat, Sep 05, 2015 at 09:21:02PM -0400, Cody P Schafer wrote:
> since v3:
> 
>  - fix memeq() return value
>  - add memends_str() helper
> 
> since v2:
> 
>  - switch all the macros to static inline to avoid multiple evaluations of args
>  - add patch adding PURE_FUNCTION attributes
> 
> since v1:
> 
>  - drop a patch that David pulled in
>  - add extra macro parens
>  - add blank lines after defintions
>  - covert memcchr's while loop to a for.
>  - change examples for mempbrkm and mempbrk to use '\0' in places
>  - fix spelling
>  - use memeq() even where we know the mem lengths are equal in memstarts()
>  - add a reviewed-by to the 2nd patch, no other changes to that patch.

Looks good, I've applied this to the repository.


-- 
David Gibson			| I'll have my music baroque, and my code
david AT gibson.dropbear.id.au	| minimalist, thank you.  NOT _the_ _other_
				| _way_ _around_!
http://www.ozlabs.org/~dgibson

_______________________________________________
ccan mailing list
[email protected]
https://lists.ozlabs.org/listinfo/ccan

signature.asc (application/pgp-signature, 819 B)