ExtUtils/Miniperl.pm not built on Win32
[p5sagit/p5-mst-13.2.git] / pp.c
diff --git a/pp.c b/pp.c
index 4ca4b3e..6423e27 100644 (file)
--- a/pp.c
+++ b/pp.c
  * floating-point type to use for NV that has adequate bits to fully
  * hold an IV/UV.  (In other words, sizeof(long) == sizeof(double).)
  *
- * It just so happens that "int" is the right size everywhere, at
- * least today.
+ * It just so happens that "int" is the right size almost everywhere.
  */
 typedef int IBW;
 typedef unsigned UBW;
 
+/*
+ * Mask used after bitwise operations.
+ *
+ * There is at least one realm (Cray word machines) that doesn't
+ * have an integral type (except char) small enough to be represented
+ * in a double without loss; that is, it has no 32-bit type.
+ */
+#if BYTEORDER > 0xFFFF && defined(_CRAY) && !defined(_CRAYMPP)
+#  define BWBITS  32
+#  define BWMASK  ((1 << BWBITS) - 1)
+#  define BWSIGN  (1 << (BWBITS - 1))
+#  define BWi(i)  (((i) & BW_SIGN) ? ((i) | ~BW_MASK) : ((i) & BW_MASK))
+#  define BWu(u)  ((u) & BW_MASK)
+#else
+#  define BWi(i)  (i)
+#  define BWu(u)  (u)
+#endif
+
+/*
+ * Offset for integer pack/unpack.
+ *
+ * On architectures where I16 and I32 aren't really 16 and 32 bits,
+ * which for now are all Crays, pack and unpack have to play games.
+ */
+
+/*
+ * These values are required for portability of pack() output.
+ * If they're not right on your machine, then pack() and unpack()
+ * wouldn't work right anyway; you'll need to apply the Cray hack.
+ * (I'd like to check them with #if, but you can't use sizeof() in
+ * the preprocessor.)
+ */
+#define SIZE16 2
+#define SIZE32 4
+
+#if BYTEORDER > 0xFFFF && defined(_CRAY) && !defined(_CRAYMPP)
+#  if BYTEORDER == 0x12345678
+#    define OFF16(p)   (char*)(p)
+#    define OFF32(p)   (char*)(p)
+#  else
+#    if BYTEORDER == 0x87654321
+#      define OFF16(p) ((char*)(p) + (sizeof(U16) - SIZE16))
+#      define OFF32(p) ((char*)(p) + (sizeof(U32) - SIZE32))
+#    else
+       }}}} bad cray byte order
+#    endif
+#  endif
+#  define COPY16(s,p)  (*(p) = 0, Copy(s, OFF16(p), SIZE16, char))
+#  define COPY32(s,p)  (*(p) = 0, Copy(s, OFF32(p), SIZE32, char))
+#  define CAT16(sv,p)  sv_catpvn(sv, OFF16(p), SIZE16)
+#  define CAT32(sv,p)  sv_catpvn(sv, OFF32(p), SIZE32)
+#else
+#  define COPY16(s,p)  Copy(s, p, SIZE16, char)
+#  define COPY32(s,p)  Copy(s, p, SIZE32, char)
+#  define CAT16(sv,p)  sv_catpvn(sv, (char*)(p), SIZE16)
+#  define CAT32(sv,p)  sv_catpvn(sv, (char*)(p), SIZE32)
+#endif
+
 static void doencodes _((SV* sv, char* s, I32 len));
 static SV* refto _((SV* sv));
 static U32 seed _((void));
@@ -40,9 +97,8 @@ static bool srand_called = FALSE;
 PP(pp_stub)
 {
     dSP;
-    if (GIMME != G_ARRAY) {
+    if (GIMME_V == G_SCALAR)
        XPUSHs(&sv_undef);
-    }
     RETURN;
 }
 
