[parted-devel] [PATCH] maint: make PED_ASSERT and PED_DEBUG	if-safe
    Jim Meyering 
    jim at meyering.net
       
    Wed Apr 21 12:04:26 UTC 2010
    
    
  
Colin Watson wrote:
> Constructions such as:
>
>   if (foo)
>     PED_DEBUG (0, "foo");
>   else
>     PED_DEBUG (0, "not foo");
>
> did not compile reliably, depending on the configuration.  Make this
> work.
>
> * include/parted/debug.h [DEBUG && __GNUC__ && !__JSFTRACE__]
> (PED_DEBUG): Remove trailing semicolon.
> [!DEBUG] (PED_ASSERT): Use do-while rather than while.
> [!DEBUG] (PED_DEBUG): Likewise.
> ---
Good fix/clean-up.  Thank you.
Pushed.
    
    
More information about the parted-devel
mailing list