diff options
author | 2019-11-17 10:50:26 -0500 | |
---|---|---|
committer | 2019-11-17 17:18:54 +0100 | |
commit | 8ffa3cf43906fa95c127c01b751590e77c7bf695 (patch) | |
tree | 3463efce4786c8cdce92a1e17acbad151346e36d /README.de.md | |
parent | Use an enum instead of a marker string for map_to_ (diff) | |
download | sway-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 'README.de.md')
0 files changed, 0 insertions, 0 deletions