Reduce switch case nesting
authorSiraaj Khandkar <siraaj@khandkar.net>
Sun, 15 Mar 2020 02:19:47 +0000 (22:19 -0400)
committerSiraaj Khandkar <siraaj@khandkar.net>
Sun, 15 Mar 2020 02:19:47 +0000 (22:19 -0400)
x5/khatus.c

index 3d81cad..d95390a 100644 (file)
@@ -210,24 +210,24 @@ void
 parse_opts_opt(Config *cfg, int argc, char *argv[], int i)
 {
        switch (argv[i][1]) {
-               case 'i':
-                       /* TODO: Generic set_int */
-                       parse_opts_opt_i(cfg, argc, argv, ++i);
-                       break;
-               case 's':
-                       /* TODO: Generic set_str */
-                       parse_opts_opt_s(cfg, argc, argv, ++i);
-                       break;
-               case 'x':
-                       cfg->output_to_x_root_window = 1;
-                       opts_parse_any(cfg, argc, argv, ++i);
-                       break;
-               case 'l':
-                       /* TODO: Generic set_int */
-                       parse_opts_opt_l(cfg, argc, argv, ++i);
-                       break;
-               default :
-                       usage("Option \"%s\" is invalid\n", argv[i]);
+       case 'i':
+               /* TODO: Generic set_int */
+               parse_opts_opt_i(cfg, argc, argv, ++i);
+               break;
+       case 's':
+               /* TODO: Generic set_str */
+               parse_opts_opt_s(cfg, argc, argv, ++i);
+               break;
+       case 'x':
+               cfg->output_to_x_root_window = 1;
+               opts_parse_any(cfg, argc, argv, ++i);
+               break;
+       case 'l':
+               /* TODO: Generic set_int */
+               parse_opts_opt_l(cfg, argc, argv, ++i);
+               break;
+       default :
+               usage("Option \"%s\" is invalid\n", argv[i]);
        }
 }
 
@@ -269,11 +269,11 @@ opts_parse_any(Config *cfg, int argc, char *argv[], int i)
 {
        if (i < argc) {
                switch (argv[i][0]) {
-                       case '-':
-                               parse_opts_opt(cfg, argc, argv, i);
-                               break;
-                       default :
-                               parse_opts_spec(cfg, argc, argv, i);
+               case '-':
+                       parse_opts_opt(cfg, argc, argv, i);
+                       break;
+               default :
+                       parse_opts_spec(cfg, argc, argv, i);
                }
        }
 }
This page took 0.022711 seconds and 4 git commands to generate.