mirror of
git://git.sv.gnu.org/coreutils.git
synced 2026-08-17 09:06:50 +02:00
tests: remove unnecessary single quotes in perl hash use: ->{'SYM'}
Run this command:
git grep -l "limits->{'" \
| xargs perl -pi -e "s/limits->{'(.*?)'}/limits->{\$1}/g"
* cfg.mk (sc_prohibit_perl_hash_quotes): New rule to match.
* tests/misc/join: Remove quotes.
* tests/misc/sort: Likewise.
* tests/misc/sort-merge: Likewise.
* tests/misc/test: Likewise.
* tests/misc/unexpand: Likewise.
* tests/misc/uniq: Likewise.
This commit is contained in:
@@ -233,6 +233,11 @@ sc_require_stdio_safer:
|
||||
else :; \
|
||||
fi
|
||||
|
||||
sc_prohibit_perl_hash_quotes:
|
||||
@prohibit="\{'[A-Z_]+' *[=}]" \
|
||||
halt="in Perl code, write \$$hash{KEY}, not \$$hash{'K''EY'}" \
|
||||
$(_sc_search_regexp)
|
||||
|
||||
# Prefer xnanosleep over other less-precise sleep methods
|
||||
sc_prohibit_sleep:
|
||||
@prohibit='\<(nano|u)?sleep \(' \
|
||||
|
||||
+2
-2
@@ -146,9 +146,9 @@ my @tv = (
|
||||
t_subst "a:1:2:\nb:1:2:\n", 0],
|
||||
|
||||
# fields > SIZE_MAX are silently interpreted as SIZE_MAX
|
||||
['bigfield1', "-1 $limits->{'UINTMAX_OFLOW'} -2 2",
|
||||
['bigfield1', "-1 $limits->{UINTMAX_OFLOW} -2 2",
|
||||
["a\n", "b\n"], " a b\n", 0],
|
||||
['bigfield2', "-1 $limits->{'SIZE_OFLOW'} -2 2",
|
||||
['bigfield2', "-1 $limits->{SIZE_OFLOW} -2 2",
|
||||
["a\n", "b\n"], " a b\n", 0],
|
||||
|
||||
# FIXME: change this to ensure the diagnostic makes sense
|
||||
|
||||
+2
-2
@@ -353,9 +353,9 @@ my @Tests =
|
||||
{IN=>"a\0z\01\nb\0y\02\n"}, {OUT=>"b\0y\02\na\0z\01\n"}],
|
||||
|
||||
# fields > SIZE_MAX are silently interpreted as SIZE_MAX
|
||||
["bigfield1", "-k $limits->{'UINTMAX_OFLOW'}",
|
||||
["bigfield1", "-k $limits->{UINTMAX_OFLOW}",
|
||||
{IN=>"2\n1\n"}, {OUT=>"1\n2\n"}],
|
||||
["bigfield2", "-k $limits->{'SIZE_OFLOW'}",
|
||||
["bigfield2", "-k $limits->{SIZE_OFLOW}",
|
||||
{IN=>"2\n1\n"}, {OUT=>"1\n2\n"}],
|
||||
|
||||
# Using an old-style key-specifying option like +1 with an invalid
|
||||
|
||||
@@ -34,7 +34,7 @@ my $big_input = "aaa\n" x 1024;
|
||||
# don't need to check for existence, since we're running in a temp dir
|
||||
my $badtmp = 'does/not/exist';
|
||||
|
||||
my $bigint = $limits->{'UINTMAX_OFLOW'};
|
||||
my $bigint = $limits->{UINTMAX_OFLOW};
|
||||
|
||||
my @Tests =
|
||||
(
|
||||
|
||||
+3
-3
@@ -158,19 +158,19 @@ my @Tests =
|
||||
['eq-3', qw(0 -eq 00)],
|
||||
['eq-4', qw(8 -eq 9), {EXIT=>1}],
|
||||
['eq-5', qw(1 -eq 0), {EXIT=>1}],
|
||||
['eq-6', "$limits->{'UINTMAX_OFLOW'} -eq 0", {EXIT=>1}],
|
||||
['eq-6', "$limits->{UINTMAX_OFLOW} -eq 0", {EXIT=>1}],
|
||||
|
||||
['gt-1', qw(5 -gt 5), {EXIT=>1}],
|
||||
['gt-2', qw(5 -gt 4)],
|
||||
['gt-3', qw(4 -gt 5), {EXIT=>1}],
|
||||
['gt-4', qw(-1 -gt -2)],
|
||||
['gt-5', "$limits->{'UINTMAX_OFLOW'} -gt $limits->{'INTMAX_UFLOW'}"],
|
||||
['gt-5', "$limits->{UINTMAX_OFLOW} -gt $limits->{INTMAX_UFLOW}"],
|
||||
|
||||
['lt-1', qw(5 -lt 5), {EXIT=>1}],
|
||||
['lt-2', qw(5 -lt 4), {EXIT=>1}],
|
||||
['lt-3', qw(4 -lt 5)],
|
||||
['lt-4', qw(-1 -lt -2), {EXIT=>1}],
|
||||
['lt-5', "$limits->{'INTMAX_UFLOW'} -lt $limits->{'UINTMAX_OFLOW'}"],
|
||||
['lt-5', "$limits->{INTMAX_UFLOW} -lt $limits->{UINTMAX_OFLOW}"],
|
||||
|
||||
['inv-1', qw(0x0 -eq 00), {EXIT=>2},
|
||||
{ERR=>"$prog: invalid integer `0x0'\n"}],
|
||||
|
||||
+1
-1
@@ -78,7 +78,7 @@ my @Tests =
|
||||
|
||||
# It is debatable whether this test should require an environment
|
||||
# setting of e.g., _POSIX2_VERSION=1.
|
||||
['obs-ovflo', "-$limits->{'UINTMAX_OFLOW'}", {IN=>''}, {OUT=>''},
|
||||
['obs-ovflo', "-$limits->{UINTMAX_OFLOW}", {IN=>''}, {OUT=>''},
|
||||
{EXIT => 1}, {ERR => "$prog: tab stop value is too large\n"}],
|
||||
);
|
||||
|
||||
|
||||
+2
-2
@@ -194,8 +194,8 @@ my @Tests =
|
||||
. $try}],
|
||||
# Check that -d and -u suppress all output, as POSIX requires.
|
||||
['120', qw(-d -u), {IN=>"a\na\n\b"}, {OUT=>""}],
|
||||
['121', "-d -u -w$limits->{'UINTMAX_OFLOW'}", {IN=>"a\na\n\b"}, {OUT=>""}],
|
||||
['122', "-d -u -w$limits->{'SIZE_OFLOW'}", {IN=>"a\na\n\b"}, {OUT=>""}],
|
||||
['121', "-d -u -w$limits->{UINTMAX_OFLOW}", {IN=>"a\na\n\b"}, {OUT=>""}],
|
||||
['122', "-d -u -w$limits->{SIZE_OFLOW}", {IN=>"a\na\n\b"}, {OUT=>""}],
|
||||
# Check that --zero-terminated is synonymous with -z.
|
||||
['123', '--zero-terminated', {IN=>"a\na\nb"}, {OUT=>"a\na\nb\0"}],
|
||||
['124', '--zero-terminated', {IN=>"a\0a\0b"}, {OUT=>"a\0b\0"}],
|
||||
|
||||
Reference in New Issue
Block a user