The following commit has been merged in the master branch:
commit c4b87525b17351ee215a0df5f731d7d1f5a1e388
Author: Adam D. Barratt <[EMAIL PROTECTED]>
Date:   Fri Jul 4 21:06:35 2008 +0100

    Add "then" to the list of expressions which can precede a bashism

diff --git a/checks/scripts b/checks/scripts
index 0fe348b..1861733 100644
--- a/checks/scripts
+++ b/checks/scripts
@@ -543,7 +543,7 @@ while (<SCRIPTS>) {
            if ($cat_string eq "" and $checkbashisms and 
!$within_another_shell) {
                my $found = 0;
                my $match = '';
-               my $LEADIN = qr'(?:(^|[`&;(|{])\s*|(if|do|while)\s+)';
+               my $LEADIN = qr'(?:(^|[`&;(|{])\s*|(if|then|do|while)\s+)';
                my @bashism_single_quote_regexs = (
                  $LEADIN . 
qr'echo\s+(?:-[^e]+\s+)?([\'])[^\']*(\\[\\abcEfnrtv0])+.*?[\']',
                        # unsafe echo with backslashes

-- 
Debian package checker


-- 
To UNSUBSCRIBE, email to [EMAIL PROTECTED]
with a subject of "unsubscribe". Trouble? Contact [EMAIL PROTECTED]

Reply via email to