paramcheck: Use + quantifier and return copy.

This commit is contained in:
Steven Schubiger 2011-10-21 22:10:02 +02:00 committed by Giuseppe Scrivano
parent fce4e757a2
commit e0a3162893
2 changed files with 13 additions and 8 deletions

View File

@ -1,3 +1,8 @@
2011-10-16 Steven Schubiger <stsc@member.fsf.org>
* util/paramcheck.pl: Match 1 or more times where applicable.
(extract_entries): Return a copy instead of reference.
2011-09-04 Alan Hourihane <alanh@fairlite.co.uk> (tiny change) 2011-09-04 Alan Hourihane <alanh@fairlite.co.uk> (tiny change)
* configure.ac: Check for libz when gnutls is used. * configure.ac: Check for libz when gnutls is used.

View File

@ -33,11 +33,11 @@ my $tex_content = read_file($tex_file);
my @args = ([ my @args = ([
$main_content, $main_content,
qr/static \s+? struct \s+? cmdline_option \s+? option_data\[\] \s+? = \s+? \{ (.*?) \}\;/sx, qr/static \s+? struct \s+? cmdline_option \s+? option_data\[\] \s+? = \s+? \{ (.+?) \}\;/sx,
[ qw(long_name short_name type data argtype) ], [ qw(long_name short_name type data argtype) ],
], [ ], [
$init_content, $init_content,
qr/commands\[\] \s+? = \s+? \{ (.*?) \}\;/sx, qr/commands\[\] \s+? = \s+? \{ (.+?) \}\;/sx,
[ qw(name place action) ], [ qw(name place action) ],
]); ]);
@ -78,18 +78,18 @@ sub extract_entries
my (@entries, %index, $i); my (@entries, %index, $i);
foreach my $chunk (@$chunks) { foreach my $chunk (@$chunks) {
my ($args) = $chunk =~ /\{ \s+? (.*?) \s+? \}/sx; my ($args) = $chunk =~ /\{ \s+? (.+?) \s+? \}/sx;
next unless defined $args; next unless defined $args;
my @args = map { my @args = map {
tr/'"//d; $_ tr/'"//d; $_
} map { } map {
/\((.*?)\)/ ? $1 : $_ /\((.+?)\)/ ? $1 : $_
} split /\,\s+/, $args; } split /\,\s+/, $args;
my $entry = { map { $_ => shift @args } @$names }; my $entry = { map { $_ => shift @args } @$names };
($entry->{line}) = $chunk =~ /^ \s+? (\{.*)/mx; ($entry->{line}) = $chunk =~ /^ \s+? (\{.+)/mx;
if ($chunk =~ /deprecated/i) { if ($chunk =~ /deprecated/i) {
$entries[-1]->{deprecated} = true; $entries[-1]->{deprecated} = true;
} }
@ -103,9 +103,9 @@ sub extract_entries
push @entries, $entry; push @entries, $entry;
} }
push @entries, \%index; push @entries, { %index };
return \@entries; return [ @entries ];
} }
sub output_results sub output_results
@ -281,7 +281,7 @@ sub emit_undocumented_opts
while ($tex =~ /^\@item\w*? \s+? --([-a-z0-9]+)/gmx) { while ($tex =~ /^\@item\w*? \s+? --([-a-z0-9]+)/gmx) {
$tex_items{$1} = true; $tex_items{$1} = true;
} }
my ($help) = $main =~ /\n print_help .*? \{\n (.*) \n\} \n/sx; my ($help) = $main =~ /\n print_help .*? \{\n (.+) \n\} \n/sx;
while ($help =~ /--([-a-z0-9]+)/g) { while ($help =~ /--([-a-z0-9]+)/g) {
$main_items{$1} = true; $main_items{$1} = true;
} }