about summary refs log tree commit diff stats
path: root/src/command/cmd_funcs.c
diff options
context:
space:
mode:
authorMichael Vetter <jubalh@iodoru.org>2019-11-08 13:00:31 +0100
committerMichael Vetter <jubalh@iodoru.org>2019-11-08 13:00:31 +0100
commitcbd85ffd1a202f811ce5445f72ca5167e11ad846 (patch)
tree660daad7e2e7088caf94028572679f76d6a4c1da /src/command/cmd_funcs.c
parentb846c4970bb8054ca2bacb8cebdbe4d9302f4cb6 (diff)
downloadprofani-tty-cbd85ffd1a202f811ce5445f72ca5167e11ad846.tar.gz
cmd_room: don't check arguments twice
Diffstat (limited to 'src/command/cmd_funcs.c')
-rw-r--r--src/command/cmd_funcs.c17
1 files changed, 4 insertions, 13 deletions
diff --git a/src/command/cmd_funcs.c b/src/command/cmd_funcs.c
index 4e3314ac..4a78d415 100644
--- a/src/command/cmd_funcs.c
+++ b/src/command/cmd_funcs.c
@@ -4329,13 +4329,6 @@ cmd_room(ProfWin *window, const char *const command, gchar **args)
         return TRUE;
     }
 
-    if ((g_strcmp0(args[0], "accept") != 0) &&
-            (g_strcmp0(args[0], "destroy") != 0) &&
-            (g_strcmp0(args[0], "config") != 0)) {
-        cons_bad_cmd_usage(command);
-        return TRUE;
-    }
-
     ProfMucWin *mucwin = (ProfMucWin*)window;
     assert(mucwin->memcheck == PROFMUCWIN_MEMCHECK);
 
@@ -4350,14 +4343,10 @@ cmd_room(ProfWin *window, const char *const command, gchar **args)
             win_println(window, THEME_ROOMINFO, '!', "Room unlocked.");
             return TRUE;
         }
-    }
-
-    if (g_strcmp0(args[0], "destroy") == 0) {
+    } else if (g_strcmp0(args[0], "destroy") == 0) {
         iq_destroy_room(mucwin->roomjid);
         return TRUE;
-    }
-
-    if (g_strcmp0(args[0], "config") == 0) {
+    } else if (g_strcmp0(args[0], "config") == 0) {
         ProfConfWin *confwin = wins_get_conf(mucwin->roomjid);
 
         if (confwin) {
@@ -4366,6 +4355,8 @@ cmd_room(ProfWin *window, const char *const command, gchar **args)
             iq_request_room_config_form(mucwin->roomjid);
         }
         return TRUE;
+    } else {
+        cons_bad_cmd_usage(command);
     }
 
     return TRUE;