On Tue, Apr 25, 2023 at 4:17 AM Ilya Maximets <[email protected]> wrote:
>
> shash_is_empty(shash) is the same as hmap_count(&shash->map) == 0.
> There should be no need to check, since we're returning the result
> of hmap_count() below anyway.
>
> Same for other structures.  And these should, likely, be just assertions.
>

Got it. In that case, I can just `ovs_assert()` that `shash`, `simap`,
and `smap` are not NULL.

Best regards,
James Raphael Tiovalen
_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Reply via email to