[Gluster-devel] Query regarding GF_ASSERT macro

Harshavardhana harsha at harshavardhana.net
Wed Jan 8 06:11:33 UTC 2014


>
> I think we need to do the following:
>
> 1. Change GF_ASSERT to be a simple wrapper over assert() and get rid of the
> -DDEBUG dependency.
>
> 2. Add -DNDEBUG for GA releases. qa releases to be built with assert()
> enabled.
>
> 3. Change GF_ASSERT to GF_VALIDATE_xxx wherever it is being used defensively
> and not as an assertion.
>
> Might be a good idea to log a bug and track this activity if everybody is
> fine with the guidelines.

+1 from me

-- 
Religious confuse piety with mere ritual, the virtuous confuse
regulation with outcomes




More information about the Gluster-devel mailing list