Merge pull request #3189 from orestisf1993/run_assignments
run_assignments: check for A_COMMAND early
This commit is contained in:
commit
670dfa0bba
|
@ -22,7 +22,7 @@ void run_assignments(i3Window *window) {
|
||||||
/* Check if any assignments match */
|
/* Check if any assignments match */
|
||||||
Assignment *current;
|
Assignment *current;
|
||||||
TAILQ_FOREACH(current, &assignments, assignments) {
|
TAILQ_FOREACH(current, &assignments, assignments) {
|
||||||
if (!match_matches_window(&(current->match), window))
|
if (current->type != A_COMMAND || !match_matches_window(&(current->match), window))
|
||||||
continue;
|
continue;
|
||||||
|
|
||||||
bool skip = false;
|
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 = srealloc(window->ran_assignments, sizeof(Assignment *) * window->nr_assignments);
|
||||||
window->ran_assignments[window->nr_assignments - 1] = current;
|
window->ran_assignments[window->nr_assignments - 1] = current;
|
||||||
|
|
||||||
DLOG("matching assignment, would do:\n");
|
DLOG("matching assignment, execute command %s\n", current->dest.command);
|
||||||
if (current->type == A_COMMAND) {
|
char *full_command;
|
||||||
DLOG("execute command %s\n", current->dest.command);
|
sasprintf(&full_command, "[id=\"%d\"] %s", window->id, current->dest.command);
|
||||||
char *full_command;
|
CommandResult *result = parse_command(full_command, NULL);
|
||||||
sasprintf(&full_command, "[id=\"%d\"] %s", window->id, current->dest.command);
|
free(full_command);
|
||||||
CommandResult *result = parse_command(full_command, NULL);
|
|
||||||
free(full_command);
|
|
||||||
|
|
||||||
if (result->needs_tree_render)
|
if (result->needs_tree_render)
|
||||||
needs_tree_render = true;
|
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. */
|
/* If any of the commands required re-rendering, we will do that now. */
|
||||||
|
|
Loading…
Reference in New Issue