Re: checks/scripts / checkbashsims updates
"Adam D. Barratt" <adam@adam-barratt.org.uk> writes:
> Sorry to be a pain, but we've tightened this further in svn to
>
> '(?<![\$\(])\(\(.*\)\)'
>
> so as not to match constructs such as
>
> progress_size=$(((100 - $PROGRESS_STATE) / 3))
What *is* this construct trying to catch, anyway? dash supports things
like $((4 + 5)), and if the above is also okay, I'm not sure when this
would not be a false positive.
--
Russ Allbery (rra@debian.org) <http://www.eyrie.org/~eagle/>
Reply to: