[SCM] Git repository for devscripts branch, master,	updated. v2.12.4-47-g1b29202
    Benjamin Drung 
    bdrung at debian.org
       
    Thu Oct 25 21:30:11 UTC 2012
    
    
  
The following commit has been merged in the master branch:
commit 1b292022be0ea50ea17bc6f2ad56d00fcd1beac3
Author: Raphael Geissert <geissert at debian.org>
Date:   Tue Oct 23 12:36:36 2012 -0500
    checkbashisms: improve check for non-alnum function names
    
    Function names with certain characters and those with the 'function'
    reserved word but without the parenthesis were not matched by the old
    regex.
    
    Signed-off-by: Benjamin Drung <bdrung at debian.org>
diff --git a/scripts/checkbashisms.pl b/scripts/checkbashisms.pl
index 6bac879..5cf9152 100755
--- a/scripts/checkbashisms.pl
+++ b/scripts/checkbashisms.pl
@@ -604,7 +604,7 @@ sub init_hashes {
 	$LEADIN . qr'alias\s+-p' =>       q<alias -p>,
 	$LEADIN . qr'unalias\s+-a' =>     q<unalias -a>,
 	$LEADIN . qr'local\s+-[a-zA-Z]+' => q<local -opt>,
-	qr'(?:^|\s+)\s*\(?\w*[^\(\w\s]+\S*?\s*\(\)\s*([\{|\(]|\Z)'
+	qr'(?:^|\s)(?<func>function\s)?\s*(?:[^<>\(\)\[\]\{\};|\s]*[^<>\(\)\[\]\{\};|\s\w][^<>\(\)\[\]\{\};|\s]*)\s*(?(<func>)(?:\(\s*\))?|\(\s*\))\s*([\{|\(]|\Z)'
 		=> q<function names should only contain [a-z0-9_]>,
 	$LEADIN . qr'(push|pop)d(\s|\Z)' =>    q<(push|pop)d>,
 	$LEADIN . qr'export\s+-[^p]' =>  q<export only takes -p as an option>,
-- 
Git repository for devscripts
    
    
More information about the devscripts-devel
mailing list