From dc6687b6706de759300b3b7b0383e54f55a741ff Mon Sep 17 00:00:00 2001 From: Michael Jennings Date: Mon, 30 Sep 2002 01:49:20 +0000 Subject: [PATCH] Sun Sep 29 21:48:10 2002 Michael Jennings (mej) Fixed Debian Bug #155960. WM_COMMAND was being set improperly because equals signs in the arg list were being NUL'd out for no reason. SVN revision: 6399 --- ChangeLog | 5 +++++ src/options.c | 10 ++++------ 2 files changed, 9 insertions(+), 6 deletions(-) diff --git a/ChangeLog b/ChangeLog index 418b032..1e8764d 100644 --- a/ChangeLog +++ b/ChangeLog @@ -4794,3 +4794,8 @@ Sun Sep 22 00:34:42 2002 Michael Jennings (mej) Added an option to disable the secondary screen as requested by Marc Merlin . ---------------------------------------------------------------------- +Sun Sep 29 21:48:10 2002 Michael Jennings (mej) + +Fixed Debian Bug #155960. WM_COMMAND was being set improperly because +equals signs in the arg list were being NUL'd out for no reason. +---------------------------------------------------------------------- diff --git a/src/options.c b/src/options.c index 30a3fb7..c82ec28 100644 --- a/src/options.c +++ b/src/options.c @@ -856,7 +856,6 @@ get_options(int argc, char *argv[]) #endif if ((val_ptr = strchr(opt, '=')) != NULL) { - *val_ptr = 0; val_ptr++; hasequal = 1; } else { @@ -875,7 +874,7 @@ get_options(int argc, char *argv[]) CHECK_BAD(); continue; } - if (!strcasecmp(opt, "exec")) { + if (!BEG_STRCASECMP(opt, "exec")) { D_OPTIONS(("--exec option detected\n")); if (!hasequal) { @@ -902,11 +901,11 @@ get_options(int argc, char *argv[]) } rs_exec_args[k] = (char *) NULL; } - } else if (!strcasecmp(opt, "help")) { + } else if (!BEG_STRCASECMP(opt, "help")) { usage(); - } else if (!strcasecmp(opt, "version")) { + } else if (!BEG_STRCASECMP(opt, "version")) { version(); - } else if (!strcasecmp(opt, "attribute")) { + } else if (!BEG_STRCASECMP(opt, "attribute")) { conf_parse_line(NULL, val_ptr); } else { /* It's not --exec */ if (optList[j].flag & OPT_BOOLEAN) { /* Boolean value */ @@ -1083,7 +1082,6 @@ get_initial_options(int argc, char *argv[]) continue; if ((val_ptr = strchr(opt, '=')) != NULL) { - *val_ptr = 0; val_ptr++; hasequal = 1; } else {