7x31/shellcheck
1
0
Fork 0
mirror of https://github.com/koalaman/shellcheck.git synced 2025-10-03 19:29:44 +08:00
Code Issues Packages Projects Releases Wiki Activity

Updated SC2233 (markdown)

Vidar Holen
2021-08-30 11:35:37 -07:00
parent 902ce9055a
commit 22e90bda5e
1 changed files with 4 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
SC2233.md

@@ -1,4 +1,4 @@
## Remove superfluous `(..)` around condition.
## Remove superfluous `(..)` around condition to avoid subshell overhead.
### Problematic code:
@@ -36,7 +36,9 @@ Just delete the surrounding `(..)` since they serve no purpose and only slows th
### Exceptions:
None.
This issue only affects performance, not correctness, so it can be safely [[ignore]]d.
If you are considering doing it to stylistically match C-like languages, please note that this is not conventional and that you'd probably recommend someone use `if (1 == 2)` over `if (system("[ 1 = 2 ]"))` in C no matter which language they're used to.
### Related resources: