[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