On Wed, Dec 11, 2024 at 11:34 AM Eelco Chaudron <[email protected]> wrote: > > This function, in its current implementation, always returns a > negative value. The issue lies in the calculation: src represents > the original number of elements, while datum->n indicates the new > number of elements. Since datum->n is always greater than or equal > to src, reversing their order resolves the problem, ensuring the > function consistently returns a positive value. > > It’s worth noting that the return value is not currently utilized > in any of the function's use cases > > Signed-off-by: Eelco Chaudron <[email protected]>
This appears to make the function match the comment. Acked-by: Mike Pattrick <[email protected]> _______________________________________________ dev mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
