Fix the return value of ibus_bus_register_component so it returns TRUE on success.
authorYusuke Sato <yusukes@chromium.org>
Mon, 10 Jan 2011 05:25:07 +0000 (14:25 +0900)
committerYusuke Sato <yusukes@chromium.org>
Mon, 10 Jan 2011 05:25:07 +0000 (14:25 +0900)
Do the same for ibus_bus_set_global_engine as well. Currently it always returns TRUE regardless of the result of the method call.

BUG=none
TEST=manually

Review URL: http://codereview.appspot.com/3902043

src/ibusbus.c

index 32880e7..cca68cd 100644 (file)
@@ -696,9 +696,9 @@ ibus_bus_register_component (IBusBus       *bus,
                                       NULL);
     if (result) {
         g_variant_unref (result);
-        return FALSE;
+        return TRUE;
     }
-    return TRUE;
+    return FALSE;
 }
 
 static GList *
@@ -886,9 +886,9 @@ ibus_bus_set_global_engine (IBusBus     *bus,
 
     if (result) {
         g_variant_unref (result);
+        return TRUE;
     }
-
-    return TRUE;
+    return FALSE;
 }
 
 static GVariant *