Author: adam-guest
Date: 2008-05-12 22:38:03 +0000 (Mon, 12 May 2008)
New Revision: 1445

Modified:
   trunk/debian/changelog
   trunk/scripts/checkbashisms.pl
Log:
Add a check for "$[" (a deprecated form of arithmetic expansion)

Modified: trunk/debian/changelog
===================================================================
--- trunk/debian/changelog      2008-05-11 20:55:09 UTC (rev 1444)
+++ trunk/debian/changelog      2008-05-12 22:38:03 UTC (rev 1445)
@@ -18,6 +18,7 @@
     + Don't flag "time)" as matching "time"
     + Apply various improvements to "quoted block" detection
     + Correctly match 'echo "$(< $0)"'
+    + Add a check for "$[" (a deprecated form of arithmetic expansion)
   * cowpoke: New script to upload a Debian source package to a cowbuilder host
     and build it, optionally also signing and uploading the result to an
     incoming queue (Closes: #479274)

Modified: trunk/scripts/checkbashisms.pl
===================================================================
--- trunk/scripts/checkbashisms.pl      2008-05-11 20:55:09 UTC (rev 1444)
+++ trunk/scripts/checkbashisms.pl      2008-05-12 22:38:03 UTC (rev 1445)
@@ -386,6 +386,7 @@
        $LEADIN . 'exec\s+-[acl]' =>    q<exec -c/-l/-a name>,
        $LEADIN . 'let\s' =>            q<let ...>,
        '(?<![\$\(])\(\(.*\)\)' =>     q<'((' should be '$(('>,
+       '\$\[' =>                      q<'$[' should be '$(('>,
        '(\[|test)\s+-a' =>            q<test with unary -a (should be -e)>,
        '\&>' =>                       q<should be \>word 2\>&1>,
        '(<\&|>\&)\s*((-|\d+)[^\s;|)`&\\\\]|[^-\d\s])' =>



-- 
To unsubscribe, send mail to [EMAIL PROTECTED]

Reply via email to