@@ -81,15 +137,18 @@ PP(pp_padav)
 PP(pp_padhv)
 {
     dSP; dTARGET;
+    I32 gimme;
+
     XPUSHs(TARG);
     if (op->op_private & OPpLVAL_INTRO)
        SAVECLEARSV(curpad[op->op_targ]);
     if (op->op_flags & OPf_REF)
        RETURN;
-    if (GIMME == G_ARRAY) { /* array wanted */
+    gimme = GIMME_V;
+    if (gimme == G_ARRAY) {
        RETURNOP(do_kv(ARGS));
     }
-    else {
+    else if (gimme == G_SCALAR) {
        SV* sv = sv_newmortal();
        if (HvFILL((HV*)TARG)) {
            sprintf(buf, "%ld/%ld",
@@ -99,8 +158,8 @@ PP(pp_padhv)
        else
            sv_setiv(sv, 0);
        SETs(sv);
-       RETURN;
     }
+    RETURN;
 }
 
 PP(pp_padany)
@@ -550,12 +609,14 @@ PP(pp_undef)
        hv_undef((HV*)sv);
        break;
     case SVt_PVCV:
-       if (!CvANON((CV*)sv) && cv_const_sv((CV*)sv))
+       if (cv_const_sv((CV*)sv))
            warn("Constant subroutine %s undefined",
-                GvENAME(CvGV((CV*)sv)));
+                CvANON((CV*)sv) ? "(anonymous)" : GvENAME(CvGV((CV*)sv)));
        /* FALL THROUGH */
     case SVt_PVFM:
-       cv_undef((CV*)sv);
+       { GV* gv = (GV*)SvREFCNT_inc(CvGV((CV*)sv));
+         cv_undef((CV*)sv);
+         CvGV((CV*)sv) = gv; }   /* let user-undef'd sub keep its identity */
        break;
     case SVt_PVGV:
        if (SvFAKE(sv))
@@ -802,11 +863,13 @@ PP(pp_left_shift)
       IBW shift = POPi;
       if (op->op_private & HINT_INTEGER) {
        IBW i = TOPi;
-       SETi( i << shift );
+       i <<= shift;
+       SETi(BWi(i));
       }
       else {
        UBW u = TOPu;
-       SETu( u << shift );
+       u <<= shift;
+       SETu(BWu(u));
       }
       RETURN;
     }
@@ -819,11 +882,13 @@ PP(pp_right_shift)
       IBW shift = POPi;
       if (op->op_private & HINT_INTEGER) {
        IBW i = TOPi;
-       SETi( i >> shift );
+       i >>= shift;
+       SETi(BWi(i));
       }
       else {
        UBW u = TOPu;
-       SETu( u >> shift );
+       u >>= shift;
+       SETu(BWu(u));
       }
       RETURN;
     }
@@ -834,7 +899,7 @@ PP(pp_lt)
     dSP; tryAMAGICbinSET(lt,0); 
     {
       dPOPnv;
-      SETs((TOPn < value) ? &sv_yes : &sv_no);
+      SETs(boolSV(TOPn < value));
       RETURN;
     }
 }
@@ -844,7 +909,7 @@ PP(pp_gt)
     dSP; tryAMAGICbinSET(gt,0); 
     {
       dPOPnv;
-      SETs((TOPn > value) ? &sv_yes : &sv_no);
+      SETs(boolSV(TOPn > value));
       RETURN;
     }
 }
@@ -854,7 +919,7 @@ PP(pp_le)
     dSP; tryAMAGICbinSET(le,0); 
     {
       dPOPnv;
-      SETs((TOPn <= value) ? &sv_yes : &sv_no);
+      SETs(boolSV(TOPn <= value));
       RETURN;
     }
 }
@@ -864,7 +929,7 @@ PP(pp_ge)
     dSP; tryAMAGICbinSET(ge,0); 
     {
       dPOPnv;
-      SETs((TOPn >= value) ? &sv_yes : &sv_no);
+      SETs(boolSV(TOPn >= value));
       RETURN;
     }
 }
@@ -874,7 +939,7 @@ PP(pp_ne)
     dSP; tryAMAGICbinSET(ne,0); 
     {
       dPOPnv;
-      SETs((TOPn != value) ? &sv_yes : &sv_no);
+      SETs(boolSV(TOPn != value));
       RETURN;
     }
 }
@@ -909,7 +974,7 @@ PP(pp_slt)
       int cmp = ((op->op_private & OPpLOCALE)
                 ? sv_cmp_locale(left, right)
                 : sv_cmp(left, right));
-      SETs( cmp < 0 ? &sv_yes : &sv_no );
+      SETs(boolSV(cmp < 0));
       RETURN;
     }
 }
