Control: reassign -1 libglib2.0-dev 2.38.2-5

On Sat, Mar 08, 2014 at 11:30:45PM +0100, Nicolas Sévelin-Radiguet wrote:
> I'm building packages with clang instead of gcc. Some packages depending on 
> glib fail to build with -Wreturn-type errors.
> See the following log excerpt:
> 
>         g_return_if_fail(widget != NULL);
>         ^
> /usr/include/glib-2.0/glib/gmessages.h:345:3: note: expanded from macro 
> 'g_return_if_fail'
>          return; \
>          ^

This seems like the fault of the packages using GLib.  Each
g_return_if_* macro has a g_return_val_if_* sibling, which you're
supposed to use in non-void contexts.  Adding return values to the
g_return_if_* macros as your patch does would lead to many packages
attempting to return values from void functions, which would also be
wrong.

-- 
Colin Watson                                       [[email protected]]


-- 
To UNSUBSCRIBE, email to [email protected]
with a subject of "unsubscribe". Trouble? Contact [email protected]

Reply via email to