[augeas-devel] [PATCH 3 of 6] Suppress assert of hash_verify

David Lutterkort dlutter at redhat.com
Tue May 6 18:07:02 UTC 2008


On Tue, 2008-05-06 at 08:11 +0200, Jim Meyering wrote:
> Rather than all of those ugly in-function #ifdefs, how about adding
> only say expensive_assert (hash_verify(hash)); statements and
> putting this at the top, after the declaration of the function?
> 
> #ifdef HASH_DEBUG_VERIFY
> # define expensive_assert(expr) assert (expr)
> #else
> # define expensive_assert(expr) /* empty */
> #endif

Yes, that is much cleaner; I just committed that patch.

David





More information about the augeas-devel mailing list