diff --git a/ChangeLog b/ChangeLog index 4adb44286..2849bca2a 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,3 +1,12 @@ +Mon Nov 8 10:45:50 2004 Manish Singh + + * glib/gasyncqueue.c + * glib/ghook.c + * glib/giochannel.c: g_return_if_fail -> g_return_val_if_fail + + * glib/gmain.c: Ditto, plus also make g_main_context_ref() actually + return the passed in pointer. + 2004-11-08 Matthias Clasen * glib/gasyncqueue.[hc]: diff --git a/ChangeLog.pre-2-10 b/ChangeLog.pre-2-10 index 4adb44286..2849bca2a 100644 --- a/ChangeLog.pre-2-10 +++ b/ChangeLog.pre-2-10 @@ -1,3 +1,12 @@ +Mon Nov 8 10:45:50 2004 Manish Singh + + * glib/gasyncqueue.c + * glib/ghook.c + * glib/giochannel.c: g_return_if_fail -> g_return_val_if_fail + + * glib/gmain.c: Ditto, plus also make g_main_context_ref() actually + return the passed in pointer. + 2004-11-08 Matthias Clasen * glib/gasyncqueue.[hc]: diff --git a/ChangeLog.pre-2-12 b/ChangeLog.pre-2-12 index 4adb44286..2849bca2a 100644 --- a/ChangeLog.pre-2-12 +++ b/ChangeLog.pre-2-12 @@ -1,3 +1,12 @@ +Mon Nov 8 10:45:50 2004 Manish Singh + + * glib/gasyncqueue.c + * glib/ghook.c + * glib/giochannel.c: g_return_if_fail -> g_return_val_if_fail + + * glib/gmain.c: Ditto, plus also make g_main_context_ref() actually + return the passed in pointer. + 2004-11-08 Matthias Clasen * glib/gasyncqueue.[hc]: diff --git a/ChangeLog.pre-2-6 b/ChangeLog.pre-2-6 index 4adb44286..2849bca2a 100644 --- a/ChangeLog.pre-2-6 +++ b/ChangeLog.pre-2-6 @@ -1,3 +1,12 @@ +Mon Nov 8 10:45:50 2004 Manish Singh + + * glib/gasyncqueue.c + * glib/ghook.c + * glib/giochannel.c: g_return_if_fail -> g_return_val_if_fail + + * glib/gmain.c: Ditto, plus also make g_main_context_ref() actually + return the passed in pointer. + 2004-11-08 Matthias Clasen * glib/gasyncqueue.[hc]: diff --git a/ChangeLog.pre-2-8 b/ChangeLog.pre-2-8 index 4adb44286..2849bca2a 100644 --- a/ChangeLog.pre-2-8 +++ b/ChangeLog.pre-2-8 @@ -1,3 +1,12 @@ +Mon Nov 8 10:45:50 2004 Manish Singh + + * glib/gasyncqueue.c + * glib/ghook.c + * glib/giochannel.c: g_return_if_fail -> g_return_val_if_fail + + * glib/gmain.c: Ditto, plus also make g_main_context_ref() actually + return the passed in pointer. + 2004-11-08 Matthias Clasen * glib/gasyncqueue.[hc]: diff --git a/glib/gasyncqueue.c b/glib/gasyncqueue.c index 6822f7dee..6efe379fd 100644 --- a/glib/gasyncqueue.c +++ b/glib/gasyncqueue.c @@ -70,8 +70,8 @@ g_async_queue_new (void) GAsyncQueue * g_async_queue_ref (GAsyncQueue *queue) { - g_return_if_fail (queue); - g_return_if_fail (g_atomic_int_get (&queue->ref_count) > 0); + g_return_val_if_fail (queue, NULL); + g_return_val_if_fail (g_atomic_int_get (&queue->ref_count) > 0, NULL); g_atomic_int_inc (&queue->ref_count); diff --git a/glib/ghook.c b/glib/ghook.c index 83ef80979..b01b83074 100644 --- a/glib/ghook.c +++ b/glib/ghook.c @@ -226,9 +226,9 @@ GHook * g_hook_ref (GHookList *hook_list, GHook *hook) { - g_return_if_fail (hook_list != NULL); - g_return_if_fail (hook != NULL); - g_return_if_fail (hook->ref_count > 0); + g_return_val_if_fail (hook_list != NULL, NULL); + g_return_val_if_fail (hook != NULL, NULL); + g_return_val_if_fail (hook->ref_count > 0, NULL); hook->ref_count++; diff --git a/glib/giochannel.c b/glib/giochannel.c index c911a2687..bb0e9d6a4 100644 --- a/glib/giochannel.c +++ b/glib/giochannel.c @@ -94,7 +94,7 @@ g_io_channel_init (GIOChannel *channel) GIOChannel * g_io_channel_ref (GIOChannel *channel) { - g_return_if_fail (channel != NULL); + g_return_val_if_fail (channel != NULL, NULL); channel->ref_count++; diff --git a/glib/gmain.c b/glib/gmain.c index 7ee0b1a05..a9382aebe 100644 --- a/glib/gmain.c +++ b/glib/gmain.c @@ -595,10 +595,12 @@ g_poll (GPollFD *fds, GMainContext * g_main_context_ref (GMainContext *context) { - g_return_if_fail (context != NULL); - g_return_if_fail (g_atomic_int_get (&context->ref_count) > 0); + g_return_val_if_fail (context != NULL, NULL); + g_return_val_if_fail (g_atomic_int_get (&context->ref_count) > 0, NULL); g_atomic_int_inc (&context->ref_count); + + return context; } /* If DISABLE_MEM_POOLS is defined, then freeing the