[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