[shell changes from patch from perl5.003_23 to perl5.003_24]

[p5sagit/p5-mst-13.2.git] / av.c

diff --git a/av.c b/av.c
index b27ec76..554f261 100644 (file)
--- a/av.c
+++ b/av.c
@@ -94,7 +94,7 @@ I32 key;
 #endif
                ary = AvALLOC(av) + AvMAX(av) + 1;
                tmp = newmax - AvMAX(av);
-               if (av == stack) {      /* Oops, grew stack (via av_store()?) */
+               if (av == curstack) {   /* Oops, grew stack (via av_store()?) */
                    stack_sp = AvALLOC(av) + (stack_sp - stack_base);
                    stack_base = AvALLOC(av);
                    stack_max = stack_base + newmax;
@@ -153,12 +153,19 @@ I32 lval;
        return av_store(av,key,sv);
     }
     if (AvARRAY(av)[key] == &sv_undef) {
+    emptyness:
        if (lval) {
            sv = NEWSV(6,0);
            return av_store(av,key,sv);
        }
        return 0;
     }
+    else if (AvREIFY(av)
+            && (!AvARRAY(av)[key]      /* eg. @_ could have freed elts */
+                || SvTYPE(AvARRAY(av)[key]) == SVTYPEMASK)) {
+       AvARRAY(av)[key] = &sv_undef;   /* 1/2 reify */
+       goto emptyness;
+    }
     return &AvARRAY(av)[key];
 }
 
@@ -172,10 +179,13 @@ SV *val;
 
     if (!av)
        return 0;
+    if (!val)
+       val = &sv_undef;
 
     if (SvRMAGICAL(av)) {
        if (mg_find((SV*)av,'P')) {
-           mg_copy((SV*)av, val, 0, key);
+           if (val != &sv_undef)
+               mg_copy((SV*)av, val, 0, key);
            return 0;
        }
     }
@@ -185,9 +195,8 @@ SV *val;
        if (key < 0)
            return 0;
     }
-    if (!val)
-       val = &sv_undef;
-
+    if (SvREADONLY(av) && key >= AvFILL(av))
+       croak(no_modify);
     if (key > AvMAX(av))
        av_extend(av,key);
     if (AvREIFY(av))
@@ -196,7 +205,7 @@ SV *val;
     ary = AvARRAY(av);
     if (AvFILL(av) < key) {
        if (!AvREAL(av)) {
-           if (av == stack && key > stack_sp - stack_base)
+           if (av == curstack && key > stack_sp - stack_base)
                stack_sp = stack_base + key;    /* XPUSH in disguise */
            do
                ary[++AvFILL(av)] = &sv_undef;
@@ -289,6 +298,11 @@ register AV *av;
     register I32 key;
     SV** ary;
 
+#ifdef DEBUGGING
+    if (SvREFCNT(av) <= 0) {
+       warn("Attempt to clear deleted array");
+    }
+#endif
     if (!av || AvMAX(av) < 0)
        return;
     /*SUPPRESS 560*/
@@ -322,10 +336,6 @@ register AV *av;
        while (key)
            SvREFCNT_dec(AvARRAY(av)[--key]);
     }
-    if (key = AvARRAY(av) - AvALLOC(av)) {
-       AvMAX(av) += key;
-       SvPVX(av) = (char*)AvALLOC(av);
-    }
     Safefree(AvALLOC(av));
     AvALLOC(av) = 0;
     SvPVX(av) = 0;
@@ -354,6 +364,8 @@ register AV *av;
 
     if (!av || AvFILL(av) < 0)
        return &sv_undef;
+    if (SvREADONLY(av))
+       croak(no_modify);
     retval = AvARRAY(av)[AvFILL(av)];
     AvARRAY(av)[AvFILL(av)--] = &sv_undef;
     if (SvSMAGICAL(av))
@@ -371,6 +383,8 @@ register I32 num;
 
     if (!av || num <= 0)
        return;
+    if (SvREADONLY(av))
+       croak(no_modify);
     if (!AvREAL(av)) {
        if (AvREIFY(av))
            av_reify(av);
@@ -414,6 +428,8 @@ register AV *av;
 
     if (!av || AvFILL(av) < 0)
        return &sv_undef;
+    if (SvREADONLY(av))
+       croak(no_modify);
     retval = *AvARRAY(av);
     if (AvREAL(av))
        *AvARRAY(av) = &sv_undef;