2 SC2219

S0AndS0 edited this page 2020-11-02 01:20:13 +00:00

Table of Contents

• Instead of let expr, prefer (( expr )) .
• Problematic code:
• Correct code:
• Rationale:
• Exceptions:
• More information

Instead of let expr, prefer (( expr )) .

Problematic code:

let a++

Correct code:

(( a++ )) || true

Note, || true bits ignore error status code when incrementing from 0 to 1

Rationale:

The (( .. )) arithmetic compound command evaluates expressions in the same way as let, except it's not subject to glob expansion and therefore requires no additional quoting or escaping.

This warning only triggers in Bash/Ksh scripts. In Sh/Dash, neither let nor (( .. )) are defined, but can be simulated with [ $(( expr )) -ne 0 ] to retain exit code, or : $(( expr )) to ignore it.

Exceptions:

None.

More information

• Bash Hacker's Wiki: The let builtin command

ShellCheck

• Installation

  • CentOS6
  • More Installation Guides

• Usage

  • Directive
  • Ignore

• Integrating and extending

  • Misc. contributions
  • Azure Pipelines
  • Buildkite
  • CircleCI
  • DevGuide
  • Integration
  • JUnit
  • Phabricator
  • TravisCI
  • GitLab CI
  • Drone
  • GitHub Actions

• Severity

• Template

Each individual ShellCheck warning has its own wiki page like SC1000. Use GitHub Wiki's "Pages" feature above to find a specific one, or see Checks.