[hackers] [dwm] setlayout should perform strcmp's if arg != NULL, because Layout is local to screen.o

From: Anselm R. Garbe <garbeam_AT_gmail.com>
Date: Wed Aug 22 19:06:53 2007

changeset: 973:98750b173cd5
tag: tip
user: Anselm R. Garbe <garbeam_AT_gmail.com>
date: Wed Aug 22 19:06:35 2007 +0200
summary: setlayout should perform strcmp's if arg != NULL, because Layout is local to screen.o

diff -r c39d60829f67 -r 98750b173cd5 screen.c
--- a/screen.c Wed Aug 22 19:02:17 2007 +0200
+++ b/screen.c Wed Aug 22 19:06:35 2007 +0200
@@ -3,6 +3,7 @@
 #include <regex.h>
 #include <stdio.h>
 #include <stdlib.h>
+#include <string.h>
 #include <X11/Xutil.h>
 
 /* static */
@@ -245,7 +246,7 @@ setlayout(const char *arg) {
         }
         else {
                 for(i = 0; i < nlayouts; i++)
- if(arg == layouts[i].symbol)
+ if(!strcmp(arg, layouts[i].symbol))
                                 break;
                 if(i == nlayouts)
                         return;
Received on Wed Aug 22 2007 - 19:06:53 UTC

This archive was generated by hypermail 2.2.0 : Sun Jul 13 2008 - 15:58:00 UTC