Re: [PATCH 5/8] rhashtable: remove rhashtable_walk_peek()
On Fri, May 04, 2018 at 01:54:14PM +1000, NeilBrown wrote: > This function has a somewhat confused behavior that is not properly > described by the documentation. > Sometimes is returns the previous object, sometimes it returns the > next one. > Sometimes it changes the iterator, sometimes it doesn't. > > This function is not currently used and is not worth keeping, so > remove it. > > Signed-off-by: NeilBrownI don't have a problem with this. But it would be good to hear from Tom Herbert who added this. -- Email: Herbert Xu Home Page: http://gondor.apana.org.au/~herbert/ PGP Key: http://gondor.apana.org.au/~herbert/pubkey.txt
Re: [PATCH 5/8] rhashtable: remove rhashtable_walk_peek()
On Fri, May 04, 2018 at 01:54:14PM +1000, NeilBrown wrote: > This function has a somewhat confused behavior that is not properly > described by the documentation. > Sometimes is returns the previous object, sometimes it returns the > next one. > Sometimes it changes the iterator, sometimes it doesn't. > > This function is not currently used and is not worth keeping, so > remove it. > > Signed-off-by: NeilBrown I don't have a problem with this. But it would be good to hear from Tom Herbert who added this. -- Email: Herbert Xu Home Page: http://gondor.apana.org.au/~herbert/ PGP Key: http://gondor.apana.org.au/~herbert/pubkey.txt