@@ -922,7 +987,7 @@ PP(pp_sgt)
       int cmp = ((op->op_private & OPpLOCALE)
                 ? sv_cmp_locale(left, right)
                 : sv_cmp(left, right));
-      SETs( cmp > 0 ? &sv_yes : &sv_no );
+      SETs(boolSV(cmp > 0));
       RETURN;
     }
 }
@@ -935,7 +1000,7 @@ PP(pp_sle)
       int cmp = ((op->op_private & OPpLOCALE)
                 ? sv_cmp_locale(left, right)
                 : sv_cmp(left, right));
-      SETs( cmp <= 0 ? &sv_yes : &sv_no );
+      SETs(boolSV(cmp <= 0));
       RETURN;
     }
 }
@@ -948,7 +1013,7 @@ PP(pp_sge)
       int cmp = ((op->op_private & OPpLOCALE)
                 ? sv_cmp_locale(left, right)
                 : sv_cmp(left, right));
-      SETs( cmp >= 0 ? &sv_yes : &sv_no );
+      SETs(boolSV(cmp >= 0));
       RETURN;
     }
 }
@@ -958,7 +1023,7 @@ PP(pp_seq)
     dSP; tryAMAGICbinSET(seq,0); 
     {
       dPOPTOPssrl;
-      SETs( sv_eq(left, right) ? &sv_yes : &sv_no );
+      SETs(boolSV(sv_eq(left, right)));
       RETURN;
     }
 }
@@ -968,7 +1033,7 @@ PP(pp_sne)
     dSP; tryAMAGICbinSET(sne,0); 
     {
       dPOPTOPssrl;
-      SETs( !sv_eq(left, right) ? &sv_yes : &sv_no );
+      SETs(boolSV(!sv_eq(left, right)));
       RETURN;
     }
 }
@@ -994,11 +1059,11 @@ PP(pp_bit_and)
       if (SvNIOKp(left) || SvNIOKp(right)) {
        if (op->op_private & HINT_INTEGER) {
          IBW value = SvIV(left) & SvIV(right); 
-         SETi( value );
+         SETi(BWi(value));
        }
        else {
          UBW value = SvUV(left) & SvUV(right); 
-         SETu( value );
+         SETu(BWu(value));
        }
       }
       else {
@@ -1017,11 +1082,11 @@ PP(pp_bit_xor)
       if (SvNIOKp(left) || SvNIOKp(right)) {
        if (op->op_private & HINT_INTEGER) {
          IBW value = (USE_LEFT(left) ? SvIV(left) : 0) ^ SvIV(right); 
-         SETi( value );
+         SETi(BWi(value));
        }
        else {
          UBW value = (USE_LEFT(left) ? SvUV(left) : 0) ^ SvUV(right); 
-         SETu( value );
+         SETu(BWu(value));
        }
       }
       else {
@@ -1040,11 +1105,11 @@ PP(pp_bit_or)
       if (SvNIOKp(left) || SvNIOKp(right)) {
        if (op->op_private & HINT_INTEGER) {
          IBW value = (USE_LEFT(left) ? SvIV(left) : 0) | SvIV(right); 
-         SETi( value );
+         SETi(BWi(value));
        }
        else {
          UBW value = (USE_LEFT(left) ? SvUV(left) : 0) | SvUV(right); 
-         SETu( value );
+         SETu(BWu(value));
        }
       }
       else {
@@ -1092,7 +1157,7 @@ PP(pp_not)
 #ifdef OVERLOAD
     dSP; tryAMAGICunSET(not);
 #endif /* OVERLOAD */
-    *stack_sp = SvTRUE(*stack_sp) ? &sv_no : &sv_yes;
+    *stack_sp = boolSV(!SvTRUE(*stack_sp));
     return NORMAL;
 }
 
@@ -1104,11 +1169,11 @@ PP(pp_complement)
       if (SvNIOKp(sv)) {
        if (op->op_private & HINT_INTEGER) {
          IBW value = ~SvIV(sv);
-         SETi( value );
+         SETi(BWi(value));
        }
        else {
          UBW value = ~SvUV(sv);
-         SETu( value );
+         SETu(BWu(value));
        }
       }
       else {
@@ -1199,7 +1264,7 @@ PP(pp_i_lt)
     dSP; tryAMAGICbinSET(lt,0); 
     {
       dPOPTOPiirl;
-      SETs((left < right) ? &sv_yes : &sv_no);
+      SETs(boolSV(left < right));
       RETURN;
     }
 }
@@ -1209,7 +1274,7 @@ PP(pp_i_gt)
     dSP; tryAMAGICbinSET(gt,0); 
     {
       dPOPTOPiirl;
-      SETs((left > right) ? &sv_yes : &sv_no);
+      SETs(boolSV(left > right));
       RETURN;
     }
 }
@@ -1219,7 +1284,7 @@ PP(pp_i_le)
     dSP; tryAMAGICbinSET(le,0); 
     {
       dPOPTOPiirl;
-      SETs((left <= right) ? &sv_yes : &sv_no);
+      SETs(boolSV(left <= right));
       RETURN;
     }
 }
