[ovs-dev] [PATCH 4/5] lib/util.h: Disable ovs_assert when build with NDEBUG

Ben Pfaff blp at nicira.com
Fri Dec 13 17:08:25 UTC 2013


On Fri, Dec 13, 2013 at 02:17:16PM +0100, Helmut Schaa wrote:
> Reduces binary size. Use a static inline function instead of
> a macro to not get "unused variable" warning everywhere.
> 
> Signed-off-by: Helmut Schaa <helmut.schaa at googlemail.com>
> ---
> 
> Not sure if this is the best idea but in my builds it reduces libopenvswitch
> by around 10%.

I guess I'm OK with this, but please update the comment, which currently
says "Not affected by NDEBUG."

If you want to reduce size as much as possible, it might be better to
update the callers to avoid those warnings.



More information about the dev mailing list