Integrate perlio:

[ 14214]
Win32-ize socketpair test
- Win32 can fork even though $Config{d_fork} is undef
- SOCK_DGRAM does not work - skip those tests.

[ 14213]
Abstract out the cloning of SvPVX and handle shared pv in a 
safe (if suboptimal) manner. Does not fix op/fork.t :-(

p4raw-link: @14214 on //depot/perlio: 26bf1728b5f258bdc26021f9eb583c46488b8859
p4raw-link: @14213 on //depot/perlio: 83841fad1c8ce4928a4239052849bad556adb3d3

p4raw-id: //depot/perl@14218

1 files changed, 44 insertions, 71 deletions

diff --git a/sv.c b/sv.c
index d5dffef883..3de686f40e 100644
--- a/sv.c
+++ b/sv.c
@@ -8881,6 +8881,40 @@ S_gv_share(pTHX_ SV *sstr)
 
 /* duplicate an SV of any type (including AV, HV etc) */
 
+void
+Perl_rvpv_dup(pTHX_ SV *dstr, SV *sstr, CLONE_PARAMS* param)
+{
+    if (SvROK(sstr)) {
+        SvRV(dstr) = SvWEAKREF(sstr)
+		     ? sv_dup(SvRV(sstr), param)
+		     : sv_dup_inc(SvRV(sstr), param);
+    }
+    else if (SvPVX(sstr)) {
+	/* Has something there */
+	if (SvLEN(sstr)) {
+	    /* Normal PV - clone whole allocated space */ 
+	    SvPVX(dstr) = SAVEPVN(SvPVX(sstr), SvLEN(sstr)-1);
+	}  
+	else {
+	    /* Special case - not normally malloced for some reason */
+	    if (SvREADONLY(sstr) && SvFAKE(sstr)) {
+		/* A "shared" PV - clone it as unshared string */
+		SvFAKE_off(dstr);
+		SvREADONLY_off(dstr);
+		SvPVX(dstr) = SAVEPVN(SvPVX(sstr), SvCUR(sstr));
+	    }
+	    else {
+		/* Some other special case - random pointer */
+		SvPVX(dstr) = SvPVX(sstr);		
+            }
+	}
+    }
+    else {
+	/* Copy the Null */
+	SvPVX(dstr) = SvPVX(sstr);
+    }
+}
+
 SV *
 Perl_sv_dup(pTHX_ SV *sstr, CLONE_PARAMS* param)
 {
@@ -8922,36 +8956,20 @@ Perl_sv_dup(pTHX_ SV *sstr, CLONE_PARAMS* param)
 	break;
     case SVt_RV:
 	SvANY(dstr)	= new_XRV();
-    SvRV(dstr)    = SvRV(sstr) && SvWEAKREF(sstr)
-			? sv_dup(SvRV(sstr), param)
-			: sv_dup_inc(SvRV(sstr), param);
+	Perl_rvpv_dup(aTHX_ dstr, sstr, param);
 	break;
     case SVt_PV:
 	SvANY(dstr)	= new_XPV();
 	SvCUR(dstr)	= SvCUR(sstr);
 	SvLEN(dstr)	= SvLEN(sstr);
-	if (SvROK(sstr))
-        SvRV(dstr)    = SvWEAKREF(sstr)
-			? sv_dup(SvRV(sstr), param)
-			: sv_dup_inc(SvRV(sstr), param);
-	else if (SvPVX(sstr) && SvLEN(sstr))
-	    SvPVX(dstr)	= SAVEPVN(SvPVX(sstr), SvLEN(sstr)-1);
-	else
-	    SvPVX(dstr)	= SvPVX(sstr);		/* XXX shared string/random ptr? */
+	Perl_rvpv_dup(aTHX_ dstr, sstr, param);
 	break;
     case SVt_PVIV:
 	SvANY(dstr)	= new_XPVIV();
 	SvCUR(dstr)	= SvCUR(sstr);
 	SvLEN(dstr)	= SvLEN(sstr);
 	SvIVX(dstr)	= SvIVX(sstr);
-	if (SvROK(sstr))
-        SvRV(dstr)    = SvWEAKREF(sstr)
-			? sv_dup(SvRV(sstr), param)
-			: sv_dup_inc(SvRV(sstr), param);
-	else if (SvPVX(sstr) && SvLEN(sstr))
-	    SvPVX(dstr)	= SAVEPVN(SvPVX(sstr), SvLEN(sstr)-1);
-	else
-	    SvPVX(dstr)	= SvPVX(sstr);		/* XXX shared string/random ptr? */
+	Perl_rvpv_dup(aTHX_ dstr, sstr, param);
 	break;
     case SVt_PVNV:
 	SvANY(dstr)	= new_XPVNV();
@@ -8959,14 +8977,7 @@ Perl_sv_dup(pTHX_ SV *sstr, CLONE_PARAMS* param)
 	SvLEN(dstr)	= SvLEN(sstr);
 	SvIVX(dstr)	= SvIVX(sstr);
 	SvNVX(dstr)	= SvNVX(sstr);
-	if (SvROK(sstr))
-        SvRV(dstr)    = SvWEAKREF(sstr)
-			? sv_dup(SvRV(sstr), param)
-			: sv_dup_inc(SvRV(sstr), param);
-	else if (SvPVX(sstr) && SvLEN(sstr))
-	    SvPVX(dstr)	= SAVEPVN(SvPVX(sstr), SvLEN(sstr)-1);
-	else
-	    SvPVX(dstr)	= SvPVX(sstr);		/* XXX shared string/random ptr? */
+	Perl_rvpv_dup(aTHX_ dstr, sstr, param);
 	break;
     case SVt_PVMG:
 	SvANY(dstr)	= new_XPVMG();
@@ -8976,14 +8987,7 @@ Perl_sv_dup(pTHX_ SV *sstr, CLONE_PARAMS* param)
 	SvNVX(dstr)	= SvNVX(sstr);
 	SvMAGIC(dstr)	= mg_dup(SvMAGIC(sstr), param);
 	SvSTASH(dstr)	= hv_dup_inc(SvSTASH(sstr), param);
-	if (SvROK(sstr))
-        SvRV(dstr)    = SvWEAKREF(sstr)
-			? sv_dup(SvRV(sstr), param)
-			: sv_dup_inc(SvRV(sstr), param);
-	else if (SvPVX(sstr) && SvLEN(sstr))
-	    SvPVX(dstr)	= SAVEPVN(SvPVX(sstr), SvLEN(sstr)-1);
-	else
-	    SvPVX(dstr)	= SvPVX(sstr);		/* XXX shared string/random ptr? */
+	Perl_rvpv_dup(aTHX_ dstr, sstr, param);
 	break;
     case SVt_PVBM:
 	SvANY(dstr)	= new_XPVBM();
@@ -8993,14 +8997,7 @@ Perl_sv_dup(pTHX_ SV *sstr, CLONE_PARAMS* param)
 	SvNVX(dstr)	= SvNVX(sstr);
 	SvMAGIC(dstr)	= mg_dup(SvMAGIC(sstr), param);
 	SvSTASH(dstr)	= hv_dup_inc(SvSTASH(sstr), param);
-	if (SvROK(sstr))
-        SvRV(dstr)    = SvWEAKREF(sstr)
-			? sv_dup(SvRV(sstr), param)
-			: sv_dup_inc(SvRV(sstr), param);
-	else if (SvPVX(sstr) && SvLEN(sstr))
-	    SvPVX(dstr)	= SAVEPVN(SvPVX(sstr), SvLEN(sstr)-1);
-	else
-	    SvPVX(dstr)	= SvPVX(sstr);		/* XXX shared string/random ptr? */
+	Perl_rvpv_dup(aTHX_ dstr, sstr, param);
 	BmRARE(dstr)	= BmRARE(sstr);
 	BmUSEFUL(dstr)	= BmUSEFUL(sstr);
 	BmPREVIOUS(dstr)= BmPREVIOUS(sstr);
@@ -9013,14 +9010,7 @@ Perl_sv_dup(pTHX_ SV *sstr, CLONE_PARAMS* param)
 	SvNVX(dstr)	= SvNVX(sstr);
 	SvMAGIC(dstr)	= mg_dup(SvMAGIC(sstr), param);
 	SvSTASH(dstr)	= hv_dup_inc(SvSTASH(sstr), param);
-	if (SvROK(sstr))
-        SvRV(dstr)    = SvWEAKREF(sstr)
-			? sv_dup(SvRV(sstr), param)
-			: sv_dup_inc(SvRV(sstr), param);
-	else if (SvPVX(sstr) && SvLEN(sstr))
-	    SvPVX(dstr)	= SAVEPVN(SvPVX(sstr), SvLEN(sstr)-1);
-	else
-	    SvPVX(dstr)	= SvPVX(sstr);		/* XXX shared string/random ptr? */
+	Perl_rvpv_dup(aTHX_ dstr, sstr, param);
 	LvTARGOFF(dstr)	= LvTARGOFF(sstr);	/* XXX sometimes holds PMOP* when DEBUGGING */
 	LvTARGLEN(dstr)	= LvTARGLEN(sstr);
 	LvTARG(dstr)	= sv_dup_inc(LvTARG(sstr), param);
@@ -9046,14 +9036,7 @@ Perl_sv_dup(pTHX_ SV *sstr, CLONE_PARAMS* param)
 	SvNVX(dstr)	= SvNVX(sstr);
 	SvMAGIC(dstr)	= mg_dup(SvMAGIC(sstr), param);
 	SvSTASH(dstr)	= hv_dup_inc(SvSTASH(sstr), param);
-	if (SvROK(sstr))
-        SvRV(dstr)    = SvWEAKREF(sstr)
-			? sv_dup(SvRV(sstr), param)
-			: sv_dup_inc(SvRV(sstr), param);
-	else if (SvPVX(sstr) && SvLEN(sstr))
-	    SvPVX(dstr)	= SAVEPVN(SvPVX(sstr), SvLEN(sstr)-1);
-	else
-	    SvPVX(dstr)	= SvPVX(sstr);		/* XXX shared string/random ptr? */
+	Perl_rvpv_dup(aTHX_ dstr, sstr, param);
 	GvNAMELEN(dstr)	= GvNAMELEN(sstr);
 	GvNAME(dstr)	= SAVEPVN(GvNAME(sstr), GvNAMELEN(sstr));
     	GvSTASH(dstr)	= hv_dup_inc(GvSTASH(sstr), param);
@@ -9069,14 +9052,7 @@ Perl_sv_dup(pTHX_ SV *sstr, CLONE_PARAMS* param)
 	SvNVX(dstr)	= SvNVX(sstr);
 	SvMAGIC(dstr)	= mg_dup(SvMAGIC(sstr), param);
 	SvSTASH(dstr)	= hv_dup_inc(SvSTASH(sstr), param);
-	if (SvROK(sstr))
-        SvRV(dstr)    = SvWEAKREF(sstr)
-			? sv_dup(SvRV(sstr), param)
-			: sv_dup_inc(SvRV(sstr), param);
-	else if (SvPVX(sstr) && SvLEN(sstr))
-	    SvPVX(dstr)	= SAVEPVN(SvPVX(sstr), SvLEN(sstr)-1);
-	else
-	    SvPVX(dstr)	= SvPVX(sstr);		/* XXX shared string/random ptr? */
+	Perl_rvpv_dup(aTHX_ dstr, sstr, param);
 	IoIFP(dstr)	= fp_dup(IoIFP(sstr), IoTYPE(sstr), param);
 	if (IoOFP(sstr) == IoIFP(sstr))
 	    IoOFP(dstr) = IoIFP(dstr);
@@ -9184,10 +9160,7 @@ Perl_sv_dup(pTHX_ SV *sstr, CLONE_PARAMS* param)
 	SvNVX(dstr)	= SvNVX(sstr);
 	SvMAGIC(dstr)	= mg_dup(SvMAGIC(sstr), param);
 	SvSTASH(dstr)	= hv_dup_inc(SvSTASH(sstr), param);
-	if (SvPVX(sstr) && SvLEN(sstr))
-	    SvPVX(dstr)	= SAVEPVN(SvPVX(sstr), SvLEN(sstr)-1);
-	else
-	    SvPVX(dstr)	= SvPVX(sstr);		/* XXX shared string/random ptr? */
+	Perl_rvpv_dup(aTHX_ dstr, sstr, param);
 	CvSTASH(dstr)	= hv_dup(CvSTASH(sstr), param); /* NOTE: not refcounted */
 	CvSTART(dstr)	= CvSTART(sstr);
 	CvROOT(dstr)	= OpREFCNT_inc(CvROOT(sstr));