Author: adam-guest
Date: 2008-03-10 22:18:19 +0000 (Mon, 10 Mar 2008)
New Revision: 1120

Modified:
   trunk/debian/changelog
   trunk/scripts/checkbashisms.pl
Log:
  + Add a check for function names containing characters other than
    alphanumerics and underscore

Modified: trunk/debian/changelog
===================================================================
--- trunk/debian/changelog      2008-03-10 21:27:12 UTC (rev 1119)
+++ trunk/debian/changelog      2008-03-10 22:18:19 UTC (rev 1120)
@@ -32,6 +32,8 @@
       source passing arguments rather than misuse of $HOSTNAME
     + Fix a potential false positive on "2>& 1"
     + Fix an unescaped hyphen in the manpage
+    + Add a check for function names containing characters other than
+      alphanumerics and underscore
   * bts: Allow the sendmail command to begin with a ~ (Closes: #469207)
   * debcommit: Set $SVKDIFF before calling "svk diff" (Closes: #469238)
   * deb[ci]:

Modified: trunk/scripts/checkbashisms.pl
===================================================================
--- trunk/scripts/checkbashisms.pl      2008-03-10 21:27:12 UTC (rev 1119)
+++ trunk/scripts/checkbashisms.pl      2008-03-10 22:18:19 UTC (rev 1120)
@@ -157,6 +157,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+)\w*\W+.*?\(\)' => q<function names should only 
contain [a-z0-9_]>,
            );
 
            my %string_bashisms = (



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

Reply via email to