Author: adam-guest
Date: 2008-03-17 18:45:40 +0000 (Mon, 17 Mar 2008)
New Revision: 1160

Modified:
   trunk/debian/changelog
   trunk/scripts/checkbashisms.pl
Log:
  + Enhance the "invalid function name" check to allow a leading bracket
    before the function name

Modified: trunk/debian/changelog
===================================================================
--- trunk/debian/changelog      2008-03-17 18:34:13 UTC (rev 1159)
+++ trunk/debian/changelog      2008-03-17 18:45:40 UTC (rev 1160)
@@ -6,6 +6,8 @@
     + Make --version actually produce version information
     + Fix explanation of VAR+=foo
     + Fix an error in the "shell script wrapper" detection
+    + Enhance the "invalid function name" check to allow a leading bracket
+      before the function name
 
  -- Adam D. Barratt <[EMAIL PROTECTED]>  Sun, 16 Mar 2008 19:40:40 +0000
 

Modified: trunk/scripts/checkbashisms.pl
===================================================================
--- trunk/scripts/checkbashisms.pl      2008-03-17 18:34:13 UTC (rev 1159)
+++ trunk/scripts/checkbashisms.pl      2008-03-17 18:45:40 UTC (rev 1160)
@@ -172,7 +172,7 @@
                '(?:^|\s+)unalias\s+-a' =>     q<unalias -a>,
                '(?:^|\s+)local\s+-[a-zA-Z]+' => q<local -opt>,
                '(?:^|\s+)local\s+\w+=' =>     q<local foo=bar>,
-               '(?:^|\s+)\s*\w*[^\w\s]+.*?\(\)' => q<function names should 
only contain [a-z0-9_]>,
+               '(?:^|\s+)\s*\(?\w*[^\(\w\s]+\S*?\s*\(\)' => q<function names 
should only contain [a-z0-9_]>,
            );
 
            my %string_bashisms = (



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

Reply via email to