Remove the VMS-specific private len in S_mayberelocate.
authorCraig A. Berry <craigberry@mac.com>
Sat, 3 Dec 2011 21:55:26 +0000 (15:55 -0600)
committerCraig A. Berry <craigberry@mac.com>
Thu, 25 Oct 2012 13:16:04 +0000 (08:16 -0500)
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.

perl.c

index 0ba5bac..fb2c408 100644 (file)
--- a/perl.c
+++ b/perl.c
@@ -4422,7 +4422,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);
@@ -4432,7 +4431,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