@@ -1229,7 +1294,7 @@ PP(pp_i_ge)
     dSP; tryAMAGICbinSET(ge,0); 
     {
       dPOPTOPiirl;
-      SETs((left >= right) ? &sv_yes : &sv_no);
+      SETs(boolSV(left >= right));
       RETURN;
     }
 }
@@ -1239,7 +1304,7 @@ PP(pp_i_eq)
     dSP; tryAMAGICbinSET(eq,0); 
     {
       dPOPTOPiirl;
-      SETs((left == right) ? &sv_yes : &sv_no);
+      SETs(boolSV(left == right));
       RETURN;
     }
 }
@@ -1249,7 +1314,7 @@ PP(pp_i_ne)
     dSP; tryAMAGICbinSET(ne,0); 
     {
       dPOPTOPiirl;
-      SETs((left != right) ? &sv_yes : &sv_no);
+      SETs(boolSV(left != right));
       RETURN;
     }
 }
@@ -1388,9 +1453,10 @@ seed()
     U32 u;
 #ifdef VMS
 #  include <starlet.h>
+    /* when[] = (low 32 bits, high 32 bits) of time since epoch
+     * in 100-ns units, typically incremented ever 10 ms.        */
     unsigned int when[2];
     _ckvmssts(sys$gettim(when));
-    /* Please tell us:  Which value is seconds and what is the other here? */
     u = (U32)SEED_C1 * when[0] + (U32)SEED_C2 * when[1];
 #else
 #  ifdef HAS_GETTIMEOFDAY
@@ -1404,9 +1470,9 @@ seed()
 #  endif
 #endif
     u += SEED_C3 * (U32)getpid();
-    u += SEED_C4 * (U32)stack_sp;
+    u += SEED_C4 * (U32)(UV)stack_sp;
 #ifndef PLAN9           /* XXX Plan9 assembler chokes on this; fix needed  */
-    u += SEED_C5 * (U32)&when;
+    u += SEED_C5 * (U32)(UV)&when;
 #endif
     return u;
 }
@@ -2021,22 +2087,23 @@ PP(pp_each)
     dSP; dTARGET;
     HV *hash = (HV*)POPs;
     HE *entry;
+    I32 gimme = GIMME_V;
     
     PUTBACK;
