[PATCH 0/4] BUG_RETURN_VAL macros

Chase Douglas chase.douglas at canonical.com
Tue May 15 11:11:25 PDT 2012


On 05/15/2012 11:06 AM, Mark Kettenis wrote:
>> From: Peter Hutterer <peter.hutterer at who-t.net>
>> Date: Tue, 15 May 2012 20:26:37 +1000
>>
>> I got annoyed having to write constructs like
>>
>> BUG_WARN(foo);
>> if (foo)
>>     return FALSE;
>>
>> and similar. glib has useful macros like g_return_if_fail and similar, these
>> are macros that essentially do the same job. They shout into the log, but
>> otherwise continue as normal.
>>
>> http://developer.gnome.org/glib/2.29/glib-Warnings-and-Assertions.html#g-return-if-fail
>>
>> These are not macros that should be used for handling normal out-of-scope
>> values, they're there to shout that there is a real bug that needs fixing.
> 
> Still I think I agree with whoever said that hiding control flow
> instructions behind a macro isn't a good idea.

I understand your viewpoint, but I don't feel the same way. I think it
is obvious enough, based on the name of the macro, what is going on.

I won't be terribly annoyed if these changes aren't merged, but I wanted
to go on the record to say that I prefer them.

-- Chase


More information about the xorg-devel mailing list