diff options
author | David Mitchell <davem@iabyn.com> | 2017-05-30 15:27:00 +0100 |
---|---|---|
committer | David Mitchell <davem@iabyn.com> | 2017-06-07 09:11:07 +0100 |
commit | 8a015f6ec969acd841529dbca037e9fc0e3046a4 (patch) | |
tree | 74cd9022f06510b51cf13875748258cae8b4b33d /sv.c | |
parent | 7f595ee351b96363e401ea506c10b3590018e1e7 (diff) | |
download | perl-8a015f6ec969acd841529dbca037e9fc0e3046a4.tar.gz |
Perl_sv_vcatpvfn_flags: s/int/STRLEN/g
There wee a few residual places that used int loop counters, e.g. to
prepend N '0's to a number. Since the N's are of type STRLEN, make the
loop counters STRLEN too.
Its a bit academic since you're unlikely to have a number needing >2Gb
worth of zero padding, but it makes things consistent and easier to audit.
At this point I believe that any remaining usage of int / I32 / U32 in
Perl_sv_vcatpvfn_flags() is legitimate.
Diffstat (limited to 'sv.c')
-rw-r--r-- | sv.c | 10 |
1 files changed, 5 insertions, 5 deletions
@@ -13268,8 +13268,8 @@ Perl_sv_vcatpvfn_flags(pTHX_ SV *const sv, const char *const pat, const STRLEN p p = SvEND(sv); if (esignlen && fill) { - int i; - for (i = 0; i < (int)esignlen; i++) + STRLEN i; + for (i = 0; i < esignlen; i++) *p++ = esignbuf[i]; } if (gap && !left) { @@ -13277,12 +13277,12 @@ Perl_sv_vcatpvfn_flags(pTHX_ SV *const sv, const char *const pat, const STRLEN p p += gap; } if (esignlen && !fill) { - int i; - for (i = 0; i < (int)esignlen; i++) + STRLEN i; + for (i = 0; i < esignlen; i++) *p++ = esignbuf[i]; } if (zeros) { - int i; + STRLEN i; for (i = zeros; i; i--) *p++ = '0'; } |