Changeset 4022Tweet

Timestamp:

Nov 23, 2009, 2:09:54 PM (11 years ago)

Author:

Pascal Terjan

Message:

• Factorize that a boolean option is true when it is "true" or "1"

File:

• neercs/trunk/src/configuration.c (modified) (6 diffs)

neercs/trunk/src/configuration.c

@@ -367 +367 @@
 
 #define IS_OPTION(t) (!strncmp(argv, t, strlen(argv)))
+#define IS_OPTION_TRUE (IS_OPTION("true") || IS_OPTION("1"))
+
 int set_window_manager(const char *argv, struct screen_list *screen_list)
 {
@@ -393 +395 @@
 int set_thumbnails(const char *argv, struct screen_list *screen_list)
 {
-    if (IS_OPTION("true") || IS_OPTION("1"))
+    if (IS_OPTION_TRUE)
         screen_list->mini = 1;
     else
@@ -403 +405 @@
 int set_status_bar(const char *argv, struct screen_list *screen_list)
 {
-    if (IS_OPTION("true") || IS_OPTION("1"))
+    if (IS_OPTION_TRUE)
         screen_list->status = 1;
     else
@@ -434 +436 @@
 int set_lock_on_detach(const char *argv, struct screen_list *screen_list)
 {
-    if (IS_OPTION("true") || IS_OPTION("1"))
+    if (IS_OPTION_TRUE)
         screen_list->lock_on_detach = 1;
     else
@@ -443 +445 @@
 int set_eyecandy(const char *argv, struct screen_list *screen_list)
 {
-    if (IS_OPTION("true") || IS_OPTION("1"))
+    if (IS_OPTION_TRUE)
         screen_list->eyecandy = 1;
     else
@@ -452 +454 @@
 int set_border(const char *argv, struct screen_list *screen_list)
 {
-    if (IS_OPTION("true") || IS_OPTION("1"))
+    if (IS_OPTION_TRUE)
         screen_list->border_size = 1;
     else