4

SC2219

Jari Aalto edited this page 2025-01-30 01:27:20 +02: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. For portability, the $(( expr )) syntax is defined in POSIX standard.

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.