On Mon, Apr 24, 2023 at 7:57 PM Simon Horman <[email protected]> wrote: > > I'd slightly prefer if this was at the top of the function. >
Sure. I will move it. On Tue, Apr 25, 2023 at 4:20 AM Ilya Maximets <[email protected]> wrote: > > This should be just ovs_assert(new || old). > Got it. I will substitute this with an assertion instead. > > With ovs_assert(new || old) at the top we should not need this as well. > Sure. > > Same here. > Sure. Best regards, James Raphael Tiovalen _______________________________________________ dev mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
