aboutsummaryrefslogtreecommitdiff
path: root/sway/desktop/idle_inhibit_v1.c
diff options
context:
space:
mode:
authorManuel Mendez <mmendez534@gmail.com>2019-11-17 10:50:26 -0500
committerSimon Ser <contact@emersion.fr>2019-11-17 17:18:54 +0100
commit8ffa3cf43906fa95c127c01b751590e77c7bf695 (patch)
tree3463efce4786c8cdce92a1e17acbad151346e36d /sway/desktop/idle_inhibit_v1.c
parent7f54495b5e62a65627cdb61f099e241f0594a6d9 (diff)
grimshot: fix branching on command exit status
The previous behavior was incorrect because `if` was checking the return status of the `[` command which was never going to be an error. `[` seems to only return an error if no args are provided. This was basically a useless use of `[` anyway since it was just meant as a straight interpretation of command exit, something that `if` can do itself. Compare: ```sh [ ]; echo ?=$? [ /bin/false ]; echo ?=$? if [ /bin/false ]; then echo this is the unintended bug; fi if /bin/false; then echo this will not be printed; fi ```
Diffstat (limited to 'sway/desktop/idle_inhibit_v1.c')
0 files changed, 0 insertions, 0 deletions