Created SC2212 (markdown)

koalaman
2017-04-16 18:36:34 -07:00
parent 895df4c284
commit b6c9a4f2f5

28
SC2212.md Normal file

@@ -0,0 +1,28 @@
## Use 'false' instead of empty [/[[ conditionals.
### Problematic code:
```sh
if [ ]
then
echo "Temporarily disabled"
fi
```
### Correct code:
```sh
if false
then
echo "Temporarily disabled"
fi
```
### Rationale:
`[ ]` is a somewhat obscure way of expressing falsehood, and the behavior is likely intended to allow the incorrectly quoted command `[ $var ]` to still work when the variable is unset.
POSIX has a more descriptive command `false` for this.
### Exceptions:
None. This is a stylistic suggestion, and has no effect on how the script works.