From 077389777e98e858e6863b72ba214a3438232e93 Mon Sep 17 00:00:00 2001 From: Vidar Holen Date: Tue, 1 Sep 2020 17:13:24 -0700 Subject: [PATCH] Created SC3018 (markdown) --- SC3018.md | 38 ++++++++++++++++++++++++++++++++++++++ 1 file changed, 38 insertions(+) create mode 100644 SC3018.md diff --git a/SC3018.md b/SC3018.md new file mode 100644 index 0000000..03c9477 --- /dev/null +++ b/SC3018.md @@ -0,0 +1,38 @@ +## In POSIX sh, ++ is undefined. + +(or "In dash, ... is not supported." when using `dash`) + +### Problematic code: + +```sh +#!/bin/sh +i=1 +echo "$((i++))" +``` + +### Correct code: + +Write out the increment/decrement operation explicitly: + +```sh +#!/bin/sh +i=1 +echo "$i" +i=$((i+1)) +``` + +### Rationale: + +Prefix and postfix increment and decrement are extensions in bash and ksh. They're not supported in dash or POSIX sh, which is what your shebang says you're using. + +### Exceptions: + +If you only intend to target shells that supports this feature, you can change +the shebang to a shell that guarantees support, or [[ignore]] this warning. + +You can use `# shellcheck disable=SC3000-SC4000` to ignore all such compatibility +warnings. + +### Related resources: + +* Help by adding links to BashFAQ, StackOverflow, man pages, POSIX, etc!