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