-    entry = hv_iternext(hash);                        /* might clobber stack_sp */
+    entry = hv_iternext(hash);         /* might clobber stack_sp */
     SPAGAIN;
 
     EXTEND(SP, 2);
     if (entry) {
-       PUSHs(hv_iterkeysv(entry));                   /* won't clobber stack_sp */
-       if (GIMME == G_ARRAY) {
+       PUSHs(hv_iterkeysv(entry));     /* won't clobber stack_sp */
+       if (gimme == G_ARRAY) {
            PUTBACK;
-           sv_setsv(TARG, hv_iterval(hash, entry));  /* might clobber stack_sp */
+           sv_setsv(TARG, hv_iterval(hash, entry));  /* might hit stack_sp */
            SPAGAIN;
            PUSHs(TARG);
        }
     }
-    else if (GIMME == G_SCALAR)
+    else if (gimme == G_SCALAR)
        RETPUSHUNDEF;
 
     RETURN;
@@ -2055,6 +2122,8 @@ PP(pp_keys)
 PP(pp_delete)
 {
     dSP;
+    I32 gimme = GIMME_V;
+    I32 discard = (gimme == G_VOID) ? G_DISCARD : 0;
     SV *sv;
     HV *hv;
 
@@ -2064,11 +2133,12 @@ PP(pp_delete)
        if (SvTYPE(hv) != SVt_PVHV)
            DIE("Not a HASH reference");
        while (++MARK <= SP) {
-           sv = hv_delete_ent(hv, *MARK,
-                       (op->op_private & OPpLEAVE_VOID ? G_DISCARD : 0), 0);
+           sv = hv_delete_ent(hv, *MARK, discard, 0);
            *MARK = sv ? sv : &sv_undef;
        }
-       if (GIMME != G_ARRAY) {
+       if (discard)
+           SP = ORIGMARK;
+       else if (gimme == G_SCALAR) {
            MARK = ORIGMARK;
            *++MARK = *SP;
            SP = MARK;
@@ -2079,11 +2149,11 @@ PP(pp_delete)
        hv = (HV*)POPs;
        if (SvTYPE(hv) != SVt_PVHV)
            DIE("Not a HASH reference");
-       sv = hv_delete_ent(hv, keysv,
-                       (op->op_private & OPpLEAVE_VOID ? G_DISCARD : 0), 0);
+       sv = hv_delete_ent(hv, keysv, discard, 0);
        if (!sv)
            sv = &sv_undef;
-       PUSHs(sv);
+       if (!discard)
+           PUSHs(sv);
     }
     RETURN;
 }
@@ -2512,7 +2582,7 @@ PP(pp_reverse)
        if (SP - MARK > 1)
            do_join(TARG, &sv_no, MARK, SP);
        else
-           sv_setsv(TARG, *SP);
+           sv_setsv(TARG, (SP > MARK) ? *SP : GvSV(defgv));
        up = SvPV_force(TARG, len);
        if (len > 1) {
            down = SvPVX(TARG) + len - 1;
@@ -2565,6 +2635,7 @@ PP(pp_unpack)
     dSP;
     dPOPPOPssrl;
     SV **oldsp = sp;
+    I32 gimme = GIMME_V;
     SV *sv;
     STRLEN llen;
     STRLEN rlen;
@@ -2598,7 +2669,7 @@ PP(pp_unpack)
     double cdouble;
     static char* bitcount = 0;
 
-    if (GIMME != G_ARRAY) {            /* arrange to do first one only */
+    if (gimme != G_ARRAY) {            /* arrange to do first one only */
        /*SUPPRESS 530*/
        for (patend = pat; !isALPHA(*patend) || *patend == 'x'; patend++) ;
        if (strchr("aAbBhHP", *patend) || *pat == '%') {
@@ -2611,7 +2682,9 @@ PP(pp_unpack)
     }
     while (pat < patend) {
       reparse:
-       datumtype = *pat++;
+       datumtype = *pat++ & 0xFF;
+       if (isSPACE(datumtype))
+           continue;
        if (pat >= patend)
            len = 1;
        else if (*pat == '*') {
@@ -2627,7 +2700,7 @@ PP(pp_unpack)
            len = (datumtype != '@');
        switch(datumtype) {
        default:
-           break;
+           croak("Invalid type in unpack: '%c'", (int)datumtype);
        case '%':
            if (len == 1 && pat[-1] != '1')
                len = 16;
@@ -2819,13 +2892,13 @@ PP(pp_unpack)
            }
            break;
        case 's':
-           along = (strend - s) / sizeof(I16);
+           along = (strend - s) / SIZE16;
            if (len > along)
                len = along;
            if (checksum) {
                while (len-- > 0) {
-                   Copy(s, &ashort, 1, I16);
-                   s += sizeof(I16);
+                   COPY16(s, &ashort);
+                   s += SIZE16;
                    culong += ashort;
                }
            }
@@ -2833,8 +2906,8 @@ PP(pp_unpack)
                EXTEND(SP, len);
                EXTEND_MORTAL(len);
                while (len-- > 0) {
-                   Copy(s, &ashort, 1, I16);
-                   s += sizeof(I16);
+                   COPY16(s, &ashort);
+                   s += SIZE16;
                    sv = NEWSV(38, 0);
                    sv_setiv(sv, (IV)ashort);
                    PUSHs(sv_2mortal(sv));
@@ -2844,13 +2917,13 @@ PP(pp_unpack)
        case 'v':
        case 'n':
        case 'S':
-           along = (strend - s) / sizeof(U16);
+           along = (strend - s) / SIZE16;
            if (len > along)
                len = along;
            if (checksum) {
                while (len-- > 0) {
-                   Copy(s, &aushort, 1, U16);
-                   s += sizeof(U16);
+                   COPY16(s, &aushort);
+                   s += SIZE16;
 #ifdef HAS_NTOHS
                    if (datumtype == 'n')
                        aushort = ntohs(aushort);
@@ -2866,8 +2939,8 @@ PP(pp_unpack)
                EXTEND(SP, len);
                EXTEND_MORTAL(len);
                while (len-- > 0) {
-                   Copy(s, &aushort, 1, U16);
-                   s += sizeof(U16);
+                   COPY16(s, &aushort);
+                   s += SIZE16;
                    sv = NEWSV(39, 0);
 #ifdef HAS_NTOHS
                    if (datumtype == 'n')
@@ -2935,13 +3008,13 @@ PP(pp_unpack)
            }
            break;
        case 'l':
-           along = (strend - s) / sizeof(I32);
+           along = (strend - s) / SIZE32;
            if (len > along)
                len = along;
            if (checksum) {
                while (len-- > 0) {
-                   Copy(s, &along, 1, I32);
-                   s += sizeof(I32);
+                   COPY32(s, &along);
+                   s += SIZE32;
                    if (checksum > 32)
                        cdouble += (double)along;
                    else
@@ -2952,8 +3025,8 @@ PP(pp_unpack)
                EXTEND(SP, len);
                EXTEND_MORTAL(len);
                while (len-- > 0) {
-                   Copy(s, &along, 1, I32);
-                   s += sizeof(I32);
+                   COPY32(s, &along);
+                   s += SIZE32;
                    sv = NEWSV(42, 0);
                    sv_setiv(sv, (IV)along);
                    PUSHs(sv_2mortal(sv));
@@ -2963,13 +3036,13 @@ PP(pp_unpack)
        case 'V':
        case 'N':
        case 'L':
-           along = (strend - s) / sizeof(U32);
+           along = (strend - s) / SIZE32;
            if (len > along)
                len = along;
            if (checksum) {
                while (len-- > 0) {
-                   Copy(s, &aulong, 1, U32);
-                   s += sizeof(U32);
+                   COPY32(s, &aulong);
+                   s += SIZE32;
 #ifdef HAS_NTOHL
                    if (datumtype == 'N')
                        aulong = ntohl(aulong);
@@ -2988,8 +3061,8 @@ PP(pp_unpack)
                EXTEND(SP, len);
                EXTEND_MORTAL(len);
                while (len-- > 0) {
-                   Copy(s, &aulong, 1, U32);
-                   s += sizeof(U32);
+                   COPY32(s, &aulong);
+                   s += SIZE32;
 #ifdef HAS_NTOHL
                    if (datumtype == 'N')
                        aulong = ntohl(aulong);
@@ -3092,7 +3165,10 @@ PP(pp_unpack)
                    s += sizeof(Quad_t);
                }
                sv = NEWSV(42, 0);
-               sv_setiv(sv, (IV)aquad);
+               if (aquad >= IV_MIN && aquad <= IV_MAX)
+                   sv_setiv(sv, (IV)aquad);
+               else
+                   sv_setnv(sv, (double)aquad);
                PUSHs(sv_2mortal(sv));
            }
            break;
@@ -3107,7 +3183,10 @@ PP(pp_unpack)
                    s += sizeof(unsigned Quad_t);
                }
                sv = NEWSV(43, 0);
-               sv_setuv(sv, (UV)auquad);
+               if (aquad <= UV_MAX)
+                   sv_setuv(sv, (UV)auquad);
+               else
+                   sv_setnv(sv, (double)auquad);
                PUSHs(sv_2mortal(sv));
            }
            break;
@@ -3228,16 +3307,16 @@ PP(pp_unpack)
            }
            else {
                if (checksum < 32) {
-                   along = (1 << checksum) - 1;
-                   culong &= (U32)along;
+                   aulong = (1 << checksum) - 1;
+                   culong &= aulong;
                }
-               sv_setnv(sv, (double)culong);
+               sv_setuv(sv, (UV)culong);
            }
            XPUSHs(sv_2mortal(sv));
            checksum = 0;
        }
     }
-    if (sp == oldsp && GIMME != G_ARRAY)
+    if (sp == oldsp && gimme == G_SCALAR)
        PUSHs(&sv_undef);
     RETURN;
 }
@@ -3383,7 +3462,9 @@ PP(pp_pack)
     sv_setpvn(cat, "", 0);
     while (pat < patend) {
 #define NEXTFROM (items-- > 0 ? *MARK++ : &sv_no)
-       datumtype = *pat++;
+       datumtype = *pat++ & 0xFF;
+       if (isSPACE(datumtype))
+           continue;
        if (*pat == '*') {
            len = strchr("@Xxu", datumtype) ? 0 : items;
            pat++;
@@ -3397,7 +3478,7 @@ PP(pp_pack)
            len = 1;
        switch(datumtype) {
        default:
-           break;
+           croak("Invalid type in pack: '%c'", (int)datumtype);
        case '%':
            DIE("%% may only be used in unpack");
        case '@':
@@ -3599,7 +3680,7 @@ PP(pp_pack)
 #ifdef HAS_HTONS
                ashort = htons(ashort);
 #endif
-               sv_catpvn(cat, (char*)&ashort, sizeof(I16));
+               CAT16(cat, &ashort);
            }
            break;
        case 'v':
@@ -3609,7 +3690,7 @@ PP(pp_pack)
 #ifdef HAS_HTOVS
                ashort = htovs(ashort);
 #endif
-               sv_catpvn(cat, (char*)&ashort, sizeof(I16));
+               CAT16(cat, &ashort);
            }
            break;
        case 'S':
@@ -3617,13 +3698,13 @@ PP(pp_pack)
            while (len-- > 0) {
                fromstr = NEXTFROM;
                ashort = (I16)SvIV(fromstr);
-               sv_catpvn(cat, (char*)&ashort, sizeof(I16));
+               CAT16(cat, &ashort);
            }
            break;
        case 'I':
            while (len-- > 0) {
                fromstr = NEXTFROM;
-               auint = U_I(SvNV(fromstr));
+               auint = SvUV(fromstr);
                sv_catpvn(cat, (char*)&auint, sizeof(unsigned int));
            }
            break;
@@ -3696,35 +3777,35 @@ PP(pp_pack)
        case 'N':
            while (len-- > 0) {
                fromstr = NEXTFROM;
-               aulong = U_L(SvNV(fromstr));
+               aulong = SvUV(fromstr);
 #ifdef HAS_HTONL
                aulong = htonl(aulong);
 #endif
-               sv_catpvn(cat, (char*)&aulong, sizeof(U32));
+               CAT32(cat, &aulong);
            }
            break;
        case 'V':
            while (len-- > 0) {
                fromstr = NEXTFROM;
-               aulong = U_L(SvNV(fromstr));
+               aulong = SvUV(fromstr);
 #ifdef HAS_HTOVL
                aulong = htovl(aulong);
 #endif
-               sv_catpvn(cat, (char*)&aulong, sizeof(U32));
+               CAT32(cat, &aulong);
            }
            break;
        case 'L':
            while (len-- > 0) {
                fromstr = NEXTFROM;
-               aulong = U_L(SvNV(fromstr));
-               sv_catpvn(cat, (char*)&aulong, sizeof(U32));
+               aulong = SvUV(fromstr);
+               CAT32(cat, &aulong);
            }
            break;
        case 'l':
            while (len-- > 0) {
                fromstr = NEXTFROM;
                along = SvIV(fromstr);
-               sv_catpvn(cat, (char*)&along, sizeof(I32));
+               CAT32(cat, &along);
            }
            break;
 #ifdef HAS_QUAD
@@ -3803,7 +3884,7 @@ PP(pp_split)
     I32 realarray = 0;
     I32 base;
     AV *oldstack = curstack;
-    I32 gimme = GIMME;
+    I32 gimme = GIMME_V;
     I32 oldsave = savestack_ix;
 
 #ifdef DEBUGGING