On Tue, Aug 18, 2015 at 11:43:18AM -0700, Russell Bryant wrote:
> The HMAP_FOR_EACH_()* macros had a usability issue where the iterator
> was only NULL at the completion of iteration if the hmap_node was the
> first struct member.  This change ensures that the iterator is set to
> NULL when iteration ends normally without a 'break'.
> 
> Signed-off-by: Russell Bryant <rbry...@redhat.com>

I applied this, thanks.

Let's try to keep in mind that this could cause nightmarishly subtle
bugs for backporting: if a bug fix takes advantage of this new behavior,
a backport of that bug fix needs to avoid it.
_______________________________________________
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev

Reply via email to