Eric Garver <e...@erig.me> writes:

> We need to use == instead of the is operator. If you're unlucky it may
> fail because they're not exactly the same object, but hold the same
> value.
>
> Example false positive:
>
> E(120): Inappropriate bracing around statement
>
> +            if (0 != nl_attr_get_u8(vxlan[IFLA_VXLAN_LEARNING])

I count an unbalanced brace.  There may be something wrong, but at least
from the example, I don't see it.

Can you include a bit more context?  Perhaps the hunk that failed?

-Aaron
_______________________________________________
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Reply via email to