summaryrefslogtreecommitdiff
path: root/perl.c
diff options
context:
space:
mode:
authorCraig A. Berry <craigberry@mac.com>2011-12-03 15:55:26 -0600
committerCraig A. Berry <craigberry@mac.com>2011-12-03 15:55:26 -0600
commit9dfa9235a81f85efedff9a00ba71734ec14a5e4d (patch)
treebf3373a7d8eb9068244d8a8b13c8964d644b988c /perl.c
parent42932e1a33ce479972ea1398a41965265d8fb06b (diff)
downloadperl-9dfa9235a81f85efedff9a00ba71734ec14a5e4d.tar.gz
Remove the VMS-specific private len in S_mayberelocate.
This particular C<STRLEN len;> came along almost 15 years ago, way back in aa6893958c2b. Back then it was a plain and simple block scope variable. But now this code lives in a function that takes as an argument a variable of the same name and having the same meaning (i.e., the length of the directory name to be added to @INC). Someone looking only at the argument list might reasonably expect to be able to use len (though luckily there are no subsequent uses of it currently). So we really ought to update that len rather than create our own when we hijack the directory name to make our unixified version of it.
Diffstat (limited to 'perl.c')
-rw-r--r--perl.c3
1 files changed, 1 insertions, 2 deletions
diff --git a/perl.c b/perl.c
index 48ef5e02dd..fe69e8ccb9 100644
--- a/perl.c
+++ b/perl.c
@@ -4453,7 +4453,6 @@ S_mayberelocate(pTHX_ const char *const dir, STRLEN len, U32 flags)
#ifdef VMS
{
char *unix;
- STRLEN len;
if ((unix = tounixspec_ts(SvPV(libdir,len),NULL)) != NULL) {
len = strlen(unix);
@@ -4463,7 +4462,7 @@ S_mayberelocate(pTHX_ const char *const dir, STRLEN len, U32 flags)
else
PerlIO_printf(Perl_error_log,
"Failed to unixify @INC element \"%s\"\n",
- SvPV(libdir,len));
+ SvPV_nolen_const(libdir));
}
#endif