[ovs-dev] [PATCH 2/2] lib/util, tests: Support NDEBUG.

Jarno Rajahalme jrajahalme at nicira.com
Thu Oct 30 16:19:58 UTC 2014


On Oct 30, 2014, at 8:43 AM, Ben Pfaff <blp at nicira.com> wrote:

> On Wed, Oct 29, 2014 at 11:41:48AM -0700, Jarno Rajahalme wrote:
>> ./configure accepts --enable-ndebug option.  Make ovs_assert() honor
>> it, and make sure all test programs disable it.
>> 
>> The order of include files in test programs is also made uniform:
>> 
>> 1. #include <config.h>
>> 2. #undef NDEBUG
>> 3. Include file of the test subject (to make sure it itself has
>>   sufficient include directives).
>> 4. System includes in alphapetical order.
>> 5. OVS includes in aplhapetical order.
>> 
>> Suggested-by: Ben Pfaff <blp at nicira.com>
>> Signed-off-by: Jarno Rajahalme <jrajahalme at nicira.com>
> 
> Acked-by: Ben Pfaff <blp at nicira.com>

Thanks for the review.

Pushed to master,

  Jarno



More information about the dev mailing list