[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