[Spice-devel] [PATCH spice-gtk] Use g_return_val_if_fail instead of wrong g_return_if_fail

Pavel Grunt pgrunt at redhat.com
Mon Jul 13 10:59:35 PDT 2015


The commit 4b5e6ec2114e1250c81027ebeac9cfe8d059153f introduced a function
returning gboolean, g_return_val_if_fail() should be used instead of
g_return_if_fail().
---
 src/channel-main.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/channel-main.c b/src/channel-main.c
index 3bd5a02..1b9c4d4 100644
--- a/src/channel-main.c
+++ b/src/channel-main.c
@@ -1420,7 +1420,7 @@ static gboolean any_display_has_dimensions(SpiceMainChannel *channel)
     SpiceMainChannelPrivate *c;
     guint i;
 
-    g_return_if_fail(SPICE_IS_MAIN_CHANNEL(channel));
+    g_return_val_if_fail(SPICE_IS_MAIN_CHANNEL(channel), FALSE);
     c = channel->priv;
 
     for (i = 0; i < MAX_DISPLAY; i++) {
-- 
2.4.5



More information about the Spice-devel mailing list