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 SC2170 (markdown)

Vidar Holen
2021-08-30 20:01:27 -07:00
parent c0a516faf3
commit 86c41d5cb2
1 changed files with 3 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files

8
SC2170.md

@@ -31,13 +31,11 @@ fi
``` ```
### Rationale: ### Rationale:
You are comparing a string value with a numerical operator, such as `-eq`, `-ne`, `-lt` or `-gt`. You are comparing a string value with a numerical operator, such as `-eq`, `-ne`, `-lt` or `-gt`. These only work for numbers.
In `[[ .. ]]`, this would automatically dereference the string, looking to see if there are variables by that name. If you want to compare the value as a string, switch to the equivalent string operator: `=`, `!=` `\<` or `\>`.
In `[ .. ]`, which you are using, the string is just treated as an invalid number. If you want to compare it as a number, such as `n=42; while [ n -gt 1024/8 ]; ..`, then keep the operator and expand the operands yourself with `$var` or `$((expr))`: `while [ "$n" -gt $((1024/8)) ]`
If you want to compare numbers, expand yourself (e.g. use `$var` instead of `var`, or `$((n+1))` instead of `n+1`). If you are trying to compare strings and not numbers, use `=`, `!=` `\<` or `\>` instead.
### Exceptions: ### Exceptions: