sv_vcatpvfn_flags(): no "redundant arg" warnings for '%*2$d'
authorAaron Crane <arc@cpan.org>
Mon, 29 Jun 2015 15:55:14 +0000 (16:55 +0100)
committerSteve Hay <steve.m.hay@googlemail.com>
Wed, 30 Sep 2015 12:10:11 +0000 (13:10 +0100)
commit802475c8d28a32d1dfca9e93e7c06560c1483093
tree55fe8410971c51e198a1fdd22aed888ec2719c58
parentfcec72c1e34238bcdafd034e46cebf3eab65c8ec
sv_vcatpvfn_flags(): no "redundant arg" warnings for '%*2$d'

This addresses RT#125469: if the format string uses an explicit index for a
field width, that should suppress any "redundant argument" warnings, in
just the same way as an explicit index does when used to specify a format
argument.

Blead has a more careful reworking of the way printf warnings work, but that
change is unsuitable for maint, as it can lead to additional warnings being
emitted for some invalid format strings. This smaller change ensures that
maint (5.22.1-to-be) always emits strictly fewer warnings than 5.22.0.

(cherry picked from commit ddd5feee1fd3e7105e11f3266509b916b9d16e55)
sv.c
t/op/sprintf.t