nixo
/
gri3-wm
1
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

Ensure config variables match on longest-length (#2306) 

fixes #2235
This commit is contained in:
Kyle Kneitinger 2016-04-26 00:20:42 -07:00 committed by Michael Stapelberg
parent 23beac46b7
commit 2123888d4d
2 changed files with 31 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

14
src/config_parser.c
View File

@ -879,9 +879,23 @@ bool parse_file(const char *f, bool use_nagbar) {
v_value++; v_value++;
struct Variable *new = scalloc(1, sizeof(struct Variable)); struct Variable *new = scalloc(1, sizeof(struct Variable));
struct Variable *test = NULL, *loc = NULL;
new->key = sstrdup(v_key); new->key = sstrdup(v_key);
new->value = sstrdup(v_value); new->value = sstrdup(v_value);
/* ensure that the correct variable is matched in case of one being
* the prefix of another */
SLIST_FOREACH(test, &variables, variables) {
if (strlen(new->key) >= strlen(test->key))
break;
loc = test;
}
if (loc == NULL) {
SLIST_INSERT_HEAD(&variables, new, variables); SLIST_INSERT_HEAD(&variables, new, variables);
} else {
SLIST_INSERT_AFTER(loc, new, variables);
}
DLOG("Got new variable %s = %s\n", v_key, v_value); DLOG("Got new variable %s = %s\n", v_key, v_value);
continue; continue;
} }

16
testcases/t/183-config-variables.t
View File

@ -79,6 +79,22 @@ EOT
is(launch_get_border($config), 'none', 'no border'); is(launch_get_border($config), 'none', 'no border');
#####################################################################
# test that longest matching variable name is substituted
#####################################################################
$config = <<'EOT';
# i3 config file (v4)
font -misc-fixed-medium-r-normal--13-120-75-75-C-70-iso10646-1
set $var normal title
set $vartest special title
set $vart mundane title
for_window [title="$vartest"] border none
EOT
is(launch_get_border($config), 'none', 'no border');
done_testing; done_testing;