aboutsummaryrefslogtreecommitdiffstats
path: root/meson.build
diff options
context:
space:
mode:
authorLibravatar Manuel Mendez <mmendez534@gmail.com>2019-11-17 10:50:26 -0500
committerLibravatar Simon Ser <contact@emersion.fr>2019-11-17 17:18:54 +0100
commit8ffa3cf43906fa95c127c01b751590e77c7bf695 (patch)
tree3463efce4786c8cdce92a1e17acbad151346e36d /meson.build
parentUse an enum instead of a marker string for map_to_ (diff)
downloadsway-8ffa3cf43906fa95c127c01b751590e77c7bf695.tar.gz
sway-8ffa3cf43906fa95c127c01b751590e77c7bf695.tar.zst
sway-8ffa3cf43906fa95c127c01b751590e77c7bf695.zip
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 'meson.build')
0 files changed, 0 insertions, 0 deletions