On 2/13/2019 6:25 PM, Flavio Leitner wrote:
On Mon, Nov 12, 2018 at 12:28:39PM +0300, Ilya Maximets wrote:
* Dropped 'is_all_zero' function, which is equal to 'is_all_zeros'
from util.h .
* util.h added to includes. Includes re-sorted within their blocks.
(it's hard to figure out where to put new one if there is no order.)
Note: linux/if.h depends on sys/socket.h .
* 'ovs_u128_is_zero' used instead of manual checking of fields.
* Code simplified by using direct pointer to 'match->wc.masks'.
* 'sizeof's rewritten to be coding-style complient.
Signed-off-by: Ilya Maximets <[email protected]>
---
LGTM
Acked-by: Flavio Leitner <[email protected]>
Thanks for this Ilya, got around to looking at it, LGTM too. Pushed to
master.
Ian
_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev
