Re: [PATCH 2/2] mem: Add memswap() function

David Gibson <[email protected]> Wed, 9 Sep 2015 17:37:59 +1000

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

On Wed, Sep 09, 2015 at 04:08:38AM +0930, Paul 'Rusty' Russell wrote:
> David Gibson <[email protected]> writes:
> > Add a memswap() function to the mem module, which exchanges two (equal
> > sized, non-overlapping) memory regions.
> 
> Nice!
> 
> I wonder about a #if CCAN_MEM_DEBUG'd test which assert() tests for
> no overlap.
> 
> Though maybe it's cheap enough that a simple assert() is always OK?

Good idea.  And actually.. an exposed function to tell if memory
ranges overlap might be nice too, why not.

-- 
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)