run_assignments: check for A_COMMAND early

next
Orestis Floros 2018-03-19 03:17:32 +02:00
parent e3e09119bf
commit 6306acdb65
1 changed files with 9 additions and 12 deletions

View File

@ -22,7 +22,7 @@ void run_assignments(i3Window *window) {
/* Check if any assignments match */
Assignment *current;
TAILQ_FOREACH(current, &assignments, assignments) {
if (!match_matches_window(&(current->match), window))
if (current->type != A_COMMAND || !match_matches_window(&(current->match), window))
continue;
bool skip = false;
@ -45,19 +45,16 @@ void run_assignments(i3Window *window) {
window->ran_assignments = srealloc(window->ran_assignments, sizeof(Assignment *) * window->nr_assignments);
window->ran_assignments[window->nr_assignments - 1] = current;
DLOG("matching assignment, would do:\n");
if (current->type == A_COMMAND) {
DLOG("execute command %s\n", current->dest.command);
char *full_command;
sasprintf(&full_command, "[id=\"%d\"] %s", window->id, current->dest.command);
CommandResult *result = parse_command(full_command, NULL);
free(full_command);
DLOG("matching assignment, execute command %s\n", current->dest.command);
char *full_command;
sasprintf(&full_command, "[id=\"%d\"] %s", window->id, current->dest.command);
CommandResult *result = parse_command(full_command, NULL);
free(full_command);
if (result->needs_tree_render)
needs_tree_render = true;
if (result->needs_tree_render)
needs_tree_render = true;
command_result_free(result);
}
command_result_free(result);
}
/* If any of the commands required re-rendering, we will do that now. */