]> git.sur5r.net Git - i3/i3/commitdiff
little style fixes for the previous patch
authorMichael Stapelberg <michael@stapelberg.de>
Thu, 4 Aug 2011 19:25:47 +0000 (21:25 +0200)
committerMichael Stapelberg <michael@stapelberg.de>
Thu, 4 Aug 2011 19:25:47 +0000 (21:25 +0200)
src/cfgparse.y

index 226363cce2201d42ff3d484e0ecfe978927073a1..ae5a7a736105a61346c2f5ad4e50ac3930c1521f 100644 (file)
@@ -311,24 +311,27 @@ void kill_configerror_nagbar(bool wait_for_it) {
 }
 
 /*
-    check_for_duplicate_bindings is function looking for duplicated
-    key bindings loaded from configuration file. It goes trought all bindings
-    and tests if exists same key mapping in bindings visited before.
-    If exists, message is printed to "stderr" and error warning is set.
-*/
+ * Checks for duplicate key bindings (the same keycode or keysym is configured
+ * more than once). If a duplicate binding is found, a message is printed to
+ * stderr and the has_errors variable is set to true, which will start
+ * i3-nagbar.
+ *
+ */
 static bool check_for_duplicate_bindings(struct context *context) {
     bool retval = true;
     Binding *bind, *current;
     TAILQ_FOREACH(current, bindings, bindings) {
         bind = TAILQ_FIRST(bindings);
-        // test only bindings visited up to current binding
+        /* test only bindings visited up to current binding */
         while ((bind != TAILQ_END(bindings)) && (bind != current)) {
-            // testing is not case sensitive
-            if ((strcasecmp(bind->symbol, current->symbol) == 0) && (bind->keycode == current->keycode) && (bind->mods == current->mods)) {
+            /* testing is not case sensitive */
+            if ((strcasecmp(bind->symbol, current->symbol) == 0) &&
+                (bind->keycode == current->keycode) &&
+                (bind->mods == current->mods)) {
                 context->has_errors = true;
                 fprintf(stderr, "Duplicated keybinding in config file:  mod%d with key %s", current->mods, current->symbol);
-                // if keycode is 0, it´s not necessary to print it.
-                if(current->keycode != 0)
+                /* if keycode is 0, this is a keysym binding */
+                if (current->keycode != 0)
                     fprintf(stderr, " and keycode %d", current->keycode);
                 fprintf(stderr, "\n");
                 retval = false;