Style: Flip the if statement
authorDenis Kenzior <denkenz@gmail.com>
Fri, 18 Dec 2009 03:25:15 +0000 (21:25 -0600)
committerDenis Kenzior <denkenz@gmail.com>
Fri, 18 Dec 2009 03:45:39 +0000 (21:45 -0600)
The if case should have the multiple expressions

src/cbs.c

index 6802dcb..72a380d 100644 (file)
--- a/src/cbs.c
+++ b/src/cbs.c
@@ -437,13 +437,13 @@ static DBusMessage *cbs_set_powered(struct ofono_cbs *cbs, gboolean value,
        if (msg)
                cbs->pending = dbus_message_ref(msg);
 
-       if (!value)
-               cbs->driver->clear_topics(cbs, cbs_set_powered_cb, cbs);
-       else {
+       if (value) {
                topic_str = cbs_topics_to_str(cbs, cbs->topics);
                cbs->driver->set_topics(cbs, topic_str,
                                        cbs_set_powered_cb, cbs);
                g_free(topic_str);
+       } else {
+               cbs->driver->clear_topics(cbs, cbs_set_powered_cb, cbs);
        }
 
        return NULL;