aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorNoah Misch <noah@leadboat.com>2020-06-07 16:27:13 -0700
committerNoah Misch <noah@leadboat.com>2020-06-07 16:27:19 -0700
commit696cb6d8f6be667af79a512db57ab0bad3248f42 (patch)
tree4d55f59b5b72ff1b12bf12403e7d6933b5f8f901
parent109387075c36ecfb84e522ecd213f69d30af61c1 (diff)
downloadpostgresql-696cb6d8f6be667af79a512db57ab0bad3248f42.tar.gz
postgresql-696cb6d8f6be667af79a512db57ab0bad3248f42.zip
MSVC: Avoid warning when testing a TAP suite without PROVE_FLAGS.
Commit 7be5d8df1f74b78620167d3abf32ee607e728919 surfaced the logic error, which had no functional implications, by adding "use warnings". The buildfarm always customizes PROVE_FLAGS, so the warning did not appear there. Back-patch to 9.5 (all supported versions).
-rw-r--r--src/tools/msvc/vcregress.pl2
1 files changed, 1 insertions, 1 deletions
diff --git a/src/tools/msvc/vcregress.pl b/src/tools/msvc/vcregress.pl
index 1a9a282494f..2986a9053ab 100644
--- a/src/tools/msvc/vcregress.pl
+++ b/src/tools/msvc/vcregress.pl
@@ -189,7 +189,7 @@ sub tap_check
unless $config->{tap_tests};
my @flags;
- foreach my $arg (0 .. scalar(@_))
+ foreach my $arg (0 .. scalar(@_) - 1)
{
next unless $_[$arg] =~ /^PROVE_FLAGS=(.*)/;
@flags = split(/\s+/, $1);