Author: adam-guest
Date: 2008-03-28 06:47:37 +0000 (Fri, 28 Mar 2008)
New Revision: 1212
Modified:
trunk/debian/changelog
trunk/scripts/checkbashisms.pl
Log:
+ Add another variation on the theme of shell wrapper detection (${1+"$@"})
Modified: trunk/debian/changelog
===================================================================
--- trunk/debian/changelog 2008-03-27 18:17:37 UTC (rev 1211)
+++ trunk/debian/changelog 2008-03-28 06:47:37 UTC (rev 1212)
@@ -9,6 +9,7 @@
'<< "\EOT" ... EOT'
+ Improve the invalid function name test to not trigger on IFS="()"
+ Modify the brace expansion test to allow "${foo},${bar}"
+ + Add another variation on the theme of shell wrapper detection (${1+"$@"})
* 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-27 18:17:37 UTC (rev 1211)
+++ trunk/scripts/checkbashisms.pl 2008-03-28 06:47:37 UTC (rev 1212)
@@ -364,7 +364,7 @@
next if /^#/o;
next if /^$/o;
last if (++$i > 55);
- if
(/(^\s*|\beval\s*[\'\"]|;\s*)exec\s*.+\s*.?\$$var.?\s*(--\s*)?.?(\${1:?\+)?\$(\@|\*)/)
{
+ if
(/(^\s*|\beval\s*[\'\"]|;\s*)exec\s*.+\s*.?\$$var.?\s*(--\s*)?.?(\${1:?\+.?)?\$(\@|\*)/)
{
$ret = 1;
last;
} elsif (/^\s*(\w+)=\$0;/) {
--
To unsubscribe, send mail to [EMAIL PROTECTED]
