Author: adam-guest
Date: 2008-03-26 21:50:43 +0000 (Wed, 26 Mar 2008)
New Revision: 1209
Modified:
trunk/debian/changelog
trunk/scripts/checkbashisms.pl
Log:
+ Improve the invalid function name test to not trigger on IFS="()"
Modified: trunk/debian/changelog
===================================================================
--- trunk/debian/changelog 2008-03-26 19:31:04 UTC (rev 1208)
+++ trunk/debian/changelog 2008-03-26 21:50:43 UTC (rev 1209)
@@ -7,6 +7,7 @@
deciding whether it's a comment line and skipping it
+ Fix heredoc detection to match '<< \EOT ... EOT' but not
'<< "\EOT" ... EOT'
+ + Improve the invalid function name test to not trigger on IFS="()"
* debcheckout:
+ Tighten the "repository owner" regex for darcs.d.o to correctly check
http://darcs.d.o/~foo/bar/baz against user foo
Modified: trunk/scripts/checkbashisms.pl
===================================================================
--- trunk/scripts/checkbashisms.pl 2008-03-26 19:31:04 UTC (rev 1208)
+++ trunk/scripts/checkbashisms.pl 2008-03-26 21:50:43 UTC (rev 1209)
@@ -178,7 +178,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]+\S*?\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]
