This is a live mirror of the Perl 5 development currently hosted at https://github.com/perl/perl5
Free tied hash iterator state immediately at the `untie` call
[perl5.git] / pp_sys.c
index 7e13e38..5f6370d 100644 (file)
--- a/pp_sys.c
+++ b/pp_sys.c
@@ -76,18 +76,16 @@ extern int h_errno;
 #ifdef HAS_PASSWD
 # ifdef I_PWD
 #  include <pwd.h>
-# else
-#  if !defined(VMS)
+# elif !defined(VMS)
     struct passwd *getpwnam (char *);
     struct passwd *getpwuid (Uid_t);
-#  endif
 # endif
 # ifdef HAS_GETPWENT
-#ifndef getpwent
+#  ifndef getpwent
   struct passwd *getpwent (void);
-#elif defined (VMS) && defined (my_getpwent)
+#  elif defined (VMS) && defined (my_getpwent)
   struct passwd *Perl_my_getpwent (pTHX);
-#endif
+#  endif
 # endif
 #endif
 
@@ -99,9 +97,9 @@ extern int h_errno;
     struct group *getgrgid (Gid_t);
 # endif
 # ifdef HAS_GETGRENT
-#ifndef getgrent
+#  ifndef getgrent
     struct group *getgrent (void);
-#endif
+#  endif
 # endif
 #endif
 
@@ -118,12 +116,10 @@ extern int h_errno;
 #   undef my_chsize
 # endif
 # define my_chsize PerlLIO_chsize
+#elif defined(HAS_TRUNCATE)
+# define my_chsize PerlLIO_chsize
 #else
-# ifdef HAS_TRUNCATE
-#   define my_chsize PerlLIO_chsize
-# else
 I32 my_chsize(int fd, Off_t length);
-# endif
 #endif
 
 #ifdef HAS_FLOCK
@@ -141,12 +137,10 @@ I32 my_chsize(int fd, Off_t length);
 #  if defined(HAS_FCNTL) && defined(FCNTL_CAN_LOCK)
 #    define FLOCK fcntl_emulate_flock
 #    define FCNTL_EMULATE_FLOCK
-#  else /* no flock() or fcntl(F_SETLK,...) */
-#    ifdef HAS_LOCKF
-#      define FLOCK lockf_emulate_flock
-#      define LOCKF_EMULATE_FLOCK
-#    endif /* lockf */
-#  endif /* no flock() or fcntl(F_SETLK,...) */
+#  elif defined(HAS_LOCKF)
+#    define FLOCK lockf_emulate_flock
+#    define LOCKF_EMULATE_FLOCK
+#  endif
 
 #  ifdef FLOCK
      static int FLOCK (int, int);
@@ -219,14 +213,14 @@ void endservent(void);
 #endif
 
 #if !defined(PERL_EFF_ACCESS) && defined(HAS_ACCESSX) && defined(ACC_SELF)
-    /* AIX */
-#   define PERL_EFF_ACCESS(p,f) (accessx((p), (f), ACC_SELF))
+    /* AIX's accessx() doesn't declare its argument const, unlike every other platform */
+#   define PERL_EFF_ACCESS(p,f) (accessx((char*)(p), (f), ACC_SELF))
 #endif
 
 
 #if !defined(PERL_EFF_ACCESS) && defined(HAS_ACCESS)   \
     && (defined(HAS_SETREUID) || defined(HAS_SETRESUID)                \
-       || defined(HAS_SETREGID) || defined(HAS_SETRESGID))
+        || defined(HAS_SETREGID) || defined(HAS_SETRESGID))
 /* The Hard Way. */
 STATIC int
 S_emulate_eaccess(pTHX_ const char* path, Mode_t mode)
@@ -240,52 +234,44 @@ S_emulate_eaccess(pTHX_ const char* path, Mode_t mode)
 #if !defined(HAS_SETREUID) && !defined(HAS_SETRESUID)
     Perl_croak(aTHX_ "switching effective uid is not implemented");
 #else
-#ifdef HAS_SETREUID
+#  ifdef HAS_SETREUID
     if (setreuid(euid, ruid))
-#else
-#ifdef HAS_SETRESUID
+#  elif defined(HAS_SETRESUID)
     if (setresuid(euid, ruid, (Uid_t)-1))
-#endif
-#endif
-       /* diag_listed_as: entering effective %s failed */
-       Perl_croak(aTHX_ "entering effective uid failed");
+#  endif
+        /* diag_listed_as: entering effective %s failed */
+        Perl_croak(aTHX_ "entering effective uid failed");
 #endif
 
 #if !defined(HAS_SETREGID) && !defined(HAS_SETRESGID)
     Perl_croak(aTHX_ "switching effective gid is not implemented");
 #else
-#ifdef HAS_SETREGID
+#  ifdef HAS_SETREGID
     if (setregid(egid, rgid))
-#else
-#ifdef HAS_SETRESGID
+#  elif defined(HAS_SETRESGID)
     if (setresgid(egid, rgid, (Gid_t)-1))
-#endif
-#endif
-       /* diag_listed_as: entering effective %s failed */
-       Perl_croak(aTHX_ "entering effective gid failed");
+#  endif
+        /* diag_listed_as: entering effective %s failed */
+        Perl_croak(aTHX_ "entering effective gid failed");
 #endif
 
     res = access(path, mode);
 
 #ifdef HAS_SETREUID
     if (setreuid(ruid, euid))
-#else
-#ifdef HAS_SETRESUID
+#elif defined(HAS_SETRESUID)
     if (setresuid(ruid, euid, (Uid_t)-1))
 #endif
-#endif
-       /* diag_listed_as: leaving effective %s failed */
-       Perl_croak(aTHX_ "leaving effective uid failed");
+        /* diag_listed_as: leaving effective %s failed */
+        Perl_croak(aTHX_ "leaving effective uid failed");
 
 #ifdef HAS_SETREGID
     if (setregid(rgid, egid))
-#else
-#ifdef HAS_SETRESGID
+#elif defined(HAS_SETRESGID)
     if (setresgid(rgid, egid, (Gid_t)-1))
 #endif
-#endif
-       /* diag_listed_as: leaving effective %s failed */
-       Perl_croak(aTHX_ "leaving effective gid failed");
+        /* diag_listed_as: leaving effective %s failed */
+        Perl_croak(aTHX_ "leaving effective gid failed");
 
     return res;
 }
@@ -302,52 +288,52 @@ PP(pp_backtick)
 
     TAINT_PROPER("``");
     if (PL_op->op_private & OPpOPEN_IN_RAW)
-       mode = "rb";
+        mode = "rb";
     else if (PL_op->op_private & OPpOPEN_IN_CRLF)
-       mode = "rt";
+        mode = "rt";
     fp = PerlProc_popen(tmps, mode);
     if (fp) {
         const char * const type = Perl_PerlIO_context_layers(aTHX_ NULL);
-       if (type && *type)
-           PerlIO_apply_layers(aTHX_ fp,mode,type);
-
-       if (gimme == G_VOID) {
-           char tmpbuf[256];
-           while (PerlIO_read(fp, tmpbuf, sizeof tmpbuf) > 0)
-               NOOP;
-       }
-       else if (gimme == G_SCALAR) {
-           ENTER_with_name("backtick");
-           SAVESPTR(PL_rs);
-           PL_rs = &PL_sv_undef;
-           sv_setpvs(TARG, "");        /* note that this preserves previous buffer */
-           while (sv_gets(TARG, fp, SvCUR(TARG)) != NULL)
-               NOOP;
-           LEAVE_with_name("backtick");
-           XPUSHs(TARG);
-           SvTAINTED_on(TARG);
-       }
-       else {
-           for (;;) {
-               SV * const sv = newSV(79);
-               if (sv_gets(sv, fp, 0) == NULL) {
-                   SvREFCNT_dec(sv);
-                   break;
-               }
-               mXPUSHs(sv);
-               if (SvLEN(sv) - SvCUR(sv) > 20) {
-                   SvPV_shrink_to_cur(sv);
-               }
-               SvTAINTED_on(sv);
-           }
-       }
-       STATUS_NATIVE_CHILD_SET(PerlProc_pclose(fp));
-       TAINT;          /* "I believe that this is not gratuitous!" */
+        if (type && *type)
+            PerlIO_apply_layers(aTHX_ fp,mode,type);
+
+        if (gimme == G_VOID) {
+            char tmpbuf[256];
+            while (PerlIO_read(fp, tmpbuf, sizeof tmpbuf) > 0)
+                NOOP;
+        }
+        else if (gimme == G_SCALAR) {
+            ENTER_with_name("backtick");
+            SAVESPTR(PL_rs);
+            PL_rs = &PL_sv_undef;
+            SvPVCLEAR(TARG);        /* note that this preserves previous buffer */
+            while (sv_gets(TARG, fp, SvCUR(TARG)) != NULL)
+                NOOP;
+            LEAVE_with_name("backtick");
+            XPUSHs(TARG);
+            SvTAINTED_on(TARG);
+        }
+        else {
+            for (;;) {
+                SV * const sv = newSV(79);
+                if (sv_gets(sv, fp, 0) == NULL) {
+                    SvREFCNT_dec(sv);
+                    break;
+                }
+                mXPUSHs(sv);
+                if (SvLEN(sv) - SvCUR(sv) > 20) {
+                    SvPV_shrink_to_cur(sv);
+                }
+                SvTAINTED_on(sv);
+            }
+        }
+        STATUS_NATIVE_CHILD_SET(PerlProc_pclose(fp));
+        TAINT;         /* "I believe that this is not gratuitous!" */
     }
     else {
-       STATUS_NATIVE_CHILD_SET(-1);
-       if (gimme == G_SCALAR)
-           RETPUSHUNDEF;
+        STATUS_NATIVE_CHILD_SET(-1);
+        if (gimme == G_SCALAR)
+            RETPUSHUNDEF;
     }
 
     RETURN;
@@ -368,15 +354,15 @@ PP(pp_glob)
     tryAMAGICunTARGETlist(iter_amg, (PL_op->op_flags & OPf_SPECIAL));
 
     if (PL_op->op_flags & OPf_SPECIAL) {
-       /* call Perl-level glob function instead. Stack args are:
-        * MARK, wildcard
-        * and following OPs should be: gv(CORE::GLOBAL::glob), entersub
-        * */
-       return NORMAL;
+        /* call Perl-level glob function instead. Stack args are:
+         * MARK, wildcard
+         * and following OPs should be: gv(CORE::GLOBAL::glob), entersub
+         * */
+        return NORMAL;
     }
     if (PL_globhook) {
-       PL_globhook(aTHX);
-       return NORMAL;
+        PL_globhook(aTHX);
+        return NORMAL;
     }
 
     /* Note that we only ever get here if File::Glob fails to load
@@ -387,12 +373,12 @@ PP(pp_glob)
 
 #ifndef VMS
     if (TAINTING_get) {
-       /*
-        * The external globbing program may use things we can't control,
-        * so for security reasons we must assume the worst.
-        */
-       TAINT;
-       taint_proper(PL_no_security, "glob");
+        /*
+         * The external globbing program may use things we can't control,
+         * so for security reasons we must assume the worst.
+         */
+        TAINT;
+        taint_proper(PL_no_security, "glob");
     }
 #endif /* !VMS */
 
@@ -424,45 +410,45 @@ PP(pp_warn)
     SV *exsv;
     STRLEN len;
     if (SP - MARK > 1) {
-       dTARGET;
-       do_join(TARG, &PL_sv_no, MARK, SP);
-       exsv = TARG;
-       SP = MARK + 1;
+        dTARGET;
+        do_join(TARG, &PL_sv_no, MARK, SP);
+        exsv = TARG;
+        SP = MARK + 1;
     }
     else if (SP == MARK) {
-       exsv = &PL_sv_no;
-       EXTEND(SP, 1);
-       SP = MARK + 1;
+        exsv = &PL_sv_no;
+        MEXTEND(SP, 1);
+        SP = MARK + 1;
     }
     else {
-       exsv = TOPs;
-       if (SvGMAGICAL(exsv)) exsv = sv_mortalcopy(exsv);
+        exsv = TOPs;
+        if (SvGMAGICAL(exsv)) exsv = sv_mortalcopy(exsv);
     }
 
     if (SvROK(exsv) || (SvPV_const(exsv, len), len)) {
-       /* well-formed exception supplied */
+        /* well-formed exception supplied */
     }
     else {
       SV * const errsv = ERRSV;
       SvGETMAGIC(errsv);
       if (SvROK(errsv)) {
-       if (SvGMAGICAL(errsv)) {
-           exsv = sv_newmortal();
-           sv_setsv_nomg(exsv, errsv);
-       }
-       else exsv = errsv;
+        if (SvGMAGICAL(errsv)) {
+            exsv = sv_newmortal();
+            sv_setsv_nomg(exsv, errsv);
+        }
+        else exsv = errsv;
       }
       else if (SvPOKp(errsv) ? SvCUR(errsv) : SvNIOKp(errsv)) {
-       exsv = sv_newmortal();
-       sv_setsv_nomg(exsv, errsv);
-       sv_catpvs(exsv, "\t...caught");
+        exsv = sv_newmortal();
+        sv_setsv_nomg(exsv, errsv);
+        sv_catpvs(exsv, "\t...caught");
       }
       else {
-       exsv = newSVpvs_flags("Warning: something's wrong", SVs_TEMP);
+        exsv = newSVpvs_flags("Warning: something's wrong", SVs_TEMP);
       }
     }
     if (SvROK(exsv) && !PL_warnhook)
-        Perl_warn(aTHX_ "%"SVf, SVfARG(exsv));
+         Perl_warn(aTHX_ "%" SVf, SVfARG(exsv));
     else warn_sv(exsv);
     RETSETYES;
 }
@@ -474,51 +460,51 @@ PP(pp_die)
     STRLEN len;
 #ifdef VMS
     VMSISH_HUSHED  =
-       VMSISH_HUSHED || (PL_curcop->op_private & OPpHUSH_VMSISH);
+        VMSISH_HUSHED || (PL_curcop->op_private & OPpHUSH_VMSISH);
 #endif
     if (SP - MARK != 1) {
-       dTARGET;
-       do_join(TARG, &PL_sv_no, MARK, SP);
-       exsv = TARG;
-       SP = MARK + 1;
+        dTARGET;
+        do_join(TARG, &PL_sv_no, MARK, SP);
+        exsv = TARG;
+        SP = MARK + 1;
     }
     else {
-       exsv = TOPs;
+        exsv = TOPs;
     }
 
     if (SvROK(exsv) || (SvPV_const(exsv, len), len)) {
-       /* well-formed exception supplied */
+        /* well-formed exception supplied */
     }
     else {
-       SV * const errsv = ERRSV;
-       SvGETMAGIC(errsv);
-       if (SvROK(errsv)) {
-           exsv = errsv;
-           if (sv_isobject(exsv)) {
-               HV * const stash = SvSTASH(SvRV(exsv));
-               GV * const gv = gv_fetchmethod(stash, "PROPAGATE");
-               if (gv) {
-                   SV * const file = sv_2mortal(newSVpv(CopFILE(PL_curcop),0));
-                   SV * const line = sv_2mortal(newSVuv(CopLINE(PL_curcop)));
-                   EXTEND(SP, 3);
-                   PUSHMARK(SP);
-                   PUSHs(exsv);
-                   PUSHs(file);
-                   PUSHs(line);
-                   PUTBACK;
-                   call_sv(MUTABLE_SV(GvCV(gv)),
-                           G_SCALAR|G_EVAL|G_KEEPERR);
-                   exsv = sv_mortalcopy(*PL_stack_sp--);
-               }
-           }
-       }
-       else if (SvPOK(errsv) && SvCUR(errsv)) {
-           exsv = sv_mortalcopy(errsv);
-           sv_catpvs(exsv, "\t...propagated");
-       }
-       else {
-           exsv = newSVpvs_flags("Died", SVs_TEMP);
-       }
+        SV * const errsv = ERRSV;
+        SvGETMAGIC(errsv);
+        if (SvROK(errsv)) {
+            exsv = errsv;
+            if (sv_isobject(exsv)) {
+                HV * const stash = SvSTASH(SvRV(exsv));
+                GV * const gv = gv_fetchmethod(stash, "PROPAGATE");
+                if (gv) {
+                    SV * const file = sv_2mortal(newSVpv(CopFILE(PL_curcop),0));
+                    SV * const line = sv_2mortal(newSVuv(CopLINE(PL_curcop)));
+                    EXTEND(SP, 3);
+                    PUSHMARK(SP);
+                    PUSHs(exsv);
+                    PUSHs(file);
+                    PUSHs(line);
+                    PUTBACK;
+                    call_sv(MUTABLE_SV(GvCV(gv)),
+                            G_SCALAR|G_EVAL|G_KEEPERR);
+                    exsv = sv_mortalcopy(*PL_stack_sp--);
+                }
+            }
+        }
+        else if (SvOK(errsv) && (SvPV_nomg(errsv,len), len)) {
+            exsv = sv_mortalcopy(errsv);
+            sv_catpvs(exsv, "\t...propagated");
+        }
+        else {
+            exsv = newSVpvs_flags("Died", SVs_TEMP);
+        }
     }
     die_sv(exsv);
     NOT_REACHED; /* NOTREACHED */
@@ -529,7 +515,7 @@ PP(pp_die)
 
 OP *
 Perl_tied_method(pTHX_ SV *methname, SV **sp, SV *const sv,
-                const MAGIC *const mg, const U32 flags, U32 argc, ...)
+                 const MAGIC *const mg, const U32 flags, U32 argc, ...)
 {
     SV **orig_sp = sp;
     I32 ret_args;
@@ -561,30 +547,30 @@ Perl_tied_method(pTHX_ SV *methname, SV **sp, SV *const sv,
     PUSHMARK(sp);
     PUSHs(SvTIED_obj(sv, mg));
     if (flags & TIED_METHOD_ARGUMENTS_ON_STACK) {
-       Copy(orig_sp + 2, sp + 1, argc, SV*); /* copy args to new stack */
-       sp += argc;
+        Copy(orig_sp + 2, sp + 1, argc, SV*); /* copy args to new stack */
+        sp += argc;
     }
     else if (argc) {
-       const U32 mortalize_not_needed
-           = flags & TIED_METHOD_MORTALIZE_NOT_NEEDED;
-       va_list args;
-       va_start(args, argc);
-       do {
-           SV *const arg = va_arg(args, SV *);
-           if(mortalize_not_needed)
-               PUSHs(arg);
-           else
-               mPUSHs(arg);
-       } while (--argc);
-       va_end(args);
+        const U32 mortalize_not_needed
+            = flags & TIED_METHOD_MORTALIZE_NOT_NEEDED;
+        va_list args;
+        va_start(args, argc);
+        do {
+            SV *const arg = va_arg(args, SV *);
+            if(mortalize_not_needed)
+                PUSHs(arg);
+            else
+                mPUSHs(arg);
+        } while (--argc);
+        va_end(args);
     }
 
     PUTBACK;
     ENTER_with_name("call_tied_method");
     if (flags & TIED_METHOD_SAY) {
-       /* local $\ = "\n" */
-       SAVEGENERICSV(PL_ors_sv);
-       PL_ors_sv = newSVpvs("\n");
+        /* local $\ = "\n" */
+        SAVEGENERICSV(PL_ors_sv);
+        PL_ors_sv = newSVpvs("\n");
     }
     ret_args = call_sv(methname, (flags & G_WANT)|G_METHOD_NAMED);
     SPAGAIN;
@@ -592,10 +578,10 @@ Perl_tied_method(pTHX_ SV *methname, SV **sp, SV *const sv,
     POPSTACK;
     SPAGAIN;
     if (ret_args) { /* copy results back to original stack */
-       EXTEND(sp, ret_args);
-       Copy(orig_sp - ret_args + 1, sp + 1, ret_args, SV*);
-       sp += ret_args;
-       PUTBACK;
+        EXTEND(sp, ret_args);
+        Copy(orig_sp - ret_args + 1, sp + 1, ret_args, SV*);
+        sp += ret_args;
+        PUTBACK;
     }
     LEAVE_with_name("call_tied_method");
     return NORMAL;
@@ -622,63 +608,64 @@ PP(pp_open)
     GV * const gv = MUTABLE_GV(*++MARK);
 
     if (!isGV(gv) && !(SvTYPE(gv) == SVt_PVLV && isGV_with_GP(gv)))
-       DIE(aTHX_ PL_no_usym, "filehandle");
+        DIE(aTHX_ PL_no_usym, "filehandle");
 
     if ((io = GvIOp(gv))) {
-       const MAGIC *mg;
-       IoFLAGS(GvIOp(gv)) &= ~IOf_UNTAINT;
-
-       if (IoDIRP(io))
-           Perl_ck_warner_d(aTHX_ packWARN2(WARN_IO, WARN_DEPRECATED),
-                            "Opening dirhandle %"HEKf" also as a file",
-                            HEKfARG(GvENAME_HEK(gv)));
-
-       mg = SvTIED_mg((const SV *)io, PERL_MAGIC_tiedscalar);
-       if (mg) {
-           /* Method's args are same as ours ... */
-           /* ... except handle is replaced by the object */
-           return Perl_tied_method(aTHX_ SV_CONST(OPEN), mark - 1, MUTABLE_SV(io), mg,
-                                   G_SCALAR | TIED_METHOD_ARGUMENTS_ON_STACK,
-                                   sp - mark);
-       }
+        const MAGIC *mg;
+        IoFLAGS(GvIOp(gv)) &= ~IOf_UNTAINT;
+
+        if (IoDIRP(io))
+            Perl_croak(aTHX_ "Cannot open %" HEKf " as a filehandle: it is already open as a dirhandle",
+                             HEKfARG(GvENAME_HEK(gv)));
+
+        mg = SvTIED_mg((const SV *)io, PERL_MAGIC_tiedscalar);
+        if (mg) {
+            /* Method's args are same as ours ... */
+            /* ... except handle is replaced by the object */
+            return Perl_tied_method(aTHX_ SV_CONST(OPEN), mark - 1, MUTABLE_SV(io), mg,
+                                    G_SCALAR | TIED_METHOD_ARGUMENTS_ON_STACK,
+                                    sp - mark);
+        }
     }
 
     if (MARK < SP) {
-       sv = *++MARK;
+        sv = *++MARK;
     }
     else {
-       sv = GvSVn(gv);
+        sv = GvSVn(gv);
     }
 
     tmps = SvPV_const(sv, len);
     ok = do_open6(gv, tmps, len, NULL, MARK+1, (SP-MARK));
     SP = ORIGMARK;
     if (ok)
-       PUSHi( (I32)PL_forkprocess );
+        PUSHi( (I32)PL_forkprocess );
     else if (PL_forkprocess == 0)              /* we are a new child */
-       PUSHi(0);
+        PUSHs(&PL_sv_zero);
     else
-       RETPUSHUNDEF;
+        RETPUSHUNDEF;
     RETURN;
 }
 
 PP(pp_close)
 {
     dSP;
+    /* pp_coreargs pushes a NULL to indicate no args passed to
+     * CORE::close() */
     GV * const gv =
-       MAXARG == 0 || (!TOPs && !POPs) ? PL_defoutgv : MUTABLE_GV(POPs);
+        MAXARG == 0 || (!TOPs && !POPs) ? PL_defoutgv : MUTABLE_GV(POPs);
 
     if (MAXARG == 0)
-       EXTEND(SP, 1);
+        EXTEND(SP, 1);
 
     if (gv) {
-       IO * const io = GvIO(gv);
-       if (io) {
-           const MAGIC * const mg = SvTIED_mg((const SV *)io, PERL_MAGIC_tiedscalar);
-           if (mg) {
-               return tied_method0(SV_CONST(CLOSE), SP, MUTABLE_SV(io), mg);
-           }
-       }
+        IO * const io = GvIO(gv);
+        if (io) {
+            const MAGIC * const mg = SvTIED_mg((const SV *)io, PERL_MAGIC_tiedscalar);
+            if (mg) {
+                return tied_method0(SV_CONST(CLOSE), SP, MUTABLE_SV(io), mg);
+            }
+        }
     }
     PUSHs(boolSV(do_close(gv, TRUE)));
     RETURN;
@@ -697,39 +684,33 @@ PP(pp_pipe_op)
 
     rstio = GvIOn(rgv);
     if (IoIFP(rstio))
-       do_close(rgv, FALSE);
+        do_close(rgv, FALSE);
 
     wstio = GvIOn(wgv);
     if (IoIFP(wstio))
-       do_close(wgv, FALSE);
+        do_close(wgv, FALSE);
 
-    if (PerlProc_pipe(fd) < 0)
-       goto badexit;
+    if (PerlProc_pipe_cloexec(fd) < 0)
+        goto badexit;
 
-    IoIFP(rstio) = PerlIO_fdopen(fd[0], "r"PIPE_OPEN_MODE);
-    IoOFP(wstio) = PerlIO_fdopen(fd[1], "w"PIPE_OPEN_MODE);
+    IoIFP(rstio) = PerlIO_fdopen(fd[0], "r" PIPE_OPEN_MODE);
+    IoOFP(wstio) = PerlIO_fdopen(fd[1], "w" PIPE_OPEN_MODE);
     IoOFP(rstio) = IoIFP(rstio);
     IoIFP(wstio) = IoOFP(wstio);
     IoTYPE(rstio) = IoTYPE_RDONLY;
     IoTYPE(wstio) = IoTYPE_WRONLY;
 
     if (!IoIFP(rstio) || !IoOFP(wstio)) {
-       if (IoIFP(rstio))
-           PerlIO_close(IoIFP(rstio));
-       else
-           PerlLIO_close(fd[0]);
-       if (IoOFP(wstio))
-           PerlIO_close(IoOFP(wstio));
-       else
-           PerlLIO_close(fd[1]);
-       goto badexit;
-    }
-#if defined(HAS_FCNTL) && defined(F_SETFD) && defined(FD_CLOEXEC)
-    /* ensure close-on-exec */
-    if ((fd[0] > PL_maxsysfd && fcntl(fd[0], F_SETFD, FD_CLOEXEC) < 0) ||
-        (fd[1] > PL_maxsysfd && fcntl(fd[1], F_SETFD, FD_CLOEXEC) < 0))
+        if (IoIFP(rstio))
+            PerlIO_close(IoIFP(rstio));
+        else
+            PerlLIO_close(fd[0]);
+        if (IoOFP(wstio))
+            PerlIO_close(IoOFP(wstio));
+        else
+            PerlLIO_close(fd[1]);
         goto badexit;
-#endif
+    }
     RETPUSHYES;
 
   badexit:
@@ -748,14 +729,14 @@ PP(pp_fileno)
     const MAGIC *mg;
 
     if (MAXARG < 1)
-       RETPUSHUNDEF;
+        RETPUSHUNDEF;
     gv = MUTABLE_GV(POPs);
     io = GvIO(gv);
 
     if (io
-       && (mg = SvTIED_mg((const SV *)io, PERL_MAGIC_tiedscalar)))
+        && (mg = SvTIED_mg((const SV *)io, PERL_MAGIC_tiedscalar)))
     {
-       return tied_method0(SV_CONST(FILENO), SP, MUTABLE_SV(io), mg);
+        return tied_method0(SV_CONST(FILENO), SP, MUTABLE_SV(io), mg);
     }
 
     if (io && IoDIRP(io)) {
@@ -775,12 +756,12 @@ PP(pp_fileno)
     }
 
     if (!io || !(fp = IoIFP(io))) {
-       /* Can't do this because people seem to do things like
-          defined(fileno($foo)) to check whether $foo is a valid fh.
+        /* Can't do this because people seem to do things like
+           defined(fileno($foo)) to check whether $foo is a valid fh.
 
-          report_evil_fh(gv);
-           */
-       RETPUSHUNDEF;
+           report_evil_fh(gv);
+            */
+        RETPUSHUNDEF;
     }
 
     PUSHi(PerlIO_fileno(fp));
@@ -795,15 +776,15 @@ PP(pp_umask)
     Mode_t anum;
 
     if (MAXARG < 1 || (!TOPs && !POPs)) {
-       anum = PerlLIO_umask(022);
-       /* setting it to 022 between the two calls to umask avoids
-        * to have a window where the umask is set to 0 -- meaning
-        * that another thread could create world-writeable files. */
-       if (anum != 022)
-           (void)PerlLIO_umask(anum);
+        anum = PerlLIO_umask(022);
+        /* setting it to 022 between the two calls to umask avoids
+         * to have a window where the umask is set to 0 -- meaning
+         * that another thread could create world-writeable files. */
+        if (anum != 022)
+            (void)PerlLIO_umask(anum);
     }
     else
-       anum = PerlLIO_umask(POPi);
+        anum = PerlLIO_umask(POPi);
     TAINT_PROPER("umask");
     XPUSHi(anum);
 #else
@@ -811,7 +792,7 @@ PP(pp_umask)
      * Otherwise it's harmless and more useful to just return undef
      * since 'group' and 'other' concepts probably don't exist here. */
     if (MAXARG >= 1 && (TOPs||POPs) && (POPi & 0700))
-       DIE(aTHX_ "umask not implemented");
+        DIE(aTHX_ "umask not implemented");
     XPUSHs(&PL_sv_undef);
 #endif
     RETURN;
@@ -826,55 +807,55 @@ PP(pp_binmode)
     SV *discp = NULL;
 
     if (MAXARG < 1)
-       RETPUSHUNDEF;
+        RETPUSHUNDEF;
     if (MAXARG > 1) {
-       discp = POPs;
+        discp = POPs;
     }
 
     gv = MUTABLE_GV(POPs);
     io = GvIO(gv);
 
     if (io) {
-       const MAGIC * const mg = SvTIED_mg((const SV *)io, PERL_MAGIC_tiedscalar);
-       if (mg) {
-           /* This takes advantage of the implementation of the varargs
-              function, which I don't think that the optimiser will be able to
-              figure out. Although, as it's a static function, in theory it
-              could.  */
-           return Perl_tied_method(aTHX_ SV_CONST(BINMODE), SP, MUTABLE_SV(io), mg,
-                                   G_SCALAR|TIED_METHOD_MORTALIZE_NOT_NEEDED,
-                                   discp ? 1 : 0, discp);
-       }
+        const MAGIC * const mg = SvTIED_mg((const SV *)io, PERL_MAGIC_tiedscalar);
+        if (mg) {
+            /* This takes advantage of the implementation of the varargs
+               function, which I don't think that the optimiser will be able to
+               figure out. Although, as it's a static function, in theory it
+               could.  */
+            return Perl_tied_method(aTHX_ SV_CONST(BINMODE), SP, MUTABLE_SV(io), mg,
+                                    G_SCALAR|TIED_METHOD_MORTALIZE_NOT_NEEDED,
+                                    discp ? 1 : 0, discp);
+        }
     }
 
     if (!io || !(fp = IoIFP(io))) {
-       report_evil_fh(gv);
-       SETERRNO(EBADF,RMS_IFI);
+        report_evil_fh(gv);
+        SETERRNO(EBADF,RMS_IFI);
         RETPUSHUNDEF;
     }
 
     PUTBACK;
     {
-       STRLEN len = 0;
-       const char *d = NULL;
-       int mode;
-       if (discp)
-           d = SvPV_const(discp, len);
-       mode = mode_from_discipline(d, len);
-       if (PerlIO_binmode(aTHX_ fp, IoTYPE(io), mode, d)) {
-           if (IoOFP(io) && IoOFP(io) != IoIFP(io)) {
-               if (!PerlIO_binmode(aTHX_ IoOFP(io), IoTYPE(io), mode, d)) {
-                   SPAGAIN;
-                   RETPUSHUNDEF;
-               }
-           }
-           SPAGAIN;
-           RETPUSHYES;
-       }
-       else {
-           SPAGAIN;
-           RETPUSHUNDEF;
-       }
+        STRLEN len = 0;
+        const char *d = NULL;
+        int mode;
+        if (discp)
+            d = SvPV_const(discp, len);
+        mode = mode_from_discipline(d, len);
+        if (PerlIO_binmode(aTHX_ fp, IoTYPE(io), mode, d)) {
+            if (IoOFP(io) && IoOFP(io) != IoIFP(io)) {
+                if (!PerlIO_binmode(aTHX_ IoOFP(io), IoTYPE(io), mode, d)) {
+                    SPAGAIN;
+                    RETPUSHUNDEF;
+                }
+            }
+            SPAGAIN;
+            RETPUSHYES;
+        }
+        else {
+            SPAGAIN;
+            RETPUSHUNDEF;
+        }
     }
 }
 
@@ -891,93 +872,120 @@ PP(pp_tie)
     SV *varsv = *++MARK;
 
     switch(SvTYPE(varsv)) {
-       case SVt_PVHV:
-       {
-           HE *entry;
-           methname = "TIEHASH";
-           if (HvLAZYDEL(varsv) && (entry = HvEITER((HV *)varsv))) {
-               HvLAZYDEL_off(varsv);
-               hv_free_ent((HV *)varsv, entry);
-           }
-           HvEITER_set(MUTABLE_HV(varsv), 0);
-           break;
-       }
-       case SVt_PVAV:
-           methname = "TIEARRAY";
-           if (!AvREAL(varsv)) {
-               if (!AvREIFY(varsv))
-                   Perl_croak(aTHX_ "Cannot tie unreifiable array");
-               av_clear((AV *)varsv);
-               AvREIFY_off(varsv);
-               AvREAL_on(varsv);
-           }
-           break;
-       case SVt_PVGV:
-       case SVt_PVLV:
-           if (isGV_with_GP(varsv) && !SvFAKE(varsv)) {
-               methname = "TIEHANDLE";
-               how = PERL_MAGIC_tiedscalar;
-               /* For tied filehandles, we apply tiedscalar magic to the IO
-                  slot of the GP rather than the GV itself. AMS 20010812 */
-               if (!GvIOp(varsv))
-                   GvIOp(varsv) = newIO();
-               varsv = MUTABLE_SV(GvIOp(varsv));
-               break;
-           }
-           if (SvTYPE(varsv) == SVt_PVLV && LvTYPE(varsv) == 'y') {
-               vivify_defelem(varsv);
-               varsv = LvTARG(varsv);
-           }
-           /* FALLTHROUGH */
-       default:
-           methname = "TIESCALAR";
-           how = PERL_MAGIC_tiedscalar;
-           break;
+        case SVt_PVHV:
+        {
+            HE *entry;
+            methname = "TIEHASH";
+            if (HvLAZYDEL(varsv) && (entry = HvEITER_get((HV *)varsv))) {
+                HvLAZYDEL_off(varsv);
+                hv_free_ent((HV *)varsv, entry);
+            }
+            HvEITER_set(MUTABLE_HV(varsv), 0);
+            HvRITER_set(MUTABLE_HV(varsv), -1);
+            break;
+        }
+        case SVt_PVAV:
+            methname = "TIEARRAY";
+            if (!AvREAL(varsv)) {
+                if (!AvREIFY(varsv))
+                    Perl_croak(aTHX_ "Cannot tie unreifiable array");
+                av_clear((AV *)varsv);
+                AvREIFY_off(varsv);
+                AvREAL_on(varsv);
+            }
+            break;
+        case SVt_PVGV:
+        case SVt_PVLV:
+            if (isGV_with_GP(varsv) && !SvFAKE(varsv)) {
+                methname = "TIEHANDLE";
+                how = PERL_MAGIC_tiedscalar;
+                /* For tied filehandles, we apply tiedscalar magic to the IO
+                   slot of the GP rather than the GV itself. AMS 20010812 */
+                if (!GvIOp(varsv))
+                    GvIOp(varsv) = newIO();
+                varsv = MUTABLE_SV(GvIOp(varsv));
+                break;
+            }
+            if (SvTYPE(varsv) == SVt_PVLV && LvTYPE(varsv) == 'y') {
+                vivify_defelem(varsv);
+                varsv = LvTARG(varsv);
+            }
+            /* FALLTHROUGH */
+        default:
+            methname = "TIESCALAR";
+            how = PERL_MAGIC_tiedscalar;
+            break;
     }
     items = SP - MARK++;
     if (sv_isobject(*MARK)) { /* Calls GET magic. */
-       ENTER_with_name("call_TIE");
-       PUSHSTACKi(PERLSI_MAGIC);
-       PUSHMARK(SP);
-       EXTEND(SP,(I32)items);
-       while (items--)
-           PUSHs(*MARK++);
-       PUTBACK;
-       call_method(methname, G_SCALAR);
+        ENTER_with_name("call_TIE");
+        PUSHSTACKi(PERLSI_MAGIC);
+        PUSHMARK(SP);
+        EXTEND(SP,(I32)items);
+        while (items--)
+            PUSHs(*MARK++);
+        PUTBACK;
+        call_method(methname, G_SCALAR);
     }
     else {
-       /* Can't use call_method here, else this: fileno FOO; tie @a, "FOO"
-        * will attempt to invoke IO::File::TIEARRAY, with (best case) the
-        * wrong error message, and worse case, supreme action at a distance.
-        * (Sorry obfuscation writers. You're not going to be given this one.)
-        */
+        /* Can't use call_method here, else this: fileno FOO; tie @a, "FOO"
+         * will attempt to invoke IO::File::TIEARRAY, with (best case) the
+         * wrong error message, and worse case, supreme action at a distance.
+         * (Sorry obfuscation writers. You're not going to be given this one.)
+         */
        stash = gv_stashsv(*MARK, 0);
-       if (!stash || !(gv = gv_fetchmethod(stash, methname))) {
-           DIE(aTHX_ "Can't locate object method \"%s\" via package \"%"SVf"\"",
-                methname, SVfARG(SvOK(*MARK) ? *MARK : &PL_sv_no));
-       }
-       ENTER_with_name("call_TIE");
-       PUSHSTACKi(PERLSI_MAGIC);
-       PUSHMARK(SP);
-       EXTEND(SP,(I32)items);
-       while (items--)
-           PUSHs(*MARK++);
-       PUTBACK;
-       call_sv(MUTABLE_SV(GvCV(gv)), G_SCALAR);
+       if (!stash) {
+           if (SvROK(*MARK))
+               DIE(aTHX_ "Can't locate object method \"%s\" via package \"%" SVf "\"",
+                   methname, SVfARG(*MARK));
+           else if (isGV(*MARK)) {
+               /* If the glob doesn't name an existing package, using
+                * SVfARG(*MARK) would yield "*Foo::Bar" or *main::Foo. So
+                * generate the name for the error message explicitly. */
+               SV *stashname = sv_2mortal(newSV(0));
+               gv_fullname4(stashname, (GV *) *MARK, NULL, FALSE);
+               DIE(aTHX_ "Can't locate object method \"%s\" via package \"%" SVf "\"",
+                   methname, SVfARG(stashname));
+           }
+           else {
+               SV *stashname = !SvPOK(*MARK) ? &PL_sv_no
+                             : SvCUR(*MARK)  ? *MARK
+                             :                 sv_2mortal(newSVpvs("main"));
+               DIE(aTHX_ "Can't locate object method \"%s\" via package \"%" SVf "\""
+                   " (perhaps you forgot to load \"%" SVf "\"?)",
+                   methname, SVfARG(stashname), SVfARG(stashname));
+           }
+       }
+       else if (!(gv = gv_fetchmethod(stash, methname))) {
+           /* The effective name can only be NULL for stashes that have
+            * been deleted from the symbol table, which this one can't
+            * be, since we just looked it up by name.
+            */
+           DIE(aTHX_ "Can't locate object method \"%s\" via package \"%" HEKf "\"",
+               methname, HvENAME_HEK_NN(stash));
+       }
+        ENTER_with_name("call_TIE");
+        PUSHSTACKi(PERLSI_MAGIC);
+        PUSHMARK(SP);
+        EXTEND(SP,(I32)items);
+        while (items--)
+            PUSHs(*MARK++);
+        PUTBACK;
+        call_sv(MUTABLE_SV(GvCV(gv)), G_SCALAR);
     }
     SPAGAIN;
 
     sv = TOPs;
     POPSTACK;
     if (sv_isobject(sv)) {
-       sv_unmagic(varsv, how);
-       /* Croak if a self-tie on an aggregate is attempted. */
-       if (varsv == SvRV(sv) &&
-           (SvTYPE(varsv) == SVt_PVAV ||
-            SvTYPE(varsv) == SVt_PVHV))
-           Perl_croak(aTHX_
-                      "Self-ties of arrays and hashes are not supported");
-       sv_magic(varsv, (SvRV(sv) == varsv ? NULL : sv), how, NULL, 0);
+        sv_unmagic(varsv, how);
+        /* Croak if a self-tie on an aggregate is attempted. */
+        if (varsv == SvRV(sv) &&
+            (SvTYPE(varsv) == SVt_PVAV ||
+             SvTYPE(varsv) == SVt_PVHV))
+            Perl_croak(aTHX_
+                       "Self-ties of arrays and hashes are not supported");
+        sv_magic(varsv, (SvRV(sv) == varsv ? NULL : sv), how, NULL, 0);
     }
     LEAVE_with_name("call_TIE");
     SP = PL_stack_base + markoff;
@@ -994,37 +1002,49 @@ PP(pp_untie)
     MAGIC *mg;
     SV *sv = POPs;
     const char how = (SvTYPE(sv) == SVt_PVHV || SvTYPE(sv) == SVt_PVAV)
-               ? PERL_MAGIC_tied : PERL_MAGIC_tiedscalar;
+                ? PERL_MAGIC_tied : PERL_MAGIC_tiedscalar;
 
     if (isGV_with_GP(sv) && !SvFAKE(sv) && !(sv = MUTABLE_SV(GvIOp(sv))))
-       RETPUSHYES;
+        RETPUSHYES;
 
     if (SvTYPE(sv) == SVt_PVLV && LvTYPE(sv) == 'y' &&
-       !(sv = defelem_target(sv, NULL))) RETPUSHUNDEF;
+        !(sv = defelem_target(sv, NULL))) RETPUSHUNDEF;
 
     if ((mg = SvTIED_mg(sv, how))) {
-       SV * const obj = SvRV(SvTIED_obj(sv, mg));
-        if (obj) {
-           GV * const gv = gv_fetchmethod_autoload(SvSTASH(obj), "UNTIE", FALSE);
-           CV *cv;
-           if (gv && isGV(gv) && (cv = GvCV(gv))) {
-              PUSHMARK(SP);
-              PUSHs(SvTIED_obj(MUTABLE_SV(gv), mg));
-              mXPUSHi(SvREFCNT(obj) - 1);
-              PUTBACK;
-              ENTER_with_name("call_UNTIE");
-              call_sv(MUTABLE_SV(cv), G_VOID);
-              LEAVE_with_name("call_UNTIE");
-              SPAGAIN;
+        SV * const obj = SvRV(SvTIED_obj(sv, mg));
+        if (obj && SvSTASH(obj)) {
+            GV * const gv = gv_fetchmethod_autoload(SvSTASH(obj), "UNTIE", FALSE);
+            CV *cv;
+            if (gv && isGV(gv) && (cv = GvCV(gv))) {
+               PUSHMARK(SP);
+               PUSHs(SvTIED_obj(MUTABLE_SV(gv), mg));
+               mXPUSHi(SvREFCNT(obj) - 1);
+               PUTBACK;
+               ENTER_with_name("call_UNTIE");
+               call_sv(MUTABLE_SV(cv), G_VOID);
+               LEAVE_with_name("call_UNTIE");
+               SPAGAIN;
+            }
+            else if (mg && SvREFCNT(obj) > 1) {
+                Perl_ck_warner(aTHX_ packWARN(WARN_UNTIE),
+                               "untie attempted while %" UVuf " inner references still exist",
+                               (UV)SvREFCNT(obj) - 1 ) ;
             }
-           else if (mg && SvREFCNT(obj) > 1) {
-               Perl_ck_warner(aTHX_ packWARN(WARN_UNTIE),
-                              "untie attempted while %"UVuf" inner references still exist",
-                              (UV)SvREFCNT(obj) - 1 ) ;
-           }
         }
     }
     sv_unmagic(sv, how) ;
+
+    if (SvTYPE(sv) == SVt_PVHV) {
+        /* If the tied hash was partway through iteration, free the iterator and
+         * any key that it is pointing to. */
+        HE *entry;
+        if (HvLAZYDEL(sv) && (entry = HvEITER_get((HV *)sv))) {
+            HvLAZYDEL_off(sv);
+            hv_free_ent((HV *)sv, entry);
+            HvEITER_set(MUTABLE_HV(sv), 0);
+        }
+    }
+
     RETPUSHYES;
 }
 
@@ -1034,17 +1054,17 @@ PP(pp_tied)
     const MAGIC *mg;
     dTOPss;
     const char how = (SvTYPE(sv) == SVt_PVHV || SvTYPE(sv) == SVt_PVAV)
-               ? PERL_MAGIC_tied : PERL_MAGIC_tiedscalar;
+                ? PERL_MAGIC_tied : PERL_MAGIC_tiedscalar;
 
     if (isGV_with_GP(sv) && !SvFAKE(sv) && !(sv = MUTABLE_SV(GvIOp(sv))))
-       goto ret_undef;
+        goto ret_undef;
 
     if (SvTYPE(sv) == SVt_PVLV && LvTYPE(sv) == 'y' &&
-       !(sv = defelem_target(sv, NULL))) goto ret_undef;
+        !(sv = defelem_target(sv, NULL))) goto ret_undef;
 
     if ((mg = SvTIED_mg(sv, how))) {
-       SETs(SvTIED_obj(sv, mg));
-       return NORMAL; /* PUTBACK not needed, pp_tied never moves SP */
+        SETs(SvTIED_obj(sv, mg));
+        return NORMAL; /* PUTBACK not needed, pp_tied never moves SP */
     }
     ret_undef:
     SETs(&PL_sv_undef);
@@ -1062,11 +1082,11 @@ PP(pp_dbmopen)
     SV * const sv = newSVpvs_flags("AnyDBM_File", SVs_TEMP);
     stash = gv_stashsv(sv, 0);
     if (!stash || !(gv = gv_fetchmethod(stash, "TIEHASH"))) {
-       PUTBACK;
-       require_pv("AnyDBM_File.pm");
-       SPAGAIN;
-       if (!stash || !(gv = gv_fetchmethod(stash, "TIEHASH")))
-           DIE(aTHX_ "No dbm on this machine");
+        PUTBACK;
+        require_pv("AnyDBM_File.pm");
+        SPAGAIN;
+        if (!stash || !(gv = gv_fetchmethod(stash, "TIEHASH")))
+            DIE(aTHX_ "No dbm on this machine");
     }
 
     ENTER;
@@ -1076,11 +1096,11 @@ PP(pp_dbmopen)
     PUSHs(sv);
     PUSHs(left);
     if (SvIV(right))
-       mPUSHu(O_RDWR|O_CREAT);
+        mPUSHu(O_RDWR|O_CREAT);
     else
     {
-       mPUSHu(O_RDWR);
-       if (!SvOK(right)) right = &PL_sv_no;
+        mPUSHu(O_RDWR);
+        if (!SvOK(right)) right = &PL_sv_no;
     }
     PUSHs(right);
     PUTBACK;
@@ -1088,22 +1108,22 @@ PP(pp_dbmopen)
     SPAGAIN;
 
     if (!sv_isobject(TOPs)) {
-       SP--;
-       PUSHMARK(SP);
-       PUSHs(sv);
-       PUSHs(left);
-       mPUSHu(O_RDONLY);
-       PUSHs(right);
-       PUTBACK;
-       call_sv(MUTABLE_SV(GvCV(gv)), G_SCALAR);
-       SPAGAIN;
+        SP--;
+        PUSHMARK(SP);
+        PUSHs(sv);
+        PUSHs(left);
+        mPUSHu(O_RDONLY);
+        PUSHs(right);
+        PUTBACK;
+        call_sv(MUTABLE_SV(GvCV(gv)), G_SCALAR);
+        SPAGAIN;
         if (sv_isobject(TOPs))
             goto retie;
     }
     else {
         retie:
-       sv_unmagic(MUTABLE_SV(hv), PERL_MAGIC_tied);
-       sv_magic(MUTABLE_SV(hv), TOPs, PERL_MAGIC_tied, NULL, 0);
+        sv_unmagic(MUTABLE_SV(hv), PERL_MAGIC_tied);
+        sv_magic(MUTABLE_SV(hv), TOPs, PERL_MAGIC_tied, NULL, 0);
     }
     LEAVE;
     RETURN;
@@ -1124,10 +1144,11 @@ PP(pp_sselect)
     struct timeval *tbuf = &timebuf;
     I32 growsize;
     char *fd_sets[4];
+    SV *svs[4];
 #if BYTEORDER != 0x1234 && BYTEORDER != 0x12345678
-       I32 masksize;
-       I32 offset;
-       I32 k;
+        I32 masksize;
+        I32 offset;
+        I32 k;
 
 #   if BYTEORDER & 0xf0000
 #      define ORDERBYTE (0x88888888 - BYTEORDER)
@@ -1139,24 +1160,29 @@ PP(pp_sselect)
 
     SP -= 4;
     for (i = 1; i <= 3; i++) {
-       SV * const sv = SP[i];
-       SvGETMAGIC(sv);
-       if (!SvOK(sv))
-           continue;
-       if (SvREADONLY(sv)) {
-           if (!(SvPOK(sv) && SvCUR(sv) == 0))
-               Perl_croak_no_modify();
-       }
-       else if (SvIsCOW(sv)) sv_force_normal_flags(sv, 0);
-       if (!SvPOK(sv)) {
-           if (!SvPOKp(sv))
-               Perl_ck_warner(aTHX_ packWARN(WARN_MISC),
-                                   "Non-string passed as bitmask");
-           SvPV_force_nomg_nolen(sv);  /* force string conversion */
-       }
-       j = SvCUR(sv);
-       if (maxlen < j)
-           maxlen = j;
+        SV * const sv = svs[i] = SP[i];
+        SvGETMAGIC(sv);
+        if (!SvOK(sv))
+            continue;
+        if (SvREADONLY(sv)) {
+            if (!(SvPOK(sv) && SvCUR(sv) == 0))
+                Perl_croak_no_modify();
+        }
+        else if (SvIsCOW(sv)) sv_force_normal_flags(sv, 0);
+        if (!SvPOK(sv)) {
+            if (!SvPOKp(sv))
+                Perl_ck_warner(aTHX_ packWARN(WARN_MISC),
+                                    "Non-string passed as bitmask");
+            if (SvGAMAGIC(sv)) {
+                svs[i] = sv_newmortal();
+                sv_copypv_nomg(svs[i], sv);
+            }
+            else
+                SvPV_force_nomg_nolen(sv); /* force string conversion */
+        }
+        j = SvCUR(svs[i]);
+        if (maxlen < j)
+            maxlen = j;
     }
 
 /* little endians can use vecs directly */
@@ -1192,42 +1218,42 @@ PP(pp_sselect)
     sv = SP[4];
     SvGETMAGIC(sv);
     if (SvOK(sv)) {
-       value = SvNV_nomg(sv);
-       if (value < 0.0)
-           value = 0.0;
-       timebuf.tv_sec = (long)value;
-       value -= (NV)timebuf.tv_sec;
-       timebuf.tv_usec = (long)(value * 1000000.0);
+        value = SvNV_nomg(sv);
+        if (value < 0.0)
+            value = 0.0;
+        timebuf.tv_sec = (long)value;
+        value -= (NV)timebuf.tv_sec;
+        timebuf.tv_usec = (long)(value * 1000000.0);
     }
     else
-       tbuf = NULL;
+        tbuf = NULL;
 
     for (i = 1; i <= 3; i++) {
-       sv = SP[i];
-       if (!SvOK(sv) || SvCUR(sv) == 0) {
-           fd_sets[i] = 0;
-           continue;
-       }
-       assert(SvPOK(sv));
-       j = SvLEN(sv);
-       if (j < growsize) {
-           Sv_Grow(sv, growsize);
-       }
-       j = SvCUR(sv);
-       s = SvPVX(sv) + j;
-       while (++j <= growsize) {
-           *s++ = '\0';
-       }
+        sv = svs[i];
+        if (!SvOK(sv) || SvCUR(sv) == 0) {
+            fd_sets[i] = 0;
+            continue;
+        }
+        assert(SvPOK(sv));
+        j = SvLEN(sv);
+        if (j < growsize) {
+            Sv_Grow(sv, growsize);
+        }
+        j = SvCUR(sv);
+        s = SvPVX(sv) + j;
+        while (++j <= growsize) {
+            *s++ = '\0';
+        }
 
 #if BYTEORDER != 0x1234 && BYTEORDER != 0x12345678
-       s = SvPVX(sv);
-       Newx(fd_sets[i], growsize, char);
-       for (offset = 0; offset < growsize; offset += masksize) {
-           for (j = 0, k=ORDERBYTE; j < masksize; j++, (k >>= 4))
-               fd_sets[i][j+offset] = s[(k % masksize) + offset];
-       }
+        s = SvPVX(sv);
+        Newx(fd_sets[i], growsize, char);
+        for (offset = 0; offset < growsize; offset += masksize) {
+            for (j = 0, k=ORDERBYTE; j < masksize; j++, (k >>= 4))
+                fd_sets[i][j+offset] = s[(k % masksize) + offset];
+        }
 #else
-       fd_sets[i] = SvPVX(sv);
+        fd_sets[i] = SvPVX(sv);
 #endif
     }
 
@@ -1235,39 +1261,42 @@ PP(pp_sselect)
     /* Can't make just the (void*) conditional because that would be
      * cpp #if within cpp macro, and not all compilers like that. */
     nfound = PerlSock_select(
-       maxlen * 8,
-       (Select_fd_set_t) fd_sets[1],
-       (Select_fd_set_t) fd_sets[2],
-       (Select_fd_set_t) fd_sets[3],
-       (void*) tbuf); /* Workaround for compiler bug. */
+        maxlen * 8,
+        (Select_fd_set_t) fd_sets[1],
+        (Select_fd_set_t) fd_sets[2],
+        (Select_fd_set_t) fd_sets[3],
+        (void*) tbuf); /* Workaround for compiler bug. */
 #else
     nfound = PerlSock_select(
-       maxlen * 8,
-       (Select_fd_set_t) fd_sets[1],
-       (Select_fd_set_t) fd_sets[2],
-       (Select_fd_set_t) fd_sets[3],
-       tbuf);
+        maxlen * 8,
+        (Select_fd_set_t) fd_sets[1],
+        (Select_fd_set_t) fd_sets[2],
+        (Select_fd_set_t) fd_sets[3],
+        tbuf);
 #endif
     for (i = 1; i <= 3; i++) {
-       if (fd_sets[i]) {
-           sv = SP[i];
+        if (fd_sets[i]) {
+            sv = svs[i];
 #if BYTEORDER != 0x1234 && BYTEORDER != 0x12345678
-           s = SvPVX(sv);
-           for (offset = 0; offset < growsize; offset += masksize) {
-               for (j = 0, k=ORDERBYTE; j < masksize; j++, (k >>= 4))
-                   s[(k % masksize) + offset] = fd_sets[i][j+offset];
-           }
-           Safefree(fd_sets[i]);
+            s = SvPVX(sv);
+            for (offset = 0; offset < growsize; offset += masksize) {
+                for (j = 0, k=ORDERBYTE; j < masksize; j++, (k >>= 4))
+                    s[(k % masksize) + offset] = fd_sets[i][j+offset];
+            }
+            Safefree(fd_sets[i]);
 #endif
-           SvSETMAGIC(sv);
-       }
+            if (sv != SP[i])
+                SvSetMagicSV(SP[i], sv);
+            else
+                SvSETMAGIC(sv);
+        }
     }
 
     PUSHi(nfound);
-    if (GIMME_V == G_ARRAY && tbuf) {
-       value = (NV)(timebuf.tv_sec) +
-               (NV)(timebuf.tv_usec) / 1000000.0;
-       mPUSHn(value);
+    if (GIMME_V == G_LIST && tbuf) {
+        value = (NV)(timebuf.tv_sec) +
+                (NV)(timebuf.tv_usec) / 1000000.0;
+        mPUSHn(value);
     }
     RETURN;
 #else
@@ -1277,7 +1306,7 @@ PP(pp_sselect)
 
 /*
 
-=head1 GV Functions
+=for apidoc_section $GV
 
 =for apidoc setdefout
 
@@ -1310,23 +1339,23 @@ PP(pp_select)
     GV * const *gvp;
 
     if (!egv)
-       egv = PL_defoutgv;
+        egv = PL_defoutgv;
     hv = isGV_with_GP(egv) ? GvSTASH(egv) : NULL;
     gvp = hv && HvENAME(hv)
-               ? (GV**)hv_fetch(hv, GvNAME(egv), HEK_UTF8(GvNAME_HEK(egv)) ? -GvNAMELEN(egv) : GvNAMELEN(egv), FALSE)
-               : NULL;
+                ? (GV**)hv_fetch(hv, GvNAME(egv), HEK_UTF8(GvNAME_HEK(egv)) ? -GvNAMELEN(egv) : GvNAMELEN(egv), FALSE)
+                : NULL;
     if (gvp && *gvp == egv) {
-           gv_efullname4(TARG, PL_defoutgv, NULL, TRUE);
-           XPUSHTARG;
+            gv_efullname4(TARG, PL_defoutgv, NULL, TRUE);
+            XPUSHTARG;
     }
     else {
-           mXPUSHs(newRV(MUTABLE_SV(egv)));
+            mXPUSHs(newRV(MUTABLE_SV(egv)));
     }
 
     if (newdefout) {
-       if (!GvIO(newdefout))
-           gv_IOadd(newdefout);
-       setdefout(newdefout);
+        if (!GvIO(newdefout))
+            gv_IOadd(newdefout);
+        setdefout(newdefout);
     }
 
     RETURN;
@@ -1335,43 +1364,45 @@ PP(pp_select)
 PP(pp_getc)
 {
     dSP; dTARGET;
+    /* pp_coreargs pushes a NULL to indicate no args passed to
+     * CORE::getc() */
     GV * const gv =
-       MAXARG==0 || (!TOPs && !POPs) ? PL_stdingv : MUTABLE_GV(POPs);
+        MAXARG==0 || (!TOPs && !POPs) ? PL_stdingv : MUTABLE_GV(POPs);
     IO *const io = GvIO(gv);
 
     if (MAXARG == 0)
-       EXTEND(SP, 1);
+        EXTEND(SP, 1);
 
     if (io) {
-       const MAGIC * const mg = SvTIED_mg((const SV *)io, PERL_MAGIC_tiedscalar);
-       if (mg) {
-           const U8 gimme = GIMME_V;
-           Perl_tied_method(aTHX_ SV_CONST(GETC), SP, MUTABLE_SV(io), mg, gimme, 0);
-           if (gimme == G_SCALAR) {
-               SPAGAIN;
-               SvSetMagicSV_nosteal(TARG, TOPs);
-           }
-           return NORMAL;
-       }
+        const MAGIC * const mg = SvTIED_mg((const SV *)io, PERL_MAGIC_tiedscalar);
+        if (mg) {
+            const U8 gimme = GIMME_V;
+            Perl_tied_method(aTHX_ SV_CONST(GETC), SP, MUTABLE_SV(io), mg, gimme, 0);
+            if (gimme == G_SCALAR) {
+                SPAGAIN;
+                SvSetMagicSV_nosteal(TARG, TOPs);
+            }
+            return NORMAL;
+        }
     }
     if (!gv || do_eof(gv)) { /* make sure we have fp with something */
-       if (!io || (!IoIFP(io) && IoTYPE(io) != IoTYPE_WRONLY))
-           report_evil_fh(gv);
-       SETERRNO(EBADF,RMS_IFI);
-       RETPUSHUNDEF;
+        if (!io || (!IoIFP(io) && IoTYPE(io) != IoTYPE_WRONLY))
+            report_evil_fh(gv);
+        SETERRNO(EBADF,RMS_IFI);
+        RETPUSHUNDEF;
     }
     TAINT;
     sv_setpvs(TARG, " ");
     *SvPVX(TARG) = PerlIO_getc(IoIFP(GvIOp(gv))); /* should never be EOF */
     if (PerlIO_isutf8(IoIFP(GvIOp(gv)))) {
-       /* Find out how many bytes the char needs */
-       Size_t len = UTF8SKIP(SvPVX_const(TARG));
-       if (len > 1) {
-           SvGROW(TARG,len+1);
-           len = PerlIO_read(IoIFP(GvIOp(gv)),SvPVX(TARG)+1,len-1);
-           SvCUR_set(TARG,1+len);
-       }
-       SvUTF8_on(TARG);
+        /* Find out how many bytes the char needs */
+        Size_t len = UTF8SKIP(SvPVX_const(TARG));
+        if (len > 1) {
+            SvGROW(TARG,len+1);
+            len = PerlIO_read(IoIFP(GvIOp(gv)),SvPVX(TARG)+1,len-1);
+            SvCUR_set(TARG,1+len);
+        }
+        SvUTF8_on(TARG);
     }
     else SvUTF8_off(TARG);
     PUSHTARG;
@@ -1387,12 +1418,12 @@ S_doform(pTHX_ CV *cv, GV *gv, OP *retop)
     PERL_ARGS_ASSERT_DOFORM;
 
     if (CvCLONE(cv))
-       cv = MUTABLE_CV(sv_2mortal(MUTABLE_SV(cv_clone(cv))));
+        cv = MUTABLE_CV(sv_2mortal(MUTABLE_SV(cv_clone(cv))));
 
     cx = cx_pushblock(CXt_FORMAT, gimme, PL_stack_sp, PL_savestack_ix);
     cx_pushformat(cx, cv, retop, gv);
     if (CvDEPTH(cv) >= 2)
-       pad_push(CvPADLIST(cv), CvDEPTH(cv));
+        pad_push(CvPADLIST(cv), CvDEPTH(cv));
     PAD_SET_CUR_NOSAVE(CvPADLIST(cv), CvDEPTH(cv));
 
     setdefout(gv);         /* locally select filehandle so $% et al work */
@@ -1406,33 +1437,32 @@ PP(pp_enterwrite)
     IO *io;
     GV *fgv;
     CV *cv = NULL;
-    SV *tmpsv = NULL;
 
     if (MAXARG == 0) {
-       EXTEND(SP, 1);
-       gv = PL_defoutgv;
+        EXTEND(SP, 1);
+        gv = PL_defoutgv;
     }
     else {
-       gv = MUTABLE_GV(POPs);
-       if (!gv)
-           gv = PL_defoutgv;
+        gv = MUTABLE_GV(POPs);
+        if (!gv)
+            gv = PL_defoutgv;
     }
     io = GvIO(gv);
     if (!io) {
-       RETPUSHNO;
+        RETPUSHNO;
     }
     if (IoFMT_GV(io))
-       fgv = IoFMT_GV(io);
+        fgv = IoFMT_GV(io);
     else
-       fgv = gv;
+        fgv = gv;
 
     assert(fgv);
 
     cv = GvFORM(fgv);
     if (!cv) {
-       tmpsv = sv_newmortal();
-       gv_efullname4(tmpsv, fgv, NULL, FALSE);
-       DIE(aTHX_ "Undefined format \"%"SVf"\" called", SVfARG(tmpsv));
+        SV * const tmpsv = sv_newmortal();
+        gv_efullname4(tmpsv, fgv, NULL, FALSE);
+        DIE(aTHX_ "Undefined format \"%" SVf "\" called", SVfARG(tmpsv));
     }
     IoFLAGS(io) &= ~IOf_DIDTOP;
     RETURNOP(doform(cv,gv,PL_op->op_next));
@@ -1453,71 +1483,72 @@ PP(pp_leavewrite)
         goto forget_top;
 
     DEBUG_f(PerlIO_printf(Perl_debug_log, "left=%ld, todo=%ld\n",
-         (long)IoLINES_LEFT(io), (long)FmLINES(PL_formtarget)));
+          (long)IoLINES_LEFT(io), (long)FmLINES(PL_formtarget)));
 
     if (IoLINES_LEFT(io) < FmLINES(PL_formtarget) &&
-       PL_formtarget != PL_toptarget)
+        PL_formtarget != PL_toptarget)
     {
-       GV *fgv;
-       CV *cv;
-       if (!IoTOP_GV(io)) {
-           GV *topgv;
-
-           if (!IoTOP_NAME(io)) {
-               SV *topname;
-               if (!IoFMT_NAME(io))
-                   IoFMT_NAME(io) = savepv(GvNAME(gv));
-               topname = sv_2mortal(Perl_newSVpvf(aTHX_ "%"HEKf"_TOP",
+        GV *fgv;
+        CV *cv;
+        if (!IoTOP_GV(io)) {
+            GV *topgv;
+
+            if (!IoTOP_NAME(io)) {
+                SV *topname;
+                if (!IoFMT_NAME(io))
+                    IoFMT_NAME(io) = savepv(GvNAME(gv));
+                topname = sv_2mortal(Perl_newSVpvf(aTHX_ "%" HEKf "_TOP",
                                         HEKfARG(GvNAME_HEK(gv))));
-               topgv = gv_fetchsv(topname, 0, SVt_PVFM);
-               if ((topgv && GvFORM(topgv)) ||
-                 !gv_fetchpvs("top", GV_NOTQUAL, SVt_PVFM))
-                   IoTOP_NAME(io) = savesvpv(topname);
-               else
-                   IoTOP_NAME(io) = savepvs("top");
-           }
-           topgv = gv_fetchpv(IoTOP_NAME(io), 0, SVt_PVFM);
-           if (!topgv || !GvFORM(topgv)) {
-               IoLINES_LEFT(io) = IoPAGE_LEN(io);
-               goto forget_top;
-           }
-           IoTOP_GV(io) = topgv;
-       }
-       if (IoFLAGS(io) & IOf_DIDTOP) { /* Oh dear.  It still doesn't fit. */
-           I32 lines = IoLINES_LEFT(io);
-           const char *s = SvPVX_const(PL_formtarget);
-           if (lines <= 0)             /* Yow, header didn't even fit!!! */
-               goto forget_top;
-           while (lines-- > 0) {
-               s = strchr(s, '\n');
-               if (!s)
-                   break;
-               s++;
-           }
-           if (s) {
-               const STRLEN save = SvCUR(PL_formtarget);
-               SvCUR_set(PL_formtarget, s - SvPVX_const(PL_formtarget));
-               do_print(PL_formtarget, ofp);
-               SvCUR_set(PL_formtarget, save);
-               sv_chop(PL_formtarget, s);
-               FmLINES(PL_formtarget) -= IoLINES_LEFT(io);
-           }
-       }
-       if (IoLINES_LEFT(io) >= 0 && IoPAGE(io) > 0)
-           do_print(GvSV(gv_fetchpvs("\f", GV_ADD, SVt_PV)), ofp);
-       IoLINES_LEFT(io) = IoPAGE_LEN(io);
-       IoPAGE(io)++;
-       PL_formtarget = PL_toptarget;
-       IoFLAGS(io) |= IOf_DIDTOP;
-       fgv = IoTOP_GV(io);
-       assert(fgv); /* IoTOP_GV(io) should have been set above */
-       cv = GvFORM(fgv);
-       if (!cv) {
-           SV * const sv = sv_newmortal();
-           gv_efullname4(sv, fgv, NULL, FALSE);
-           DIE(aTHX_ "Undefined top format \"%"SVf"\" called", SVfARG(sv));
-       }
-       return doform(cv, gv, PL_op);
+                topgv = gv_fetchsv(topname, 0, SVt_PVFM);
+                if ((topgv && GvFORM(topgv)) ||
+                  !gv_fetchpvs("top", GV_NOTQUAL, SVt_PVFM))
+                    IoTOP_NAME(io) = savesvpv(topname);
+                else
+                    IoTOP_NAME(io) = savepvs("top");
+            }
+            topgv = gv_fetchpv(IoTOP_NAME(io), 0, SVt_PVFM);
+            if (!topgv || !GvFORM(topgv)) {
+                IoLINES_LEFT(io) = IoPAGE_LEN(io);
+                goto forget_top;
+            }
+            IoTOP_GV(io) = topgv;
+        }
+        if (IoFLAGS(io) & IOf_DIDTOP) {        /* Oh dear.  It still doesn't fit. */
+            I32 lines = IoLINES_LEFT(io);
+            const char *s = SvPVX_const(PL_formtarget);
+            const char *e = SvEND(PL_formtarget);
+            if (lines <= 0)            /* Yow, header didn't even fit!!! */
+                goto forget_top;
+            while (lines-- > 0) {
+                s = (char *) memchr(s, '\n', e - s);
+                if (!s)
+                    break;
+                s++;
+            }
+            if (s) {
+                const STRLEN save = SvCUR(PL_formtarget);
+                SvCUR_set(PL_formtarget, s - SvPVX_const(PL_formtarget));
+                do_print(PL_formtarget, ofp);
+                SvCUR_set(PL_formtarget, save);
+                sv_chop(PL_formtarget, s);
+                FmLINES(PL_formtarget) -= IoLINES_LEFT(io);
+            }
+        }
+        if (IoLINES_LEFT(io) >= 0 && IoPAGE(io) > 0)
+            do_print(GvSV(gv_fetchpvs("\f", GV_ADD, SVt_PV)), ofp);
+        IoLINES_LEFT(io) = IoPAGE_LEN(io);
+        IoPAGE(io)++;
+        PL_formtarget = PL_toptarget;
+        IoFLAGS(io) |= IOf_DIDTOP;
+        fgv = IoTOP_GV(io);
+        assert(fgv); /* IoTOP_GV(io) should have been set above */
+        cv = GvFORM(fgv);
+        if (!cv) {
+            SV * const sv = sv_newmortal();
+            gv_efullname4(sv, fgv, NULL, FALSE);
+            DIE(aTHX_ "Undefined top format \"%" SVf "\" called", SVfARG(sv));
+        }
+        return doform(cv, gv, PL_op);
     }
 
   forget_top:
@@ -1530,33 +1561,35 @@ PP(pp_leavewrite)
     retop = cx->blk_sub.retop;
     CX_POP(cx);
 
+    EXTEND(SP, 1);
+
     if (is_return)
         /* XXX the semantics of doing 'return' in a format aren't documented.
          * Currently we ignore any args to 'return' and just return
          * a single undef in both scalar and list contexts
          */
-       PUSHs(&PL_sv_undef);
+        PUSHs(&PL_sv_undef);
     else if (!io || !(fp = IoOFP(io))) {
-       if (io && IoIFP(io))
-           report_wrongway_fh(gv, '<');
-       else
-           report_evil_fh(gv);
-       PUSHs(&PL_sv_no);
+        if (io && IoIFP(io))
+            report_wrongway_fh(gv, '<');
+        else
+            report_evil_fh(gv);
+        PUSHs(&PL_sv_no);
     }
     else {
-       if ((IoLINES_LEFT(io) -= FmLINES(PL_formtarget)) < 0) {
-           Perl_ck_warner(aTHX_ packWARN(WARN_IO), "page overflow");
-       }
-       if (!do_print(PL_formtarget, fp))
-           PUSHs(&PL_sv_no);
-       else {
-           FmLINES(PL_formtarget) = 0;
-           SvCUR_set(PL_formtarget, 0);
-           *SvEND(PL_formtarget) = '\0';
-           if (IoFLAGS(io) & IOf_FLUSH)
-               (void)PerlIO_flush(fp);
-           PUSHs(&PL_sv_yes);
-       }
+        if ((IoLINES_LEFT(io) -= FmLINES(PL_formtarget)) < 0) {
+            Perl_ck_warner(aTHX_ packWARN(WARN_IO), "page overflow");
+        }
+        if (!do_print(PL_formtarget, fp))
+            PUSHs(&PL_sv_no);
+        else {
+            FmLINES(PL_formtarget) = 0;
+            SvCUR_set(PL_formtarget, 0);
+            *SvEND(PL_formtarget) = '\0';
+            if (IoFLAGS(io) & IOf_FLUSH)
+                (void)PerlIO_flush(fp);
+            PUSHs(&PL_sv_yes);
+        }
     }
     PL_formtarget = PL_bodytarget;
     RETURNOP(retop);
@@ -1568,50 +1601,50 @@ PP(pp_prtf)
     PerlIO *fp;
 
     GV * const gv
-       = (PL_op->op_flags & OPf_STACKED) ? MUTABLE_GV(*++MARK) : PL_defoutgv;
+        = (PL_op->op_flags & OPf_STACKED) ? MUTABLE_GV(*++MARK) : PL_defoutgv;
     IO *const io = GvIO(gv);
 
     /* Treat empty list as "" */
     if (MARK == SP) XPUSHs(&PL_sv_no);
 
     if (io) {
-       const MAGIC * const mg = SvTIED_mg((const SV *)io, PERL_MAGIC_tiedscalar);
-       if (mg) {
-           if (MARK == ORIGMARK) {
-               MEXTEND(SP, 1);
-               ++MARK;
-               Move(MARK, MARK + 1, (SP - MARK) + 1, SV*);
-               ++SP;
-           }
-           return Perl_tied_method(aTHX_ SV_CONST(PRINTF), mark - 1, MUTABLE_SV(io),
-                                   mg,
-                                   G_SCALAR | TIED_METHOD_ARGUMENTS_ON_STACK,
-                                   sp - mark);
-       }
+        const MAGIC * const mg = SvTIED_mg((const SV *)io, PERL_MAGIC_tiedscalar);
+        if (mg) {
+            if (MARK == ORIGMARK) {
+                MEXTEND(SP, 1);
+                ++MARK;
+                Move(MARK, MARK + 1, (SP - MARK) + 1, SV*);
+                ++SP;
+            }
+            return Perl_tied_method(aTHX_ SV_CONST(PRINTF), mark - 1, MUTABLE_SV(io),
+                                    mg,
+                                    G_SCALAR | TIED_METHOD_ARGUMENTS_ON_STACK,
+                                    sp - mark);
+        }
     }
 
     if (!io) {
-       report_evil_fh(gv);
-       SETERRNO(EBADF,RMS_IFI);
-       goto just_say_no;
+        report_evil_fh(gv);
+        SETERRNO(EBADF,RMS_IFI);
+        goto just_say_no;
     }
     else if (!(fp = IoOFP(io))) {
-       if (IoIFP(io))
-           report_wrongway_fh(gv, '<');
-       else if (ckWARN(WARN_CLOSED))
-           report_evil_fh(gv);
-       SETERRNO(EBADF,IoIFP(io)?RMS_FAC:RMS_IFI);
-       goto just_say_no;
+        if (IoIFP(io))
+            report_wrongway_fh(gv, '<');
+        else if (ckWARN(WARN_CLOSED))
+            report_evil_fh(gv);
+        SETERRNO(EBADF,IoIFP(io)?RMS_FAC:RMS_IFI);
+        goto just_say_no;
     }
     else {
-       SV *sv = sv_newmortal();
-       do_sprintf(sv, SP - MARK, MARK + 1);
-       if (!do_print(sv, fp))
-           goto just_say_no;
+        SV *sv = sv_newmortal();
+        do_sprintf(sv, SP - MARK, MARK + 1);
+        if (!do_print(sv, fp))
+            goto just_say_no;
 
-       if (IoFLAGS(io) & IOf_FLUSH)
-           if (PerlIO_flush(fp) == EOF)
-               goto just_say_no;
+        if (IoFLAGS(io) & IOf_FLUSH)
+            if (PerlIO_flush(fp) == EOF)
+                goto just_say_no;
     }
     SP = ORIGMARK;
     PUSHs(&PL_sv_yes);
@@ -1634,12 +1667,12 @@ PP(pp_sysopen)
 
     /* Need TIEHANDLE method ? */
     const char * const tmps = SvPV_const(sv, len);
-    if (do_open_raw(gv, tmps, len, mode, perm)) {
-       IoLINES(GvIOp(gv)) = 0;
-       PUSHs(&PL_sv_yes);
+    if (do_open_raw(gv, tmps, len, mode, perm, NULL)) {
+        IoLINES(GvIOp(gv)) = 0;
+        PUSHs(&PL_sv_yes);
     }
     else {
-       PUSHs(&PL_sv_undef);
+        PUSHs(&PL_sv_undef);
     }
     RETURN;
 }
@@ -1670,34 +1703,34 @@ PP(pp_sysread)
     int fd;
 
     if ((PL_op->op_type == OP_READ || PL_op->op_type == OP_SYSREAD)
-       && gv && (io = GvIO(gv)) )
+        && gv && (io = GvIO(gv)) )
     {
-       const MAGIC *const mg = SvTIED_mg((const SV *)io, PERL_MAGIC_tiedscalar);
-       if (mg) {
-           return Perl_tied_method(aTHX_ SV_CONST(READ), mark - 1, MUTABLE_SV(io), mg,
-                                   G_SCALAR | TIED_METHOD_ARGUMENTS_ON_STACK,
-                                   sp - mark);
-       }
+        const MAGIC *const mg = SvTIED_mg((const SV *)io, PERL_MAGIC_tiedscalar);
+        if (mg) {
+            return Perl_tied_method(aTHX_ SV_CONST(READ), mark - 1, MUTABLE_SV(io), mg,
+                                    G_SCALAR | TIED_METHOD_ARGUMENTS_ON_STACK,
+                                    sp - mark);
+        }
     }
 
     if (!gv)
-       goto say_undef;
+        goto say_undef;
     bufsv = *++MARK;
     if (! SvOK(bufsv))
-       sv_setpvs(bufsv, "");
+        SvPVCLEAR(bufsv);
     length = SvIVx(*++MARK);
     if (length < 0)
-       DIE(aTHX_ "Negative length");
+        DIE(aTHX_ "Negative length");
     SETERRNO(0,0);
     if (MARK < SP)
-       offset = SvIVx(*++MARK);
+        offset = SvIVx(*++MARK);
     else
-       offset = 0;
+        offset = 0;
     io = GvIO(gv);
     if (!io || !IoIFP(io)) {
-       report_evil_fh(gv);
-       SETERRNO(EBADF,RMS_IFI);
-       goto say_undef;
+        report_evil_fh(gv);
+        SETERRNO(EBADF,RMS_IFI);
+        goto say_undef;
     }
 
     /* Note that fd can here validly be -1, don't check it yet. */
@@ -1705,21 +1738,21 @@ PP(pp_sysread)
 
     if ((fp_utf8 = PerlIO_isutf8(IoIFP(io))) && !IN_BYTES) {
         if (PL_op->op_type == OP_SYSREAD || PL_op->op_type == OP_RECV) {
-            Perl_ck_warner(aTHX_ packWARN(WARN_DEPRECATED),
-                           "%s() is deprecated on :utf8 handles",
-                           OP_DESC(PL_op));
+            Perl_croak(aTHX_
+                       "%s() isn't allowed on :utf8 handles",
+                       OP_DESC(PL_op));
         }
-       buffer = SvPVutf8_force(bufsv, blen);
-       /* UTF-8 may not have been set if they are all low bytes */
-       SvUTF8_on(bufsv);
-       buffer_utf8 = 0;
+        buffer = SvPVutf8_force(bufsv, blen);
+        /* UTF-8 may not have been set if they are all low bytes */
+        SvUTF8_on(bufsv);
+        buffer_utf8 = 0;
     }
     else {
-       buffer = SvPV_force(bufsv, blen);
-       buffer_utf8 = !IN_BYTES && SvUTF8(bufsv);
+        buffer = SvPV_force(bufsv, blen);
+        buffer_utf8 = DO_UTF8(bufsv);
     }
     if (DO_UTF8(bufsv)) {
-       blen = sv_len_utf8_nomg(bufsv);
+        blen = sv_len_utf8_nomg(bufsv);
     }
 
     charstart = TRUE;
@@ -1729,40 +1762,40 @@ PP(pp_sysread)
 
 #ifdef HAS_SOCKET
     if (PL_op->op_type == OP_RECV) {
-       Sock_size_t bufsize;
-       char namebuf[MAXPATHLEN];
+        Sock_size_t bufsize;
+        char namebuf[MAXPATHLEN];
         if (fd < 0) {
             SETERRNO(EBADF,SS_IVCHAN);
-            RETPUSHUNDEF;
+            goto say_undef;
         }
 #if (defined(VMS_DO_SOCKETS) && defined(DECCRTL_SOCKETS)) || defined(__QNXNTO__)
-       bufsize = sizeof (struct sockaddr_in);
+        bufsize = sizeof (struct sockaddr_in);
 #else
-       bufsize = sizeof namebuf;
+        bufsize = sizeof namebuf;
 #endif
 #ifdef OS2     /* At least Warp3+IAK: only the first byte of bufsize set */
-       if (bufsize >= 256)
-           bufsize = 255;
-#endif
-       buffer = SvGROW(bufsv, (STRLEN)(length+1));
-       /* 'offset' means 'flags' here */
-       count = PerlSock_recvfrom(fd, buffer, length, offset,
-                                 (struct sockaddr *)namebuf, &bufsize);
-       if (count < 0)
-           RETPUSHUNDEF;
-       /* MSG_TRUNC can give oversized count; quietly lose it */
-       if (count > length)
-           count = length;
-       SvCUR_set(bufsv, count);
-       *SvEND(bufsv) = '\0';
-       (void)SvPOK_only(bufsv);
-       if (fp_utf8)
-           SvUTF8_on(bufsv);
-       SvSETMAGIC(bufsv);
-       /* This should not be marked tainted if the fp is marked clean */
-       if (!(IoFLAGS(io) & IOf_UNTAINT))
-           SvTAINTED_on(bufsv);
-       SP = ORIGMARK;
+        if (bufsize >= 256)
+            bufsize = 255;
+#endif
+        buffer = SvGROW(bufsv, (STRLEN)(length+1));
+        /* 'offset' means 'flags' here */
+        count = PerlSock_recvfrom(fd, buffer, length, offset,
+                                  (struct sockaddr *)namebuf, &bufsize);
+        if (count < 0)
+            goto say_undef;
+        /* MSG_TRUNC can give oversized count; quietly lose it */
+        if (count > length)
+            count = length;
+        SvCUR_set(bufsv, count);
+        *SvEND(bufsv) = '\0';
+        (void)SvPOK_only(bufsv);
+        if (fp_utf8)
+            SvUTF8_on(bufsv);
+        SvSETMAGIC(bufsv);
+        /* This should not be marked tainted if the fp is marked clean */
+        if (!(IoFLAGS(io) & IOf_UNTAINT))
+            SvTAINTED_on(bufsv);
+        SP = ORIGMARK;
 #if defined(__CYGWIN__)
         /* recvfrom() on cygwin doesn't set bufsize at all for
            connected sockets, leaving us with trash in the returned
@@ -1771,22 +1804,22 @@ PP(pp_sysread)
         if (bufsize == sizeof namebuf)
             bufsize = 0;
 #endif
-       sv_setpvn(TARG, namebuf, bufsize);
-       PUSHs(TARG);
-       RETURN;
+        sv_setpvn(TARG, namebuf, bufsize);
+        PUSHs(TARG);
+        RETURN;
     }
 #endif
     if (offset < 0) {
-       if (-offset > (SSize_t)blen)
-           DIE(aTHX_ "Offset outside string");
-       offset += blen;
+        if (-offset > (SSize_t)blen)
+            DIE(aTHX_ "Offset outside string");
+        offset += blen;
     }
     if (DO_UTF8(bufsv)) {
-       /* convert offset-as-chars to offset-as-bytes */
-       if (offset >= (SSize_t)blen)
-           offset += SvCUR(bufsv) - blen;
-       else
-           offset = utf8_hop((U8 *)buffer,offset) - (U8 *) buffer;
+        /* convert offset-as-chars to offset-as-bytes */
+        if (offset >= (SSize_t)blen)
+            offset += SvCUR(bufsv) - blen;
+        else
+            offset = utf8_hop((U8 *)buffer,offset) - (U8 *) buffer;
     }
 
  more_bytes:
@@ -1801,104 +1834,104 @@ PP(pp_sysread)
        IN_ENCODING Is true) */
     buffer  = SvGROW(bufsv, (STRLEN)(length+offset+1));
     if (offset > 0 && offset > (SSize_t)orig_size) { /* Zero any newly allocated space */
-       Zero(buffer+orig_size, offset-orig_size, char);
+        Zero(buffer+orig_size, offset-orig_size, char);
     }
     buffer = buffer + offset;
     if (!buffer_utf8) {
-       read_target = bufsv;
+        read_target = bufsv;
     } else {
-       /* Best to read the bytes into a new SV, upgrade that to UTF8, then
-          concatenate it to the current buffer.  */
+        /* Best to read the bytes into a new SV, upgrade that to UTF8, then
+           concatenate it to the current buffer.  */
 
-       /* Truncate the existing buffer to the start of where we will be
-          reading to:  */
-       SvCUR_set(bufsv, offset);
+        /* Truncate the existing buffer to the start of where we will be
+           reading to:  */
+        SvCUR_set(bufsv, offset);
 
-       read_target = sv_newmortal();
-       SvUPGRADE(read_target, SVt_PV);
-       buffer = SvGROW(read_target, (STRLEN)(length + 1));
+        read_target = sv_newmortal();
+        SvUPGRADE(read_target, SVt_PV);
+        buffer = SvGROW(read_target, (STRLEN)(length + 1));
     }
 
     if (PL_op->op_type == OP_SYSREAD) {
 #ifdef PERL_SOCK_SYSREAD_IS_RECV
-       if (IoTYPE(io) == IoTYPE_SOCKET) {
+        if (IoTYPE(io) == IoTYPE_SOCKET) {
             if (fd < 0) {
                 SETERRNO(EBADF,SS_IVCHAN);
                 count = -1;
             }
             else
                 count = PerlSock_recv(fd, buffer, length, 0);
-       }
-       else
+        }
+        else
 #endif
-       {
+        {
             if (fd < 0) {
                 SETERRNO(EBADF,RMS_IFI);
                 count = -1;
             }
             else
                 count = PerlLIO_read(fd, buffer, length);
-       }
+        }
     }
     else
     {
-       count = PerlIO_read(IoIFP(io), buffer, length);
-       /* PerlIO_read() - like fread() returns 0 on both error and EOF */
-       if (count == 0 && PerlIO_error(IoIFP(io)))
-           count = -1;
+        count = PerlIO_read(IoIFP(io), buffer, length);
+        /* PerlIO_read() - like fread() returns 0 on both error and EOF */
+        if (count == 0 && PerlIO_error(IoIFP(io)))
+            count = -1;
     }
     if (count < 0) {
-       if (IoTYPE(io) == IoTYPE_WRONLY)
-           report_wrongway_fh(gv, '>');
-       goto say_undef;
+        if (IoTYPE(io) == IoTYPE_WRONLY)
+            report_wrongway_fh(gv, '>');
+        goto say_undef;
     }
     SvCUR_set(read_target, count+(buffer - SvPVX_const(read_target)));
     *SvEND(read_target) = '\0';
     (void)SvPOK_only(read_target);
     if (fp_utf8 && !IN_BYTES) {
-       /* Look at utf8 we got back and count the characters */
-       const char *bend = buffer + count;
-       while (buffer < bend) {
-           if (charstart) {
-               skip = UTF8SKIP(buffer);
-               charskip = 0;
-           }
-           if (buffer - charskip + skip > bend) {
-               /* partial character - try for rest of it */
-               length = skip - (bend-buffer);
-               offset = bend - SvPVX_const(bufsv);
-               charstart = FALSE;
-               charskip += count;
-               goto more_bytes;
-           }
-           else {
-               got++;
-               buffer += skip;
-               charstart = TRUE;
-               charskip  = 0;
-           }
+        /* Look at utf8 we got back and count the characters */
+        const char *bend = buffer + count;
+        while (buffer < bend) {
+            if (charstart) {
+                skip = UTF8SKIP(buffer);
+                charskip = 0;
+            }
+            if (buffer - charskip + skip > bend) {
+                /* partial character - try for rest of it */
+                length = skip - (bend-buffer);
+                offset = bend - SvPVX_const(bufsv);
+                charstart = FALSE;
+                charskip += count;
+                goto more_bytes;
+            }
+            else {
+                got++;
+                buffer += skip;
+                charstart = TRUE;
+                charskip  = 0;
+            }
         }
-       /* If we have not 'got' the number of _characters_ we 'wanted' get some more
-          provided amount read (count) was what was requested (length)
-        */
-       if (got < wanted && count == length) {
-           length = wanted - got;
-           offset = bend - SvPVX_const(bufsv);
-           goto more_bytes;
-       }
-       /* return value is character count */
-       count = got;
-       SvUTF8_on(bufsv);
+        /* If we have not 'got' the number of _characters_ we 'wanted' get some more
+           provided amount read (count) was what was requested (length)
+         */
+        if (got < wanted && count == length) {
+            length = wanted - got;
+            offset = bend - SvPVX_const(bufsv);
+            goto more_bytes;
+        }
+        /* return value is character count */
+        count = got;
+        SvUTF8_on(bufsv);
     }
     else if (buffer_utf8) {
-       /* Let svcatsv upgrade the bytes we read in to utf8.
-          The buffer is a mortal so will be freed soon.  */
-       sv_catsv_nomg(bufsv, read_target);
+        /* Let svcatsv upgrade the bytes we read in to utf8.
+           The buffer is a mortal so will be freed soon.  */
+        sv_catsv_nomg(bufsv, read_target);
     }
     SvSETMAGIC(bufsv);
     /* This should not be marked tainted if the fp is marked clean */
     if (!(IoFLAGS(io) & IOf_UNTAINT))
-       SvTAINTED_on(bufsv);
+        SvTAINTED_on(bufsv);
     SP = ORIGMARK;
     PUSHi(count);
     RETURN;
@@ -1918,7 +1951,6 @@ PP(pp_syswrite)
     const char *buffer;
     SSize_t retval;
     STRLEN blen;
-    STRLEN orig_blen_bytes;
     const int op_type = PL_op->op_type;
     bool doing_utf8;
     U8 *tmpbuf = NULL;
@@ -1927,33 +1959,33 @@ PP(pp_syswrite)
     int fd;
 
     if (op_type == OP_SYSWRITE && io) {
-       const MAGIC * const mg = SvTIED_mg((const SV *)io, PERL_MAGIC_tiedscalar);
-       if (mg) {
-           if (MARK == SP - 1) {
-               SV *sv = *SP;
-               mXPUSHi(sv_len(sv));
-               PUTBACK;
-           }
-
-           return Perl_tied_method(aTHX_ SV_CONST(WRITE), mark - 1, MUTABLE_SV(io), mg,
-                                   G_SCALAR | TIED_METHOD_ARGUMENTS_ON_STACK,
-                                   sp - mark);
-       }
+        const MAGIC * const mg = SvTIED_mg((const SV *)io, PERL_MAGIC_tiedscalar);
+        if (mg) {
+            if (MARK == SP - 1) {
+                SV *sv = *SP;
+                mXPUSHi(sv_len(sv));
+                PUTBACK;
+            }
+
+            return Perl_tied_method(aTHX_ SV_CONST(WRITE), mark - 1, MUTABLE_SV(io), mg,
+                                    G_SCALAR | TIED_METHOD_ARGUMENTS_ON_STACK,
+                                    sp - mark);
+        }
     }
     if (!gv)
-       goto say_undef;
+        goto say_undef;
 
     bufsv = *++MARK;
 
     SETERRNO(0,0);
     if (!io || !IoIFP(io) || IoTYPE(io) == IoTYPE_RDONLY) {
-       retval = -1;
-       if (io && IoIFP(io))
-           report_wrongway_fh(gv, '<');
-       else
-           report_evil_fh(gv);
-       SETERRNO(EBADF,RMS_IFI);
-       goto say_undef;
+        retval = -1;
+        if (io && IoIFP(io))
+            report_wrongway_fh(gv, '<');
+        else
+            report_evil_fh(gv);
+        SETERRNO(EBADF,RMS_IFI);
+        goto say_undef;
     }
     fd = PerlIO_fileno(IoIFP(io));
     if (fd < 0) {
@@ -1964,142 +1996,93 @@ PP(pp_syswrite)
 
     /* Do this first to trigger any overloading.  */
     buffer = SvPV_const(bufsv, blen);
-    orig_blen_bytes = blen;
     doing_utf8 = DO_UTF8(bufsv);
 
     if (PerlIO_isutf8(IoIFP(io))) {
-        Perl_ck_warner(aTHX_ packWARN(WARN_DEPRECATED),
-                       "%s() is deprecated on :utf8 handles",
-                       OP_DESC(PL_op));
-       if (!SvUTF8(bufsv)) {
-           /* We don't modify the original scalar.  */
-           tmpbuf = bytes_to_utf8((const U8*) buffer, &blen);
-           buffer = (char *) tmpbuf;
-           doing_utf8 = TRUE;
-       }
+        Perl_croak(aTHX_
+                   "%s() isn't allowed on :utf8 handles",
+                   OP_DESC(PL_op));
     }
     else if (doing_utf8) {
-       STRLEN tmplen = blen;
-       U8 * const result = bytes_from_utf8((const U8*) buffer, &tmplen, &doing_utf8);
-       if (!doing_utf8) {
-           tmpbuf = result;
-           buffer = (char *) tmpbuf;
-           blen = tmplen;
-       }
-       else {
-           assert((char *)result == buffer);
-           Perl_croak(aTHX_ "Wide character in %s", OP_DESC(PL_op));
-       }
+        STRLEN tmplen = blen;
+        U8 * const result = bytes_from_utf8((const U8*) buffer, &tmplen, &doing_utf8);
+        if (!doing_utf8) {
+            tmpbuf = result;
+            buffer = (char *) tmpbuf;
+            blen = tmplen;
+        }
+        else {
+            assert((char *)result == buffer);
+            Perl_croak(aTHX_ "Wide character in %s", OP_DESC(PL_op));
+        }
     }
 
 #ifdef HAS_SOCKET
     if (op_type == OP_SEND) {
-       const int flags = SvIVx(*++MARK);
-       if (SP > MARK) {
-           STRLEN mlen;
-           char * const sockbuf = SvPVx(*++MARK, mlen);
-           retval = PerlSock_sendto(fd, buffer, blen,
-                                    flags, (struct sockaddr *)sockbuf, mlen);
-       }
-       else {
-           retval = PerlSock_send(fd, buffer, blen, flags);
-       }
+        const int flags = SvIVx(*++MARK);
+        if (SP > MARK) {
+            STRLEN mlen;
+            char * const sockbuf = SvPVx(*++MARK, mlen);
+            retval = PerlSock_sendto(fd, buffer, blen,
+                                     flags, (struct sockaddr *)sockbuf, mlen);
+        }
+        else {
+            retval = PerlSock_send(fd, buffer, blen, flags);
+        }
     }
     else
 #endif
     {
-       Size_t length = 0; /* This length is in characters.  */
-       STRLEN blen_chars;
-       IV offset;
-
-       if (doing_utf8) {
-           if (tmpbuf) {
-               /* The SV is bytes, and we've had to upgrade it.  */
-               blen_chars = orig_blen_bytes;
-           } else {
-               /* The SV really is UTF-8.  */
-               /* Don't call sv_len_utf8 on a magical or overloaded
-                  scalar, as we might get back a different result.  */
-               blen_chars = sv_or_pv_len_utf8(bufsv, buffer, blen);
-           }
-       } else {
-           blen_chars = blen;
-       }
-
-       if (MARK >= SP) {
-           length = blen_chars;
-       } else {
+        Size_t length = 0; /* This length is in characters.  */
+        IV offset;
+
+        if (MARK >= SP) {
+            length = blen;
+        } else {
 #if Size_t_size > IVSIZE
-           length = (Size_t)SvNVx(*++MARK);
+            length = (Size_t)SvNVx(*++MARK);
 #else
-           length = (Size_t)SvIVx(*++MARK);
-#endif
-           if ((SSize_t)length < 0) {
-               Safefree(tmpbuf);
-               DIE(aTHX_ "Negative length");
-           }
-       }
-
-       if (MARK < SP) {
-           offset = SvIVx(*++MARK);
-           if (offset < 0) {
-               if (-offset > (IV)blen_chars) {
-                   Safefree(tmpbuf);
-                   DIE(aTHX_ "Offset outside string");
-               }
-               offset += blen_chars;
-           } else if (offset > (IV)blen_chars) {
-               Safefree(tmpbuf);
-               DIE(aTHX_ "Offset outside string");
-           }
-       } else
-           offset = 0;
-       if (length > blen_chars - offset)
-           length = blen_chars - offset;
-       if (doing_utf8) {
-           /* Here we convert length from characters to bytes.  */
-           if (tmpbuf || SvGMAGICAL(bufsv) || SvAMAGIC(bufsv)) {
-               /* Either we had to convert the SV, or the SV is magical, or
-                  the SV has overloading, in which case we can't or mustn't
-                  or mustn't call it again.  */
-
-               buffer = (const char*)utf8_hop((const U8 *)buffer, offset);
-               length = utf8_hop((U8 *)buffer, length) - (U8 *)buffer;
-           } else {
-               /* It's a real UTF-8 SV, and it's not going to change under
-                  us.  Take advantage of any cache.  */
-               I32 start = offset;
-               I32 len_I32 = length;
-
-               /* Convert the start and end character positions to bytes.
-                  Remember that the second argument to sv_pos_u2b is relative
-                  to the first.  */
-               sv_pos_u2b(bufsv, &start, &len_I32);
-
-               buffer += start;
-               length = len_I32;
-           }
-       }
-       else {
-           buffer = buffer+offset;
-       }
+            length = (Size_t)SvIVx(*++MARK);
+#endif
+            if ((SSize_t)length < 0) {
+                Safefree(tmpbuf);
+                DIE(aTHX_ "Negative length");
+            }
+        }
+
+        if (MARK < SP) {
+            offset = SvIVx(*++MARK);
+            if (offset < 0) {
+                if (-offset > (IV)blen) {
+                    Safefree(tmpbuf);
+                    DIE(aTHX_ "Offset outside string");
+                }
+                offset += blen;
+            } else if (offset > (IV)blen) {
+                Safefree(tmpbuf);
+                DIE(aTHX_ "Offset outside string");
+            }
+        } else
+            offset = 0;
+        if (length > blen - offset)
+            length = blen - offset;
+        buffer = buffer+offset;
+
 #ifdef PERL_SOCK_SYSWRITE_IS_SEND
-       if (IoTYPE(io) == IoTYPE_SOCKET) {
-           retval = PerlSock_send(fd, buffer, length, 0);
-       }
-       else
+        if (IoTYPE(io) == IoTYPE_SOCKET) {
+            retval = PerlSock_send(fd, buffer, length, 0);
+        }
+        else
 #endif
-       {
-           /* See the note at doio.c:do_print about filesize limits. --jhi */
+        {
+            /* See the note at doio.c:do_print about filesize limits. --jhi */
             retval = PerlLIO_write(fd, buffer, length);
-       }
+        }
     }
 
     if (retval < 0)
-       goto say_undef;
+        goto say_undef;
     SP = ORIGMARK;
-    if (doing_utf8)
-        retval = utf8_length((U8*)buffer, (U8*)buffer + retval);
 
     Safefree(tmpbuf);
 #if Size_t_size > IVSIZE
@@ -2134,48 +2117,48 @@ PP(pp_eof)
     unsigned int which;
 
     if (MAXARG) {
-       gv = PL_last_in_gv = MUTABLE_GV(POPs);  /* eof(FH) */
-       which = 1;
+        gv = PL_last_in_gv = MUTABLE_GV(POPs); /* eof(FH) */
+        which = 1;
     }
     else {
-       EXTEND(SP, 1);
+        EXTEND(SP, 1);
 
-       if (PL_op->op_flags & OPf_SPECIAL) {
-           gv = PL_last_in_gv = GvEGVx(PL_argvgv);     /* eof() - ARGV magic */
-           which = 2;
-       }
-       else {
-           gv = PL_last_in_gv;                 /* eof */
-           which = 0;
-       }
+        if (PL_op->op_flags & OPf_SPECIAL) {
+            gv = PL_last_in_gv = GvEGVx(PL_argvgv);    /* eof() - ARGV magic */
+            which = 2;
+        }
+        else {
+            gv = PL_last_in_gv;                        /* eof */
+            which = 0;
+        }
     }
 
     if (!gv)
-       RETPUSHNO;
+        RETPUSHYES;
 
     if ((io = GvIO(gv)) && (mg = SvTIED_mg((const SV *)io, PERL_MAGIC_tiedscalar))) {
-       return tied_method1(SV_CONST(EOF), SP, MUTABLE_SV(io), mg, newSVuv(which));
+        return tied_method1(SV_CONST(EOF), SP, MUTABLE_SV(io), mg, newSVuv(which));
     }
 
     if (!MAXARG && (PL_op->op_flags & OPf_SPECIAL)) {  /* eof() */
-       if (io && !IoIFP(io)) {
-           if ((IoFLAGS(io) & IOf_START) && av_tindex(GvAVn(gv)) < 0) {
-               SV ** svp;
-               IoLINES(io) = 0;
-               IoFLAGS(io) &= ~IOf_START;
-               do_open6(gv, "-", 1, NULL, NULL, 0);
-               svp = &GvSV(gv);
-               if (*svp) {
-                   SV * sv = *svp;
-                   sv_setpvs(sv, "-");
-                   SvSETMAGIC(sv);
-               }
-               else
-                   *svp = newSVpvs("-");
-           }
-           else if (!nextargv(gv, FALSE))
-               RETPUSHYES;
-       }
+        if (io && !IoIFP(io)) {
+            if ((IoFLAGS(io) & IOf_START) && av_count(GvAVn(gv)) == 0) {
+                SV ** svp;
+                IoLINES(io) = 0;
+                IoFLAGS(io) &= ~IOf_START;
+                do_open6(gv, "-", 1, NULL, NULL, 0);
+                svp = &GvSV(gv);
+                if (*svp) {
+                    SV * sv = *svp;
+                    sv_setpvs(sv, "-");
+                    SvSETMAGIC(sv);
+                }
+                else
+                    *svp = newSVpvs("-");
+            }
+            else if (!nextargv(gv, FALSE))
+                RETPUSHYES;
+        }
     }
 
     PUSHs(boolSV(do_eof(gv)));
@@ -2189,29 +2172,29 @@ PP(pp_tell)
     IO *io;
 
     if (MAXARG != 0 && (TOPs || POPs))
-       PL_last_in_gv = MUTABLE_GV(POPs);
+        PL_last_in_gv = MUTABLE_GV(POPs);
     else
-       EXTEND(SP, 1);
+        EXTEND(SP, 1);
     gv = PL_last_in_gv;
 
     io = GvIO(gv);
     if (io) {
-       const MAGIC * const mg = SvTIED_mg((const SV *)io, PERL_MAGIC_tiedscalar);
-       if (mg) {
-           return tied_method0(SV_CONST(TELL), SP, MUTABLE_SV(io), mg);
-       }
+        const MAGIC * const mg = SvTIED_mg((const SV *)io, PERL_MAGIC_tiedscalar);
+        if (mg) {
+            return tied_method0(SV_CONST(TELL), SP, MUTABLE_SV(io), mg);
+        }
     }
     else if (!gv) {
-       if (!errno)
-           SETERRNO(EBADF,RMS_IFI);
-       PUSHi(-1);
-       RETURN;
+        if (!errno)
+            SETERRNO(EBADF,RMS_IFI);
+        PUSHi(-1);
+        RETURN;
     }
 
 #if LSEEKSIZE > IVSIZE
-    PUSHn( do_tell(gv) );
+    PUSHn( (NV)do_tell(gv) );
 #else
-    PUSHi( do_tell(gv) );
+    PUSHi( (IV)do_tell(gv) );
 #endif
     RETURN;
 }
@@ -2233,23 +2216,23 @@ PP(pp_sysseek)
     IO *const io = GvIO(gv);
 
     if (io) {
-       const MAGIC * const mg = SvTIED_mg((const SV *)io, PERL_MAGIC_tiedscalar);
-       if (mg) {
+        const MAGIC * const mg = SvTIED_mg((const SV *)io, PERL_MAGIC_tiedscalar);
+        if (mg) {
 #if LSEEKSIZE > IVSIZE
-           SV *const offset_sv = newSVnv((NV) offset);
+            SV *const offset_sv = newSVnv((NV) offset);
 #else
-           SV *const offset_sv = newSViv(offset);
+            SV *const offset_sv = newSViv(offset);
 #endif
 
-           return tied_method2(SV_CONST(SEEK), SP, MUTABLE_SV(io), mg, offset_sv,
-                               newSViv(whence));
-       }
+            return tied_method2(SV_CONST(SEEK), SP, MUTABLE_SV(io), mg, offset_sv,
+                                newSViv(whence));
+        }
     }
 
     if (PL_op->op_type == OP_SEEK)
-       PUSHs(boolSV(do_seek(gv, offset, whence)));
+        PUSHs(boolSV(do_seek(gv, offset, whence)));
     else {
-       const Off_t sought = do_sysseek(gv, offset, whence);
+        const Off_t sought = do_sysseek(gv, offset, whence);
         if (sought < 0)
             PUSHs(&PL_sv_undef);
         else {
@@ -2286,25 +2269,25 @@ PP(pp_truncate)
     /* XXX Configure probe for the signedness of the length type of *truncate() needed? XXX */
     SETERRNO(0,0);
     {
-       SV * const sv = POPs;
-       int result = 1;
-       GV *tmpgv;
-       IO *io;
-
-       if (PL_op->op_flags & OPf_SPECIAL
-                      ? (tmpgv = gv_fetchsv(sv, 0, SVt_PVIO), 1)
-                      : !!(tmpgv = MAYBE_DEREF_GV(sv)) ) {
-           io = GvIO(tmpgv);
-           if (!io)
-               result = 0;
-           else {
-               PerlIO *fp;
-           do_ftruncate_io:
-               TAINT_PROPER("truncate");
-               if (!(fp = IoIFP(io))) {
-                   result = 0;
-               }
-               else {
+        SV * const sv = POPs;
+        int result = 1;
+        GV *tmpgv;
+        IO *io;
+
+        if (PL_op->op_flags & OPf_SPECIAL
+                       ? (tmpgv = gv_fetchsv(sv, 0, SVt_PVIO), 1)
+                       : !!(tmpgv = MAYBE_DEREF_GV(sv)) ) {
+            io = GvIO(tmpgv);
+            if (!io)
+                result = 0;
+            else {
+                PerlIO *fp;
+            do_ftruncate_io:
+                TAINT_PROPER("truncate");
+                if (!(fp = IoIFP(io))) {
+                    result = 0;
+                }
+                else {
                     int fd = PerlIO_fileno(fp);
                     if (fd < 0) {
                         SETERRNO(EBADF,RMS_IFI);
@@ -2323,21 +2306,21 @@ PP(pp_truncate)
                                result = 0;
                         }
                     }
-               }
-           }
-       }
-       else if (SvROK(sv) && SvTYPE(SvRV(sv)) == SVt_PVIO) {
-               io = MUTABLE_IO(SvRV(sv)); /* *main::FRED{IO} for example */
-               goto do_ftruncate_io;
-       }
-       else {
-           const char * const name = SvPV_nomg_const_nolen(sv);
-           TAINT_PROPER("truncate");
+                }
+            }
+        }
+        else if (SvROK(sv) && SvTYPE(SvRV(sv)) == SVt_PVIO) {
+                io = MUTABLE_IO(SvRV(sv)); /* *main::FRED{IO} for example */
+                goto do_ftruncate_io;
+        }
+        else {
+            const char * const name = SvPV_nomg_const_nolen(sv);
+            TAINT_PROPER("truncate");
 #ifdef HAS_TRUNCATE
-           if (truncate(name, len) < 0)
-               result = 0;
+            if (truncate(name, len) < 0)
+                result = 0;
 #else
-           {
+            {
                 int mode = O_RDWR;
                 int tmpfd;
 
@@ -2351,24 +2334,24 @@ PP(pp_truncate)
                  */
                 mode |= O_BINARY;
 #endif
-                tmpfd = PerlLIO_open(name, mode);
+                tmpfd = PerlLIO_open_cloexec(name, mode);
 
-               if (tmpfd < 0) {
-                   result = 0;
-               } else {
-                   if (my_chsize(tmpfd, len) < 0)
-                       result = 0;
-                   PerlLIO_close(tmpfd);
-               }
-           }
+                if (tmpfd < 0) {
+                    result = 0;
+                } else {
+                    if (my_chsize(tmpfd, len) < 0)
+                        result = 0;
+                    PerlLIO_close(tmpfd);
+                }
+            }
 #endif
-       }
+        }
 
-       if (result)
-           RETPUSHYES;
-       if (!errno)
-           SETERRNO(EBADF,RMS_IFI);
-       RETPUSHUNDEF;
+        if (result)
+            RETPUSHYES;
+        if (!errno)
+            SETERRNO(EBADF,RMS_IFI);
+        RETPUSHUNDEF;
     }
 }
 
@@ -2387,26 +2370,26 @@ PP(pp_ioctl)
     IV retval;
 
     if (!IoIFP(io)) {
-       report_evil_fh(gv);
-       SETERRNO(EBADF,RMS_IFI);        /* well, sort of... */
-       RETPUSHUNDEF;
+        report_evil_fh(gv);
+        SETERRNO(EBADF,RMS_IFI);       /* well, sort of... */
+        RETPUSHUNDEF;
     }
 
     if (SvPOK(argsv) || !SvNIOK(argsv)) {
-       STRLEN len;
-       STRLEN need;
-       s = SvPV_force(argsv, len);
-       need = IOCPARM_LEN(func);
-       if (len < need) {
-           s = Sv_Grow(argsv, need + 1);
-           SvCUR_set(argsv, need);
-       }
+        STRLEN len;
+        STRLEN need;
+        s = SvPV_force(argsv, len);
+        need = IOCPARM_LEN(func);
+        if (len < need) {
+            s = Sv_Grow(argsv, need + 1);
+            SvCUR_set(argsv, need);
+        }
 
-       s[SvCUR(argsv)] = 17;   /* a little sanity check here */
+        s[SvCUR(argsv)] = 17;  /* a little sanity check here */
     }
     else {
-       retval = SvIV(argsv);
-       s = INT2PTR(char*,retval);              /* ouch */
+        retval = SvIV(argsv);
+        s = INT2PTR(char*,retval);             /* ouch */
     }
 
     optype = PL_op->op_type;
@@ -2414,37 +2397,35 @@ PP(pp_ioctl)
 
     if (optype == OP_IOCTL)
 #ifdef HAS_IOCTL
-       retval = PerlLIO_ioctl(PerlIO_fileno(IoIFP(io)), func, s);
+        retval = PerlLIO_ioctl(PerlIO_fileno(IoIFP(io)), func, s);
 #else
-       DIE(aTHX_ "ioctl is not implemented");
+        DIE(aTHX_ "ioctl is not implemented");
 #endif
     else
 #ifndef HAS_FCNTL
       DIE(aTHX_ "fcntl is not implemented");
+#elif defined(OS2) && defined(__EMX__)
+        retval = fcntl(PerlIO_fileno(IoIFP(io)), func, (int)s);
 #else
-#if defined(OS2) && defined(__EMX__)
-       retval = fcntl(PerlIO_fileno(IoIFP(io)), func, (int)s);
-#else
-       retval = fcntl(PerlIO_fileno(IoIFP(io)), func, s);
-#endif
+        retval = fcntl(PerlIO_fileno(IoIFP(io)), func, s);
 #endif
 
 #if defined(HAS_IOCTL) || defined(HAS_FCNTL)
     if (SvPOK(argsv)) {
-       if (s[SvCUR(argsv)] != 17)
-           DIE(aTHX_ "Possible memory corruption: %s overflowed 3rd argument",
-               OP_NAME(PL_op));
-       s[SvCUR(argsv)] = 0;            /* put our null back */
-       SvSETMAGIC(argsv);              /* Assume it has changed */
+        if (s[SvCUR(argsv)] != 17)
+            DIE(aTHX_ "Possible memory corruption: %s overflowed 3rd argument",
+                OP_NAME(PL_op));
+        s[SvCUR(argsv)] = 0;           /* put our null back */
+        SvSETMAGIC(argsv);             /* Assume it has changed */
     }
 
     if (retval == -1)
-       RETPUSHUNDEF;
+        RETPUSHUNDEF;
     if (retval != 0) {
-       PUSHi(retval);
+        PUSHi(retval);
     }
     else {
-       PUSHp(zero_but_true, ZBTLEN);
+        PUSHp(zero_but_true, ZBTLEN);
     }
 #endif
     RETURN;
@@ -2462,13 +2443,13 @@ PP(pp_flock)
 
     /* XXX Looks to me like io is always NULL at this point */
     if (fp) {
-       (void)PerlIO_flush(fp);
-       value = (I32)(PerlLIO_flock(PerlIO_fileno(fp), argtype) >= 0);
+        (void)PerlIO_flush(fp);
+        value = (I32)(PerlLIO_flock(PerlIO_fileno(fp), argtype) >= 0);
     }
     else {
-       report_evil_fh(gv);
-       value = 0;
-       SETERRNO(EBADF,RMS_IFI);
+        report_evil_fh(gv);
+        value = 0;
+        SETERRNO(EBADF,RMS_IFI);
     }
     PUSHi(value);
     RETURN;
@@ -2492,27 +2473,22 @@ PP(pp_socket)
     int fd;
 
     if (IoIFP(io))
-       do_close(gv, FALSE);
+        do_close(gv, FALSE);
 
     TAINT_PROPER("socket");
-    fd = PerlSock_socket(domain, type, protocol);
+    fd = PerlSock_socket_cloexec(domain, type, protocol);
     if (fd < 0) {
-       RETPUSHUNDEF;
+        RETPUSHUNDEF;
     }
-    IoIFP(io) = PerlIO_fdopen(fd, "r"SOCKET_OPEN_MODE);        /* stdio gets confused about sockets */
-    IoOFP(io) = PerlIO_fdopen(fd, "w"SOCKET_OPEN_MODE);
+    IoIFP(io) = PerlIO_fdopen(fd, "r" SOCKET_OPEN_MODE); /* stdio gets confused about sockets */
+    IoOFP(io) = PerlIO_fdopen(fd, "w" SOCKET_OPEN_MODE);
     IoTYPE(io) = IoTYPE_SOCKET;
     if (!IoIFP(io) || !IoOFP(io)) {
-       if (IoIFP(io)) PerlIO_close(IoIFP(io));
-       if (IoOFP(io)) PerlIO_close(IoOFP(io));
-       if (!IoIFP(io) && !IoOFP(io)) PerlLIO_close(fd);
-       RETPUSHUNDEF;
+        if (IoIFP(io)) PerlIO_close(IoIFP(io));
+        if (IoOFP(io)) PerlIO_close(IoOFP(io));
+        if (!IoIFP(io) && !IoOFP(io)) PerlLIO_close(fd);
+        RETPUSHUNDEF;
     }
-#if defined(HAS_FCNTL) && defined(F_SETFD) && defined(FD_CLOEXEC)
-    /* ensure close-on-exec */
-    if (fd > PL_maxsysfd && fcntl(fd, F_SETFD, FD_CLOEXEC) < 0)
-       RETPUSHUNDEF;
-#endif
 
     RETPUSHYES;
 }
@@ -2533,34 +2509,28 @@ PP(pp_sockpair)
     IO * const io1 = GvIOn(gv1);
 
     if (IoIFP(io1))
-       do_close(gv1, FALSE);
+        do_close(gv1, FALSE);
     if (IoIFP(io2))
-       do_close(gv2, FALSE);
+        do_close(gv2, FALSE);
 
     TAINT_PROPER("socketpair");
-    if (PerlSock_socketpair(domain, type, protocol, fd) < 0)
-       RETPUSHUNDEF;
-    IoIFP(io1) = PerlIO_fdopen(fd[0], "r"SOCKET_OPEN_MODE);
-    IoOFP(io1) = PerlIO_fdopen(fd[0], "w"SOCKET_OPEN_MODE);
+    if (PerlSock_socketpair_cloexec(domain, type, protocol, fd) < 0)
+        RETPUSHUNDEF;
+    IoIFP(io1) = PerlIO_fdopen(fd[0], "r" SOCKET_OPEN_MODE);
+    IoOFP(io1) = PerlIO_fdopen(fd[0], "w" SOCKET_OPEN_MODE);
     IoTYPE(io1) = IoTYPE_SOCKET;
-    IoIFP(io2) = PerlIO_fdopen(fd[1], "r"SOCKET_OPEN_MODE);
-    IoOFP(io2) = PerlIO_fdopen(fd[1], "w"SOCKET_OPEN_MODE);
+    IoIFP(io2) = PerlIO_fdopen(fd[1], "r" SOCKET_OPEN_MODE);
+    IoOFP(io2) = PerlIO_fdopen(fd[1], "w" SOCKET_OPEN_MODE);
     IoTYPE(io2) = IoTYPE_SOCKET;
     if (!IoIFP(io1) || !IoOFP(io1) || !IoIFP(io2) || !IoOFP(io2)) {
-       if (IoIFP(io1)) PerlIO_close(IoIFP(io1));
-       if (IoOFP(io1)) PerlIO_close(IoOFP(io1));
-       if (!IoIFP(io1) && !IoOFP(io1)) PerlLIO_close(fd[0]);
-       if (IoIFP(io2)) PerlIO_close(IoIFP(io2));
-       if (IoOFP(io2)) PerlIO_close(IoOFP(io2));
-       if (!IoIFP(io2) && !IoOFP(io2)) PerlLIO_close(fd[1]);
-       RETPUSHUNDEF;
+        if (IoIFP(io1)) PerlIO_close(IoIFP(io1));
+        if (IoOFP(io1)) PerlIO_close(IoOFP(io1));
+        if (!IoIFP(io1) && !IoOFP(io1)) PerlLIO_close(fd[0]);
+        if (IoIFP(io2)) PerlIO_close(IoIFP(io2));
+        if (IoOFP(io2)) PerlIO_close(IoOFP(io2));
+        if (!IoIFP(io2) && !IoOFP(io2)) PerlLIO_close(fd[1]);
+        RETPUSHUNDEF;
     }
-#if defined(HAS_FCNTL) && defined(F_SETFD) && defined(FD_CLOEXEC)
-    /* ensure close-on-exec */
-    if ((fd[0] > PL_maxsysfd && fcntl(fd[0], F_SETFD, FD_CLOEXEC) < 0) ||
-        (fd[1] > PL_maxsysfd && fcntl(fd[1], F_SETFD, FD_CLOEXEC) < 0))
-       RETPUSHUNDEF;
-#endif
 
     RETPUSHYES;
 #else
@@ -2585,7 +2555,7 @@ PP(pp_bind)
     int fd;
 
     if (!IoIFP(io))
-       goto nuts;
+        goto nuts;
     fd = PerlIO_fileno(IoIFP(io));
     if (fd < 0)
         goto nuts;
@@ -2594,12 +2564,12 @@ PP(pp_bind)
     op_type = PL_op->op_type;
     TAINT_PROPER(PL_op_desc[op_type]);
     if ((op_type == OP_BIND
-        ? PerlSock_bind(fd, (struct sockaddr *)addr, len)
-        : PerlSock_connect(fd, (struct sockaddr *)addr, len))
-       >= 0)
-       RETPUSHYES;
+         ? PerlSock_bind(fd, (struct sockaddr *)addr, len)
+         : PerlSock_connect(fd, (struct sockaddr *)addr, len))
+        >= 0)
+        RETPUSHYES;
     else
-       RETPUSHUNDEF;
+        RETPUSHUNDEF;
 
   nuts:
     report_evil_fh(gv);
@@ -2615,12 +2585,12 @@ PP(pp_listen)
     IO * const io = GvIOn(gv);
 
     if (!IoIFP(io))
-       goto nuts;
+        goto nuts;
 
     if (PerlSock_listen(PerlIO_fileno(IoIFP(io)), backlog) >= 0)
-       RETPUSHYES;
+        RETPUSHYES;
     else
-       RETPUSHUNDEF;
+        RETPUSHUNDEF;
 
   nuts:
     report_evil_fh(gv);
@@ -2644,39 +2614,34 @@ PP(pp_accept)
 
     IO * const gstio = GvIO(ggv);
     if (!gstio || !IoIFP(gstio))
-       goto nuts;
+        goto nuts;
 
     nstio = GvIOn(ngv);
-    fd = PerlSock_accept(PerlIO_fileno(IoIFP(gstio)), (struct sockaddr *) namebuf, &len);
+    fd = PerlSock_accept_cloexec(PerlIO_fileno(IoIFP(gstio)), (struct sockaddr *) namebuf, &len);
 #if defined(OEMVS)
     if (len == 0) {
-       /* Some platforms indicate zero length when an AF_UNIX client is
-        * not bound. Simulate a non-zero-length sockaddr structure in
-        * this case. */
-       namebuf[0] = 0;        /* sun_len */
-       namebuf[1] = AF_UNIX;  /* sun_family */
-       len = 2;
+        /* Some platforms indicate zero length when an AF_UNIX client is
+         * not bound. Simulate a non-zero-length sockaddr structure in
+         * this case. */
+        namebuf[0] = 0;        /* sun_len */
+        namebuf[1] = AF_UNIX;  /* sun_family */
+        len = 2;
     }
 #endif
 
     if (fd < 0)
-       goto badexit;
+        goto badexit;
     if (IoIFP(nstio))
-       do_close(ngv, FALSE);
-    IoIFP(nstio) = PerlIO_fdopen(fd, "r"SOCKET_OPEN_MODE);
-    IoOFP(nstio) = PerlIO_fdopen(fd, "w"SOCKET_OPEN_MODE);
+        do_close(ngv, FALSE);
+    IoIFP(nstio) = PerlIO_fdopen(fd, "r" SOCKET_OPEN_MODE);
+    IoOFP(nstio) = PerlIO_fdopen(fd, "w" SOCKET_OPEN_MODE);
     IoTYPE(nstio) = IoTYPE_SOCKET;
     if (!IoIFP(nstio) || !IoOFP(nstio)) {
-       if (IoIFP(nstio)) PerlIO_close(IoIFP(nstio));
-       if (IoOFP(nstio)) PerlIO_close(IoOFP(nstio));
-       if (!IoIFP(nstio) && !IoOFP(nstio)) PerlLIO_close(fd);
-       goto badexit;
-    }
-#if defined(HAS_FCNTL) && defined(F_SETFD) && defined(FD_CLOEXEC)
-    /* ensure close-on-exec */
-    if (fd > PL_maxsysfd && fcntl(fd, F_SETFD, FD_CLOEXEC) < 0)
+        if (IoIFP(nstio)) PerlIO_close(IoIFP(nstio));
+        if (IoOFP(nstio)) PerlIO_close(IoOFP(nstio));
+        if (!IoIFP(nstio) && !IoOFP(nstio)) PerlLIO_close(fd);
         goto badexit;
-#endif
+    }
 
 #ifdef __SCO_VERSION__
     len = sizeof (struct sockaddr_in); /* OpenUNIX 8 somehow truncates info */
@@ -2702,7 +2667,7 @@ PP(pp_shutdown)
     IO * const io = GvIOn(gv);
 
     if (!IoIFP(io))
-       goto nuts;
+        goto nuts;
 
     PUSHi( PerlSock_shutdown(PerlIO_fileno(IoIFP(io)), how) >= 0 );
     RETURN;
@@ -2729,61 +2694,48 @@ PP(pp_ssockopt)
     Sock_size_t len;
 
     if (!IoIFP(io))
-       goto nuts;
+        goto nuts;
 
     fd = PerlIO_fileno(IoIFP(io));
     if (fd < 0)
         goto nuts;
     switch (optype) {
     case OP_GSOCKOPT:
-       SvGROW(sv, 257);
-       (void)SvPOK_only(sv);
-       SvCUR_set(sv,256);
-       *SvEND(sv) ='\0';
-       len = SvCUR(sv);
-       if (PerlSock_getsockopt(fd, lvl, optname, SvPVX(sv), &len) < 0)
-           goto nuts2;
+        SvGROW(sv, 257);
+        (void)SvPOK_only(sv);
+        SvCUR_set(sv,256);
+        *SvEND(sv) ='\0';
+        len = SvCUR(sv);
+        if (PerlSock_getsockopt(fd, lvl, optname, SvPVX(sv), &len) < 0)
+            goto nuts2;
 #if defined(_AIX)
         /* XXX Configure test: does getsockopt set the length properly? */
         if (len == 256)
             len = sizeof(int);
 #endif
-       SvCUR_set(sv, len);
-       *SvEND(sv) ='\0';
-       PUSHs(sv);
-       break;
+        SvCUR_set(sv, len);
+        *SvEND(sv) ='\0';
+        PUSHs(sv);
+        break;
     case OP_SSOCKOPT: {
-#if defined(__SYMBIAN32__)
-# define SETSOCKOPT_OPTION_VALUE_T void *
-#else
-# define SETSOCKOPT_OPTION_VALUE_T const char *
-#endif
-       /* XXX TODO: We need to have a proper type (a Configure probe,
-        * etc.) for what the C headers think of the third argument of
-        * setsockopt(), the option_value read-only buffer: is it
-        * a "char *", or a "void *", const or not.  Some compilers
-        * don't take kindly to e.g. assuming that "char *" implicitly
-        * promotes to a "void *", or to explicitly promoting/demoting
-        * consts to non/vice versa.  The "const void *" is the SUS
-        * definition, but that does not fly everywhere for the above
-        * reasons. */
-           SETSOCKOPT_OPTION_VALUE_T buf;
-           int aint;
-           if (SvPOKp(sv)) {
-               STRLEN l;
-               buf = (SETSOCKOPT_OPTION_VALUE_T) SvPV_const(sv, l);
-               len = l;
-           }
-           else {
-               aint = (int)SvIV(sv);
-               buf = (SETSOCKOPT_OPTION_VALUE_T) &aint;
-               len = sizeof(int);
-           }
-           if (PerlSock_setsockopt(fd, lvl, optname, buf, len) < 0)
-               goto nuts2;
-           PUSHs(&PL_sv_yes);
-       }
-       break;
+            const char *buf;
+            int aint;
+            SvGETMAGIC(sv);
+            if (SvPOKp(sv)) {
+                STRLEN l;
+                buf = SvPVbyte_nomg(sv, l);
+                len = l;
+            }
+            else {
+                aint = (int)SvIV_nomg(sv);
+                buf = (const char *) &aint;
+                len = sizeof(int);
+            }
+            if (PerlSock_setsockopt(fd, lvl, optname, buf, len) < 0)
+                goto nuts2;
+            PUSHs(&PL_sv_yes);
+        }
+        break;
     }
     RETURN;
 
@@ -2809,11 +2761,15 @@ PP(pp_getpeername)
     int fd;
 
     if (!IoIFP(io))
-       goto nuts;
+        goto nuts;
 
-    sv = sv_2mortal(newSV(257));
-    (void)SvPOK_only(sv);
+#ifdef HAS_SOCKADDR_STORAGE
+    len = sizeof(struct sockaddr_storage);
+#else
     len = 256;
+#endif
+    sv = sv_2mortal(newSV(len+1));
+    (void)SvPOK_only(sv);
     SvCUR_set(sv, len);
     *SvEND(sv) ='\0';
     fd = PerlIO_fileno(IoIFP(io));
@@ -2821,30 +2777,30 @@ PP(pp_getpeername)
         goto nuts;
     switch (optype) {
     case OP_GETSOCKNAME:
-       if (PerlSock_getsockname(fd, (struct sockaddr *)SvPVX(sv), &len) < 0)
-           goto nuts2;
-       break;
+        if (PerlSock_getsockname(fd, (struct sockaddr *)SvPVX(sv), &len) < 0)
+            goto nuts2;
+        break;
     case OP_GETPEERNAME:
-       if (PerlSock_getpeername(fd, (struct sockaddr *)SvPVX(sv), &len) < 0)
-           goto nuts2;
+        if (PerlSock_getpeername(fd, (struct sockaddr *)SvPVX(sv), &len) < 0)
+            goto nuts2;
 #if defined(VMS_DO_SOCKETS) && defined (DECCRTL_SOCKETS)
-       {
-           static const char nowhere[] = "\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0";
-           /* If the call succeeded, make sure we don't have a zeroed port/addr */
-           if (((struct sockaddr *)SvPVX_const(sv))->sa_family == AF_INET &&
-               !memcmp(SvPVX_const(sv) + sizeof(u_short), nowhere,
-                       sizeof(u_short) + sizeof(struct in_addr))) {
-               goto nuts2;     
-           }
-       }
-#endif
-       break;
+        {
+            static const char nowhere[] = "\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0";
+            /* If the call succeeded, make sure we don't have a zeroed port/addr */
+            if (((struct sockaddr *)SvPVX_const(sv))->sa_family == AF_INET &&
+                !memcmp(SvPVX_const(sv) + sizeof(u_short), nowhere,
+                        sizeof(u_short) + sizeof(struct in_addr))) {
+                goto nuts2;    
+            }
+        }
+#endif
+        break;
     }
 #ifdef BOGUS_GETNAME_RETURN
     /* Interactive Unix, getpeername() and getsockname()
       does not return valid namelen */
     if (len == BOGUS_GETNAME_RETURN)
-       len = sizeof(struct sockaddr);
+        len = sizeof(struct sockaddr);
 #endif
     SvCUR_set(sv, len);
     *SvEND(sv) ='\0';
@@ -2875,135 +2831,223 @@ PP(pp_stat)
 
     if (PL_op->op_flags & OPf_REF ? (gv = cGVOP_gv, 1)
                                   : !!(sv=POPs, gv = MAYBE_DEREF_GV(sv))) {
-       if (PL_op->op_type == OP_LSTAT) {
-           if (gv != PL_defgv) {
-           do_fstat_warning_check:
-               Perl_ck_warner(aTHX_ packWARN(WARN_IO),
-                              "lstat() on filehandle%s%"SVf,
-                               gv ? " " : "",
-                               SVfARG(gv
+        if (PL_op->op_type == OP_LSTAT) {
+            if (gv != PL_defgv) {
+            do_fstat_warning_check:
+                Perl_ck_warner(aTHX_ packWARN(WARN_IO),
+                               "lstat() on filehandle%s%" SVf,
+                                gv ? " " : "",
+                                SVfARG(gv
                                         ? sv_2mortal(newSVhek(GvENAME_HEK(gv)))
                                         : &PL_sv_no));
-           } else if (PL_laststype != OP_LSTAT)
-               /* diag_listed_as: The stat preceding %s wasn't an lstat */
-               Perl_croak(aTHX_ "The stat preceding lstat() wasn't an lstat");
-       }
+            } else if (PL_laststype != OP_LSTAT)
+                /* diag_listed_as: The stat preceding %s wasn't an lstat */
+                Perl_croak(aTHX_ "The stat preceding lstat() wasn't an lstat");
+        }
 
-       if (gv != PL_defgv) {
-           bool havefp;
+        if (gv == PL_defgv) {
+            if (PL_laststatval < 0)
+                SETERRNO(EBADF,RMS_IFI);
+        } else {
           do_fstat_have_io:
-           havefp = FALSE;
-           PL_laststype = OP_STAT;
-           PL_statgv = gv ? gv : (GV *)io;
-           sv_setpvs(PL_statname, "");
+            PL_laststype = OP_STAT;
+            PL_statgv = gv ? gv : (GV *)io;
+            SvPVCLEAR(PL_statname);
             if(gv) {
                 io = GvIO(gv);
-           }
+            }
             if (io) {
                     if (IoIFP(io)) {
                         int fd = PerlIO_fileno(IoIFP(io));
                         if (fd < 0) {
+                            report_evil_fh(gv);
                             PL_laststatval = -1;
                             SETERRNO(EBADF,RMS_IFI);
                         } else {
                             PL_laststatval = PerlLIO_fstat(fd, &PL_statcache);
-                            havefp = TRUE;
                         }
                     } else if (IoDIRP(io)) {
                         PL_laststatval =
                             PerlLIO_fstat(my_dirfd(IoDIRP(io)), &PL_statcache);
-                        havefp = TRUE;
                     } else {
+                        report_evil_fh(gv);
                         PL_laststatval = -1;
+                        SETERRNO(EBADF,RMS_IFI);
                     }
+            } else {
+                report_evil_fh(gv);
+                PL_laststatval = -1;
+                SETERRNO(EBADF,RMS_IFI);
             }
-           else PL_laststatval = -1;
-           if (PL_laststatval < 0 && !havefp) report_evil_fh(gv);
         }
 
-       if (PL_laststatval < 0) {
-           max = 0;
-       }
+        if (PL_laststatval < 0) {
+            max = 0;
+        }
     }
     else {
         const char *file;
-       if (SvROK(sv) && SvTYPE(SvRV(sv)) == SVt_PVIO) { 
+        const char *temp;
+        STRLEN len;
+        if (SvROK(sv) && SvTYPE(SvRV(sv)) == SVt_PVIO) { 
             io = MUTABLE_IO(SvRV(sv));
             if (PL_op->op_type == OP_LSTAT)
                 goto do_fstat_warning_check;
             goto do_fstat_have_io; 
         }
-        
-       SvTAINTED_off(PL_statname); /* previous tainting irrelevant */
-       sv_setpv(PL_statname, SvPV_nomg_const_nolen(sv));
-       PL_statgv = NULL;
-       PL_laststype = PL_op->op_type;
+        SvTAINTED_off(PL_statname); /* previous tainting irrelevant */
+        temp = SvPV_nomg_const(sv, len);
+        sv_setpv(PL_statname, temp);
+        PL_statgv = NULL;
+        PL_laststype = PL_op->op_type;
         file = SvPV_nolen_const(PL_statname);
-       if (PL_op->op_type == OP_LSTAT)
-           PL_laststatval = PerlLIO_lstat(file, &PL_statcache);
-       else
-           PL_laststatval = PerlLIO_stat(file, &PL_statcache);
-       if (PL_laststatval < 0) {
-           if (ckWARN(WARN_NEWLINE) && should_warn_nl(file)) {
+        if (!IS_SAFE_PATHNAME(temp, len, OP_NAME(PL_op))) {
+            PL_laststatval = -1;
+        }
+        else if (PL_op->op_type == OP_LSTAT)
+            PL_laststatval = PerlLIO_lstat(file, &PL_statcache);
+        else
+            PL_laststatval = PerlLIO_stat(file, &PL_statcache);
+        if (PL_laststatval < 0) {
+            if (ckWARN(WARN_NEWLINE) && should_warn_nl(file)) {
                 /* PL_warn_nl is constant */
-                GCC_DIAG_IGNORE(-Wformat-nonliteral);
-               Perl_warner(aTHX_ packWARN(WARN_NEWLINE), PL_warn_nl, "stat");
-                GCC_DIAG_RESTORE;
+                GCC_DIAG_IGNORE_STMT(-Wformat-nonliteral);
+                Perl_warner(aTHX_ packWARN(WARN_NEWLINE), PL_warn_nl, "stat");
+                GCC_DIAG_RESTORE_STMT;
             }
-           max = 0;
-       }
+            max = 0;
+        }
     }
 
     gimme = GIMME_V;
-    if (gimme != G_ARRAY) {
-       if (gimme != G_VOID)
-           XPUSHs(boolSV(max));
-       RETURN;
+    if (gimme != G_LIST) {
+        if (gimme != G_VOID)
+            XPUSHs(boolSV(max));
+        RETURN;
     }
     if (max) {
-       EXTEND(SP, max);
-       EXTEND_MORTAL(max);
-       mPUSHi(PL_statcache.st_dev);
-#if ST_INO_SIZE > IVSIZE
-       mPUSHn(PL_statcache.st_ino);
+        EXTEND(SP, max);
+        EXTEND_MORTAL(max);
+#if ST_DEV_SIZE < IVSIZE || (ST_DEV_SIZE == IVSIZE && ST_DEV_SIGN < 0)
+        mPUSHi(PL_statcache.st_dev);
+#elif ST_DEV_SIZE == IVSIZE
+        mPUSHu(PL_statcache.st_dev);
 #else
-#   if ST_INO_SIGN <= 0
-       mPUSHi(PL_statcache.st_ino);
-#   else
-       mPUSHu(PL_statcache.st_ino);
-#   endif
+#  if ST_DEV_SIGN < 0
+        if (LIKELY((IV)PL_statcache.st_dev == PL_statcache.st_dev)) {
+            mPUSHi((IV)PL_statcache.st_dev);
+        }
+#  else
+        if (LIKELY((UV)PL_statcache.st_dev == PL_statcache.st_dev)) {
+            mPUSHu((UV)PL_statcache.st_dev);
+        }
+#  endif
+        else {
+            char buf[sizeof(PL_statcache.st_dev)*3+1];
+            /* sv_catpvf() casts 'j' size values down to IV, so it
+               isn't suitable for use here.
+            */
+#    if defined(I_INTTYPES) && defined(HAS_SNPRINTF)
+#      if ST_DEV_SIGN < 0
+            int size = snprintf(buf, sizeof(buf), "%" PRIdMAX, (intmax_t)PL_statcache.st_dev);
+#      else
+            int size = snprintf(buf, sizeof(buf), "%" PRIuMAX, (uintmax_t)PL_statcache.st_dev);
+#      endif
+            STATIC_ASSERT_STMT(sizeof(intmax_t) >= sizeof(PL_statcache.st_dev));
+            mPUSHp(buf, size);
+#    else
+#      error extraordinarily large st_dev but no inttypes.h or no snprintf
+#    endif
+        }
 #endif
-       mPUSHu(PL_statcache.st_mode);
-       mPUSHu(PL_statcache.st_nlink);
-       
+        {
+            /*
+             * We try to represent st_ino as a native IV or UV where
+             * possible, but fall back to a decimal string where
+             * necessary.  The code to generate these decimal strings
+             * is quite obtuse, because (a) we're portable to non-POSIX
+             * platforms where st_ino might be signed; (b) we didn't
+             * necessarily detect at Configure time whether st_ino is
+             * signed; (c) we're portable to non-POSIX platforms where
+             * ino_t isn't defined, so have no name for the type of
+             * st_ino; and (d) sprintf() doesn't necessarily support
+             * integers as large as st_ino.
+             */
+            bool neg;
+            Stat_t s;
+            CLANG_DIAG_IGNORE_STMT(-Wtautological-compare);
+            GCC_DIAG_IGNORE_STMT(-Wtype-limits);
+            neg = PL_statcache.st_ino < 0;
+            GCC_DIAG_RESTORE_STMT;
+            CLANG_DIAG_RESTORE_STMT;
+            if (neg) {
+                s.st_ino = (IV)PL_statcache.st_ino;
+                if (LIKELY(s.st_ino == PL_statcache.st_ino)) {
+                    mPUSHi(s.st_ino);
+                } else {
+                    char buf[sizeof(s.st_ino)*3+1], *p;
+                    s.st_ino = PL_statcache.st_ino;
+                    for (p = buf + sizeof(buf); p != buf+1; ) {
+                        Stat_t t;
+                        t.st_ino = s.st_ino / 10;
+                        *--p = '0' + (int)(t.st_ino*10 - s.st_ino);
+                        s.st_ino = t.st_ino;
+                    }
+                    while (*p == '0')
+                        p++;
+                    *--p = '-';
+                    mPUSHp(p, buf+sizeof(buf) - p);
+                }
+            } else {
+                s.st_ino = (UV)PL_statcache.st_ino;
+                if (LIKELY(s.st_ino == PL_statcache.st_ino)) {
+                    mPUSHu(s.st_ino);
+                } else {
+                    char buf[sizeof(s.st_ino)*3], *p;
+                    s.st_ino = PL_statcache.st_ino;
+                    for (p = buf + sizeof(buf); p != buf; ) {
+                        Stat_t t;
+                        t.st_ino = s.st_ino / 10;
+                        *--p = '0' + (int)(s.st_ino - t.st_ino*10);
+                        s.st_ino = t.st_ino;
+                    }
+                    while (*p == '0')
+                        p++;
+                    mPUSHp(p, buf+sizeof(buf) - p);
+                }
+            }
+        }
+        mPUSHu(PL_statcache.st_mode);
+        mPUSHu(PL_statcache.st_nlink);
+        
         sv_setuid(PUSHmortal, PL_statcache.st_uid);
         sv_setgid(PUSHmortal, PL_statcache.st_gid);
 
 #ifdef USE_STAT_RDEV
-       mPUSHi(PL_statcache.st_rdev);
+        mPUSHi(PL_statcache.st_rdev);
 #else
-       PUSHs(newSVpvs_flags("", SVs_TEMP));
+        PUSHs(newSVpvs_flags("", SVs_TEMP));
 #endif
 #if Off_t_size > IVSIZE
-       mPUSHn(PL_statcache.st_size);
+        mPUSHn(PL_statcache.st_size);
 #else
-       mPUSHi(PL_statcache.st_size);
+        mPUSHi(PL_statcache.st_size);
 #endif
 #ifdef BIG_TIME
-       mPUSHn(PL_statcache.st_atime);
-       mPUSHn(PL_statcache.st_mtime);
-       mPUSHn(PL_statcache.st_ctime);
+        mPUSHn(PL_statcache.st_atime);
+        mPUSHn(PL_statcache.st_mtime);
+        mPUSHn(PL_statcache.st_ctime);
 #else
-       mPUSHi(PL_statcache.st_atime);
-       mPUSHi(PL_statcache.st_mtime);
-       mPUSHi(PL_statcache.st_ctime);
+        mPUSHi(PL_statcache.st_atime);
+        mPUSHi(PL_statcache.st_mtime);
+        mPUSHi(PL_statcache.st_ctime);
 #endif
 #ifdef USE_STAT_BLOCKS
-       mPUSHu(PL_statcache.st_blksize);
-       mPUSHu(PL_statcache.st_blocks);
+        mPUSHu(PL_statcache.st_blksize);
+        mPUSHu(PL_statcache.st_blocks);
 #else
-       PUSHs(newSVpvs_flags("", SVs_TEMP));
-       PUSHs(newSVpvs_flags("", SVs_TEMP));
+        PUSHs(newSVpvs_flags("", SVs_TEMP));
+        PUSHs(newSVpvs_flags("", SVs_TEMP));
 #endif
     }
     RETURN;
@@ -3033,7 +3077,7 @@ S_ft_return_false(pTHX_ SV *ret) {
     PUTBACK;
 
     if (PL_op->op_private & OPpFT_STACKING) {
-        while (OP_IS_FILETEST(next->op_type)
+        while (next && OP_IS_FILETEST(next->op_type)
                && next->op_private & OPpFT_STACKED)
             next = next->op_next;
     }
@@ -3056,11 +3100,11 @@ S_ft_return_true(pTHX_ SV *ret) {
 #define FT_RETURNYES   return S_ft_return_true(aTHX_ &PL_sv_yes)
 
 #define tryAMAGICftest_MG(chr) STMT_START { \
-       if ( (SvFLAGS(*PL_stack_sp) & (SVf_ROK|SVs_GMG)) \
-               && PL_op->op_flags & OPf_KIDS) {     \
-           OP *next = S_try_amagic_ftest(aTHX_ chr);   \
-           if (next) return next;                        \
-                                                        \
+        if ( (SvFLAGS(*PL_stack_sp) & (SVf_ROK|SVs_GMG)) \
+                && PL_op->op_flags & OPf_KIDS) {     \
+            OP *next = S_try_amagic_ftest(aTHX_ chr);  \
+            if (next) return next;                       \
+        }                                                 \
     } STMT_END
 
 STATIC OP *
@@ -3068,19 +3112,19 @@ S_try_amagic_ftest(pTHX_ char chr) {
     SV *const arg = *PL_stack_sp;
 
     assert(chr != '?');
-    if (!(PL_op->op_private & OPpFT_STACKING)) SvGETMAGIC(arg);
+    if (!(PL_op->op_private & OPpFT_STACKED)) SvGETMAGIC(arg);
 
     if (SvAMAGIC(arg))
     {
-       const char tmpchr = chr;
-       SV * const tmpsv = amagic_call(arg,
-                               newSVpvn_flags(&tmpchr, 1, SVs_TEMP),
-                               ftest_amg, AMGf_unary);
+        const char tmpchr = chr;
+        SV * const tmpsv = amagic_call(arg,
+                                newSVpvn_flags(&tmpchr, 1, SVs_TEMP),
+                                ftest_amg, AMGf_unary);
 
-       if (!tmpsv)
-           return NULL;
+        if (!tmpsv)
+            return NULL;
 
-       return SvTRUE(tmpsv)
+        return SvTRUE(tmpsv)
             ? S_ft_return_true(aTHX_ tmpsv) : S_ft_return_false(aTHX_ tmpsv);
     }
     return NULL;
@@ -3126,84 +3170,88 @@ PP(pp_ftrread)
     switch (PL_op->op_type) {
     case OP_FTRREAD:
 #if !(defined(HAS_ACCESS) && defined(R_OK))
-       use_access = 0;
+        use_access = 0;
 #endif
-       break;
+        break;
 
     case OP_FTRWRITE:
 #if defined(HAS_ACCESS) && defined(W_OK)
-       access_mode = W_OK;
+        access_mode = W_OK;
 #else
-       use_access = 0;
+        use_access = 0;
 #endif
-       stat_mode = S_IWUSR;
-       break;
+        stat_mode = S_IWUSR;
+        break;
 
     case OP_FTREXEC:
 #if defined(HAS_ACCESS) && defined(X_OK)
-       access_mode = X_OK;
+        access_mode = X_OK;
 #else
-       use_access = 0;
+        use_access = 0;
 #endif
-       stat_mode = S_IXUSR;
-       break;
+        stat_mode = S_IXUSR;
+        break;
 
     case OP_FTEWRITE:
 #ifdef PERL_EFF_ACCESS
-       access_mode = W_OK;
+        access_mode = W_OK;
 #endif
-       stat_mode = S_IWUSR;
-       /* FALLTHROUGH */
+        stat_mode = S_IWUSR;
+        /* FALLTHROUGH */
 
     case OP_FTEREAD:
 #ifndef PERL_EFF_ACCESS
-       use_access = 0;
+        use_access = 0;
 #endif
-       effective = TRUE;
-       break;
+        effective = TRUE;
+        break;
 
     case OP_FTEEXEC:
 #ifdef PERL_EFF_ACCESS
-       access_mode = X_OK;
+        access_mode = X_OK;
 #else
-       use_access = 0;
+        use_access = 0;
 #endif
-       stat_mode = S_IXUSR;
-       effective = TRUE;
-       break;
+        stat_mode = S_IXUSR;
+        effective = TRUE;
+        break;
     }
 
     if (use_access) {
 #if defined(HAS_ACCESS) || defined (PERL_EFF_ACCESS)
-       const char *name = SvPV_nolen(*PL_stack_sp);
-       if (effective) {
+        STRLEN len;
+        const char *name = SvPV(*PL_stack_sp, len);
+        if (!IS_SAFE_PATHNAME(name, len, OP_NAME(PL_op))) {
+            result = -1;
+        }
+        else if (effective) {
 #  ifdef PERL_EFF_ACCESS
-           result = PERL_EFF_ACCESS(name, access_mode);
+            result = PERL_EFF_ACCESS(name, access_mode);
 #  else
-           DIE(aTHX_ "panic: attempt to call PERL_EFF_ACCESS in %s",
-               OP_NAME(PL_op));
+            DIE(aTHX_ "panic: attempt to call PERL_EFF_ACCESS in %s",
+                OP_NAME(PL_op));
 #  endif
-       }
-       else {
+        }
+        else {
 #  ifdef HAS_ACCESS
-           result = access(name, access_mode);
+            result = access(name, access_mode);
 #  else
-           DIE(aTHX_ "panic: attempt to call access() in %s", OP_NAME(PL_op));
+            DIE(aTHX_ "panic: attempt to call access() in %s", OP_NAME(PL_op));
 #  endif
-       }
-       if (result == 0)
-           FT_RETURNYES;
-       if (result < 0)
-           FT_RETURNUNDEF;
-       FT_RETURNNO;
+        }
+        if (result == 0)
+            FT_RETURNYES;
+        if (result < 0)
+            FT_RETURNUNDEF;
+        FT_RETURNNO;
 #endif
     }
 
     result = my_stat_flags(0);
     if (result < 0)
-       FT_RETURNUNDEF;
+        FT_RETURNUNDEF;
     if (cando(stat_mode, effective, &PL_statcache))
-       FT_RETURNYES;
+        FT_RETURNYES;
     FT_RETURNNO;
 }
 
@@ -3227,36 +3275,36 @@ PP(pp_ftis)
 
     result = my_stat_flags(0);
     if (result < 0)
-       FT_RETURNUNDEF;
+        FT_RETURNUNDEF;
     if (op_type == OP_FTIS)
-       FT_RETURNYES;
+        FT_RETURNYES;
     {
-       /* You can't dTARGET inside OP_FTIS, because you'll get
-          "panic: pad_sv po" - the op is not flagged to have a target.  */
-       dTARGET;
-       switch (op_type) {
-       case OP_FTSIZE:
+        /* You can't dTARGET inside OP_FTIS, because you'll get
+           "panic: pad_sv po" - the op is not flagged to have a target.  */
+        dTARGET;
+        switch (op_type) {
+        case OP_FTSIZE:
 #if Off_t_size > IVSIZE
-           sv_setnv(TARG, (NV)PL_statcache.st_size);
+            sv_setnv(TARG, (NV)PL_statcache.st_size);
 #else
-           sv_setiv(TARG, (IV)PL_statcache.st_size);
-#endif
-           break;
-       case OP_FTMTIME:
-           sv_setnv(TARG,
-                   ((NV)PL_basetime - PL_statcache.st_mtime) / 86400.0 );
-           break;
-       case OP_FTATIME:
-           sv_setnv(TARG,
-                   ((NV)PL_basetime - PL_statcache.st_atime) / 86400.0 );
-           break;
-       case OP_FTCTIME:
-           sv_setnv(TARG,
-                   ((NV)PL_basetime - PL_statcache.st_ctime) / 86400.0 );
-           break;
-       }
-       SvSETMAGIC(TARG);
-       return SvTRUE_nomg(TARG)
+            sv_setiv(TARG, (IV)PL_statcache.st_size);
+#endif
+            break;
+        case OP_FTMTIME:
+            sv_setnv(TARG,
+                    ((NV)PL_basetime - PL_statcache.st_mtime) / 86400.0 );
+            break;
+        case OP_FTATIME:
+            sv_setnv(TARG,
+                    ((NV)PL_basetime - PL_statcache.st_atime) / 86400.0 );
+            break;
+        case OP_FTCTIME:
+            sv_setnv(TARG,
+                    ((NV)PL_basetime - PL_statcache.st_ctime) / 86400.0 );
+            break;
+        }
+        SvSETMAGIC(TARG);
+        return SvTRUE_nomg_NN(TARG)
             ? S_ft_return_true(aTHX_ TARG) : S_ft_return_false(aTHX_ TARG);
     }
 }
@@ -3287,81 +3335,63 @@ PP(pp_ftrowned)
     }
     tryAMAGICftest_MG(opchar);
 
-    /* I believe that all these three are likely to be defined on most every
-       system these days.  */
-#ifndef S_ISUID
-    if(PL_op->op_type == OP_FTSUID) {
-       FT_RETURNNO;
-    }
-#endif
-#ifndef S_ISGID
-    if(PL_op->op_type == OP_FTSGID) {
-       FT_RETURNNO;
-    }
-#endif
-#ifndef S_ISVTX
-    if(PL_op->op_type == OP_FTSVTX) {
-       FT_RETURNNO;
-    }
-#endif
-
     result = my_stat_flags(0);
     if (result < 0)
-       FT_RETURNUNDEF;
+        FT_RETURNUNDEF;
     switch (PL_op->op_type) {
     case OP_FTROWNED:
-       if (PL_statcache.st_uid == PerlProc_getuid())
-           FT_RETURNYES;
-       break;
+        if (PL_statcache.st_uid == PerlProc_getuid())
+            FT_RETURNYES;
+        break;
     case OP_FTEOWNED:
-       if (PL_statcache.st_uid == PerlProc_geteuid())
-           FT_RETURNYES;
-       break;
+        if (PL_statcache.st_uid == PerlProc_geteuid())
+            FT_RETURNYES;
+        break;
     case OP_FTZERO:
-       if (PL_statcache.st_size == 0)
-           FT_RETURNYES;
-       break;
+        if (PL_statcache.st_size == 0)
+            FT_RETURNYES;
+        break;
     case OP_FTSOCK:
-       if (S_ISSOCK(PL_statcache.st_mode))
-           FT_RETURNYES;
-       break;
+        if (S_ISSOCK(PL_statcache.st_mode))
+            FT_RETURNYES;
+        break;
     case OP_FTCHR:
-       if (S_ISCHR(PL_statcache.st_mode))
-           FT_RETURNYES;
-       break;
+        if (S_ISCHR(PL_statcache.st_mode))
+            FT_RETURNYES;
+        break;
     case OP_FTBLK:
-       if (S_ISBLK(PL_statcache.st_mode))
-           FT_RETURNYES;
-       break;
+        if (S_ISBLK(PL_statcache.st_mode))
+            FT_RETURNYES;
+        break;
     case OP_FTFILE:
-       if (S_ISREG(PL_statcache.st_mode))
-           FT_RETURNYES;
-       break;
+        if (S_ISREG(PL_statcache.st_mode))
+            FT_RETURNYES;
+        break;
     case OP_FTDIR:
-       if (S_ISDIR(PL_statcache.st_mode))
-           FT_RETURNYES;
-       break;
+        if (S_ISDIR(PL_statcache.st_mode))
+            FT_RETURNYES;
+        break;
     case OP_FTPIPE:
-       if (S_ISFIFO(PL_statcache.st_mode))
-           FT_RETURNYES;
-       break;
+        if (S_ISFIFO(PL_statcache.st_mode))
+            FT_RETURNYES;
+        break;
 #ifdef S_ISUID
     case OP_FTSUID:
-       if (PL_statcache.st_mode & S_ISUID)
-           FT_RETURNYES;
-       break;
+        if (PL_statcache.st_mode & S_ISUID)
+            FT_RETURNYES;
+        break;
 #endif
 #ifdef S_ISGID
     case OP_FTSGID:
-       if (PL_statcache.st_mode & S_ISGID)
-           FT_RETURNYES;
-       break;
+        if (PL_statcache.st_mode & S_ISGID)
+            FT_RETURNYES;
+        break;
 #endif
 #ifdef S_ISVTX
     case OP_FTSVTX:
-       if (PL_statcache.st_mode & S_ISVTX)
-           FT_RETURNYES;
-       break;
+        if (PL_statcache.st_mode & S_ISVTX)
+            FT_RETURNYES;
+        break;
 #endif
     }
     FT_RETURNNO;
@@ -3375,9 +3405,9 @@ PP(pp_ftlink)
     result = my_lstat_flags(0);
 
     if (result < 0)
-       FT_RETURNUNDEF;
+        FT_RETURNUNDEF;
     if (S_ISLNK(PL_statcache.st_mode))
-       FT_RETURNYES;
+        FT_RETURNYES;
     FT_RETURNNO;
 }
 
@@ -3392,27 +3422,27 @@ PP(pp_fttty)
     tryAMAGICftest_MG('t');
 
     if (PL_op->op_flags & OPf_REF)
-       gv = cGVOP_gv;
+        gv = cGVOP_gv;
     else {
       SV *tmpsv = *PL_stack_sp;
       if (!(gv = MAYBE_DEREF_GV_nomg(tmpsv))) {
-       name = SvPV_nomg(tmpsv, namelen);
-       gv = gv_fetchpvn_flags(name, namelen, SvUTF8(tmpsv), SVt_PVIO);
+        name = SvPV_nomg(tmpsv, namelen);
+        gv = gv_fetchpvn_flags(name, namelen, SvUTF8(tmpsv), SVt_PVIO);
       }
     }
 
     if (GvIO(gv) && IoIFP(GvIOp(gv)))
-       fd = PerlIO_fileno(IoIFP(GvIOp(gv)));
+        fd = PerlIO_fileno(IoIFP(GvIOp(gv)));
     else if (name && isDIGIT(*name) && grok_atoUV(name, &uv, NULL) && uv <= PERL_INT_MAX)
         fd = (int)uv;
     else
-       FT_RETURNUNDEF;
+        fd = -1;
     if (fd < 0) {
         SETERRNO(EBADF,RMS_IFI);
-       FT_RETURNUNDEF;
+        FT_RETURNUNDEF;
     }
     if (PerlLIO_isatty(fd))
-       FT_RETURNYES;
+        FT_RETURNYES;
     FT_RETURNNO;
 }
 
@@ -3430,120 +3460,129 @@ PP(pp_fttext)
     SV *sv = NULL;
     GV *gv;
     PerlIO *fp;
+    const U8 * first_variant;
 
     tryAMAGICftest_MG(PL_op->op_type == OP_FTTEXT ? 'T' : 'B');
 
     if (PL_op->op_flags & OPf_REF)
-       gv = cGVOP_gv;
+        gv = cGVOP_gv;
     else if ((PL_op->op_private & (OPpFT_STACKED|OPpFT_AFTER_t))
-            == OPpFT_STACKED)
-       gv = PL_defgv;
+             == OPpFT_STACKED)
+        gv = PL_defgv;
     else {
-       sv = *PL_stack_sp;
-       gv = MAYBE_DEREF_GV_nomg(sv);
+        sv = *PL_stack_sp;
+        gv = MAYBE_DEREF_GV_nomg(sv);
     }
 
     if (gv) {
-       if (gv == PL_defgv) {
-           if (PL_statgv)
-               io = SvTYPE(PL_statgv) == SVt_PVIO
-                   ? (IO *)PL_statgv
-                   : GvIO(PL_statgv);
-           else {
-               goto really_filename;
-           }
-       }
-       else {
-           PL_statgv = gv;
-           sv_setpvs(PL_statname, "");
-           io = GvIO(PL_statgv);
-       }
-       PL_laststatval = -1;
-       PL_laststype = OP_STAT;
-       if (io && IoIFP(io)) {
-           int fd;
-           if (! PerlIO_has_base(IoIFP(io)))
-               DIE(aTHX_ "-T and -B not implemented on filehandles");
-           fd = PerlIO_fileno(IoIFP(io));
-           if (fd < 0) {
+        if (gv == PL_defgv) {
+            if (PL_statgv)
+                io = SvTYPE(PL_statgv) == SVt_PVIO
+                    ? (IO *)PL_statgv
+                    : GvIO(PL_statgv);
+            else {
+                goto really_filename;
+            }
+        }
+        else {
+            PL_statgv = gv;
+            SvPVCLEAR(PL_statname);
+            io = GvIO(PL_statgv);
+        }
+        PL_laststatval = -1;
+        PL_laststype = OP_STAT;
+        if (io && IoIFP(io)) {
+            int fd;
+            if (! PerlIO_has_base(IoIFP(io)))
+                DIE(aTHX_ "-T and -B not implemented on filehandles");
+            fd = PerlIO_fileno(IoIFP(io));
+            if (fd < 0) {
                 SETERRNO(EBADF,RMS_IFI);
-               FT_RETURNUNDEF;
+                FT_RETURNUNDEF;
             }
-           PL_laststatval = PerlLIO_fstat(fd, &PL_statcache);
-           if (PL_laststatval < 0)
-               FT_RETURNUNDEF;
-           if (S_ISDIR(PL_statcache.st_mode)) { /* handle NFS glitch */
-               if (PL_op->op_type == OP_FTTEXT)
-                   FT_RETURNNO;
-               else
-                   FT_RETURNYES;
+            PL_laststatval = PerlLIO_fstat(fd, &PL_statcache);
+            if (PL_laststatval < 0)
+                FT_RETURNUNDEF;
+            if (S_ISDIR(PL_statcache.st_mode)) { /* handle NFS glitch */
+                if (PL_op->op_type == OP_FTTEXT)
+                    FT_RETURNNO;
+                else
+                    FT_RETURNYES;
             }
-           if (PerlIO_get_cnt(IoIFP(io)) <= 0) {
-               i = PerlIO_getc(IoIFP(io));
-               if (i != EOF)
-                   (void)PerlIO_ungetc(IoIFP(io),i);
+            if (PerlIO_get_cnt(IoIFP(io)) <= 0) {
+                i = PerlIO_getc(IoIFP(io));
+                if (i != EOF)
+                    (void)PerlIO_ungetc(IoIFP(io),i);
                 else
                     /* null file is anything */
                     FT_RETURNYES;
-           }
-           len = PerlIO_get_bufsiz(IoIFP(io));
-           s = (STDCHAR *) PerlIO_get_base(IoIFP(io));
-           /* sfio can have large buffers - limit to 512 */
-           if (len > 512)
-               len = 512;
-       }
-       else {
-           SETERRNO(EBADF,RMS_IFI);
-           report_evil_fh(gv);
-           SETERRNO(EBADF,RMS_IFI);
-           FT_RETURNUNDEF;
-       }
+            }
+            len = PerlIO_get_bufsiz(IoIFP(io));
+            s = (STDCHAR *) PerlIO_get_base(IoIFP(io));
+            /* sfio can have large buffers - limit to 512 */
+            if (len > 512)
+                len = 512;
+        }
+        else {
+            SETERRNO(EBADF,RMS_IFI);
+            report_evil_fh(gv);
+            SETERRNO(EBADF,RMS_IFI);
+            FT_RETURNUNDEF;
+        }
     }
     else {
         const char *file;
+        const char *temp;
+        STRLEN temp_len;
         int fd; 
 
         assert(sv);
-       sv_setpv(PL_statname, SvPV_nomg_const_nolen(sv));
+        temp = SvPV_nomg_const(sv, temp_len);
+        sv_setpv(PL_statname, temp);
+        if (!IS_SAFE_PATHNAME(temp, temp_len, OP_NAME(PL_op))) {
+            PL_laststatval = -1;
+            PL_laststype = OP_STAT;
+            FT_RETURNUNDEF;
+        }
       really_filename:
         file = SvPVX_const(PL_statname);
-       PL_statgv = NULL;
-       if (!(fp = PerlIO_open(file, "r"))) {
-           if (!gv) {
-               PL_laststatval = -1;
-               PL_laststype = OP_STAT;
-           }
-           if (ckWARN(WARN_NEWLINE) && should_warn_nl(file)) {
+        PL_statgv = NULL;
+        if (!(fp = PerlIO_open(file, "r"))) {
+            if (!gv) {
+                PL_laststatval = -1;
+                PL_laststype = OP_STAT;
+            }
+            if (ckWARN(WARN_NEWLINE) && should_warn_nl(file)) {
                 /* PL_warn_nl is constant */
-                GCC_DIAG_IGNORE(-Wformat-nonliteral);
-               Perl_warner(aTHX_ packWARN(WARN_NEWLINE), PL_warn_nl, "open");
-                GCC_DIAG_RESTORE;
+                GCC_DIAG_IGNORE_STMT(-Wformat-nonliteral);
+                Perl_warner(aTHX_ packWARN(WARN_NEWLINE), PL_warn_nl, "open");
+                GCC_DIAG_RESTORE_STMT;
             }
-           FT_RETURNUNDEF;
-       }
-       PL_laststype = OP_STAT;
+            FT_RETURNUNDEF;
+        }
+        PL_laststype = OP_STAT;
         fd = PerlIO_fileno(fp);
         if (fd < 0) {
-           (void)PerlIO_close(fp);
+            (void)PerlIO_close(fp);
             SETERRNO(EBADF,RMS_IFI);
-           FT_RETURNUNDEF;
+            FT_RETURNUNDEF;
         }
-       PL_laststatval = PerlLIO_fstat(fd, &PL_statcache);
-       if (PL_laststatval < 0) {
+        PL_laststatval = PerlLIO_fstat(fd, &PL_statcache);
+        if (PL_laststatval < 0)        {
             dSAVE_ERRNO;
-           (void)PerlIO_close(fp);
+            (void)PerlIO_close(fp);
             RESTORE_ERRNO;
-           FT_RETURNUNDEF;
-       }
-       PerlIO_binmode(aTHX_ fp, '<', O_BINARY, NULL);
-       len = PerlIO_read(fp, tbuf, sizeof(tbuf));
-       (void)PerlIO_close(fp);
-       if (len <= 0) {
-           if (S_ISDIR(PL_statcache.st_mode) && PL_op->op_type == OP_FTTEXT)
-               FT_RETURNNO;            /* special case NFS directories */
-           FT_RETURNYES;               /* null file is anything */
-       }
-       s = tbuf;
+            FT_RETURNUNDEF;
+        }
+        PerlIO_binmode(aTHX_ fp, '<', O_BINARY, NULL);
+        len = PerlIO_read(fp, tbuf, sizeof(tbuf));
+        (void)PerlIO_close(fp);
+        if (len <= 0) {
+            if (S_ISDIR(PL_statcache.st_mode) && PL_op->op_type == OP_FTTEXT)
+                FT_RETURNNO;           /* special case NFS directories */
+            FT_RETURNYES;              /* null file is anything */
+        }
+        s = tbuf;
     }
 
     /* now scan s to look for textiness */
@@ -3551,18 +3590,17 @@ PP(pp_fttext)
 #if defined(DOSISH) || defined(USEMYBINMODE)
     /* ignore trailing ^Z on short files */
     if (len && len < (I32)sizeof(tbuf) && tbuf[len-1] == 26)
-       --len;
+        --len;
 #endif
 
     assert(len);
-    if (! is_utf8_invariant_string((U8 *) s, len)) {
-        const U8 *ep;
+    if (! is_utf8_invariant_string_loc((U8 *) s, len, &first_variant)) {
 
         /* Here contains a variant under UTF-8 .  See if the entire string is
-         * UTF-8.  But the buffer may end in a partial character, so consider
-         * it UTF-8 if the first non-UTF8 char is an ending partial */
-        if (is_utf8_string_loc((U8 *) s, len, &ep)
-            || ep + UTF8SKIP(ep)  > (U8 *) (s + len))
+         * UTF-8. */
+        if (is_utf8_fixed_width_buf_flags(first_variant,
+                                          len - ((char *) first_variant - (char *) s),
+                                          0))
         {
             if (PL_op->op_type == OP_FTTEXT) {
                 FT_RETURNYES;
@@ -3577,26 +3615,26 @@ PP(pp_fttext)
      * things that wouldn't be in ASCII text or rich ASCII text.  Count these
      * in 'odd' */
     for (i = 0; i < len; i++, s++) {
-       if (!*s) {                      /* null never allowed in text */
-           odd += len;
-           break;
-       }
+        if (!*s) {                     /* null never allowed in text */
+            odd += len;
+            break;
+        }
 #ifdef USE_LOCALE_CTYPE
         if (IN_LC_RUNTIME(LC_CTYPE)) {
             if ( isPRINT_LC(*s) || isSPACE_LC(*s)) {
-               continue;
+                continue;
             }
         }
         else
 #endif
-        if (isPRINT_A(*s)
-                   /* VT occurs so rarely in text, that we consider it odd */
-                || (isSPACE_A(*s) && *s != VT_NATIVE)
+             if (  isPRINT_A(*s)
+                    /* VT occurs so rarely in text, that we consider it odd */
+                 || (isSPACE_A(*s) && *s != VT_NATIVE)
 
                     /* But there is a fair amount of backspaces and escapes in
                      * some text */
-                || *s == '\b'
-                || *s == ESC_NATIVE)
+                 || *s == '\b'
+                 || *s == ESC_NATIVE)
         {
             continue;
         }
@@ -3604,9 +3642,9 @@ PP(pp_fttext)
     }
 
     if ((odd * 3 > len) == (PL_op->op_type == OP_FTTEXT)) /* allow 1/3 odd */
-       FT_RETURNNO;
+        FT_RETURNNO;
     else
-       FT_RETURNYES;
+        FT_RETURNYES;
 }
 
 /* File calls. */
@@ -3618,27 +3656,28 @@ PP(pp_chdir)
     GV *gv = NULL;
 
     if( MAXARG == 1 ) {
-       SV * const sv = POPs;
-       if (PL_op->op_flags & OPf_SPECIAL) {
-           gv = gv_fetchsv(sv, 0, SVt_PVIO);
+        SV * const sv = POPs;
+        if (PL_op->op_flags & OPf_SPECIAL) {
+            gv = gv_fetchsv(sv, 0, SVt_PVIO);
             if (!gv) {
                 if (ckWARN(WARN_UNOPENED)) {
                     Perl_warner(aTHX_ packWARN(WARN_UNOPENED),
                                 "chdir() on unopened filehandle %" SVf, sv);
                 }
                 SETERRNO(EBADF,RMS_IFI);
-                PUSHi(0);
+                PUSHs(&PL_sv_zero);
                 TAINT_PROPER("chdir");
                 RETURN;
             }
-       }
+        }
         else if (!(gv = MAYBE_DEREF_GV(sv)))
-               tmps = SvPV_nomg_const_nolen(sv);
+                tmps = SvPV_nomg_const_nolen(sv);
     }
     else {
-       HV * const table = GvHVn(PL_envgv);
-       SV **svp;
+        HV * const table = GvHVn(PL_envgv);
+        SV **svp;
 
+        EXTEND(SP, 1);
         if (    (svp = hv_fetchs(table, "HOME", FALSE))
              || (svp = hv_fetchs(table, "LOGDIR", FALSE))
 #ifdef VMS
@@ -3649,7 +3688,7 @@ PP(pp_chdir)
             tmps = SvPV_nolen_const(*svp);
         }
         else {
-            PUSHi(0);
+            PUSHs(&PL_sv_zero);
             SETERRNO(EINVAL, LIB_INVARG);
             TAINT_PROPER("chdir");
             RETURN;
@@ -3659,26 +3698,26 @@ PP(pp_chdir)
     TAINT_PROPER("chdir");
     if (gv) {
 #ifdef HAS_FCHDIR
-       IO* const io = GvIO(gv);
-       if (io) {
-           if (IoDIRP(io)) {
-               PUSHi(fchdir(my_dirfd(IoDIRP(io))) >= 0);
-           } else if (IoIFP(io)) {
+        IO* const io = GvIO(gv);
+        if (io) {
+            if (IoDIRP(io)) {
+                PUSHi(fchdir(my_dirfd(IoDIRP(io))) >= 0);
+            } else if (IoIFP(io)) {
                 int fd = PerlIO_fileno(IoIFP(io));
                 if (fd < 0) {
                     goto nuts;
                 }
                 PUSHi(fchdir(fd) >= 0);
-           }
-           else {
+            }
+            else {
                 goto nuts;
-           }
+            }
         } else {
             goto nuts;
         }
 
 #else
-       DIE(aTHX_ PL_no_func, "fchdir");
+        DIE(aTHX_ PL_no_func, "fchdir");
 #endif
     }
     else 
@@ -3694,7 +3733,7 @@ PP(pp_chdir)
  nuts:
     report_evil_fh(gv);
     SETERRNO(EBADF,RMS_IFI);
-    PUSHi(0);
+    PUSHs(&PL_sv_zero);
     RETURN;
 #endif
 }
@@ -3739,14 +3778,14 @@ PP(pp_rename)
     anum = PerlLIO_rename(tmps, tmps2);
 #else
     if (!(anum = PerlLIO_stat(tmps, &statbuf))) {
-       if (same_dirent(tmps2, tmps))   /* can always rename to same name */
-           anum = 1;
-       else {
-           if (PerlProc_geteuid() || PerlLIO_stat(tmps2, &statbuf) < 0 || !S_ISDIR(statbuf.st_mode))
-               (void)UNLINK(tmps2);
-           if (!(anum = link(tmps, tmps2)))
-               anum = UNLINK(tmps);
-       }
+        if (same_dirent(tmps2, tmps))  /* can always rename to same name */
+            anum = 1;
+        else {
+            if (PerlProc_geteuid() || PerlLIO_stat(tmps2, &statbuf) < 0 || !S_ISDIR(statbuf.st_mode))
+                (void)UNLINK(tmps2);
+            if (!(anum = link(tmps, tmps2)))
+                anum = UNLINK(tmps);
+        }
     }
 #endif
     SETi( anum >= 0 );
@@ -3765,32 +3804,28 @@ PP(pp_link)
 
 #  ifndef HAS_LINK
     if (op_type == OP_LINK)
-       DIE(aTHX_ PL_no_func, "link");
+        DIE(aTHX_ PL_no_func, "link");
 #  endif
 #  ifndef HAS_SYMLINK
     if (op_type == OP_SYMLINK)
-       DIE(aTHX_ PL_no_func, "symlink");
+        DIE(aTHX_ PL_no_func, "symlink");
 #  endif
 
     {
-       const char * const tmps2 = POPpconstx;
-       const char * const tmps = SvPV_nolen_const(TOPs);
-       TAINT_PROPER(PL_op_desc[op_type]);
-       result =
-#  if defined(HAS_LINK)
-#    if defined(HAS_SYMLINK)
-           /* Both present - need to choose which.  */
-           (op_type == OP_LINK) ?
-           PerlLIO_link(tmps, tmps2) : symlink(tmps, tmps2);
-#    else
+        const char * const tmps2 = POPpconstx;
+        const char * const tmps = SvPV_nolen_const(TOPs);
+        TAINT_PROPER(PL_op_desc[op_type]);
+        result =
+#  if defined(HAS_LINK) && defined(HAS_SYMLINK)
+            /* Both present - need to choose which.  */
+            (op_type == OP_LINK) ?
+            PerlLIO_link(tmps, tmps2) : PerlLIO_symlink(tmps, tmps2);
+#  elif defined(HAS_LINK)
     /* Only have link, so calls to pp_symlink will have DIE()d above.  */
-       PerlLIO_link(tmps, tmps2);
-#    endif
-#  else
-#    if defined(HAS_SYMLINK)
+        PerlLIO_link(tmps, tmps2);
+#  elif defined(HAS_SYMLINK)
     /* Only have symlink, so calls to pp_link will have DIE()d above.  */
-       symlink(tmps, tmps2);
-#    endif
+        PerlLIO_symlink(tmps, tmps2);
 #  endif
     }
 
@@ -3821,11 +3856,10 @@ PP(pp_readlink)
     tmps = POPpconstx;
     /* NOTE: if the length returned by readlink() is sizeof(buf) - 1,
      * it is impossible to know whether the result was truncated. */
-    len = readlink(tmps, buf, sizeof(buf) - 1);
+    len = PerlLIO_readlink(tmps, buf, sizeof(buf) - 1);
     if (len < 0)
-       RETPUSHUNDEF;
-    if (len != -1)
-        buf[len] = '\0';
+        RETPUSHUNDEF;
+    buf[len] = '\0';
     PUSHp(buf, len);
     RETURN;
 #else
@@ -3851,72 +3885,72 @@ S_dooneliner(pTHX_ const char *cmd, const char *filename)
     my_strlcpy(cmdline, cmd, size);
     my_strlcat(cmdline, " ", size);
     for (s = cmdline + strlen(cmdline); *filename; ) {
-       *s++ = '\\';
-       *s++ = *filename++;
+        *s++ = '\\';
+        *s++ = *filename++;
     }
     if (s - cmdline < size)
-       my_strlcpy(s, " 2>&1", size - (s - cmdline));
+        my_strlcpy(s, " 2>&1", size - (s - cmdline));
     myfp = PerlProc_popen(cmdline, "r");
     Safefree(cmdline);
 
     if (myfp) {
-       SV * const tmpsv = sv_newmortal();
-       /* Need to save/restore 'PL_rs' ?? */
-       s = sv_gets(tmpsv, myfp, 0);
-       (void)PerlProc_pclose(myfp);
-       if (s != NULL) {
-           int e;
-           for (e = 1;
+        SV * const tmpsv = sv_newmortal();
+        /* Need to save/restore 'PL_rs' ?? */
+        s = sv_gets(tmpsv, myfp, 0);
+        (void)PerlProc_pclose(myfp);
+        if (s != NULL) {
+            int e;
+            for (e = 1;
 #ifdef HAS_SYS_ERRLIST
-                e <= sys_nerr
-#endif
-                ; e++)
-           {
-               /* you don't see this */
-               const char * const errmsg = Strerror(e) ;
-               if (!errmsg)
-                   break;
-               if (instr(s, errmsg)) {
-                   SETERRNO(e,0);
-                   return 0;
-               }
-           }
-           SETERRNO(0,0);
+                 e <= sys_nerr
+#endif
+                 ; e++)
+            {
+                /* you don't see this */
+                const char * const errmsg = Strerror(e) ;
+                if (!errmsg)
+                    break;
+                if (instr(s, errmsg)) {
+                    SETERRNO(e,0);
+                    return 0;
+                }
+            }
+            SETERRNO(0,0);
 #ifndef EACCES
 #define EACCES EPERM
 #endif
-           if (instr(s, "cannot make"))
-               SETERRNO(EEXIST,RMS_FEX);
-           else if (instr(s, "existing file"))
-               SETERRNO(EEXIST,RMS_FEX);
-           else if (instr(s, "ile exists"))
-               SETERRNO(EEXIST,RMS_FEX);
-           else if (instr(s, "non-exist"))
-               SETERRNO(ENOENT,RMS_FNF);
-           else if (instr(s, "does not exist"))
-               SETERRNO(ENOENT,RMS_FNF);
-           else if (instr(s, "not empty"))
-               SETERRNO(EBUSY,SS_DEVOFFLINE);
-           else if (instr(s, "cannot access"))
-               SETERRNO(EACCES,RMS_PRV);
-           else
-               SETERRNO(EPERM,RMS_PRV);
-           return 0;
-       }
-       else {  /* some mkdirs return no failure indication */
-           Stat_t statbuf;
-           anum = (PerlLIO_stat(save_filename, &statbuf) >= 0);
-           if (PL_op->op_type == OP_RMDIR)
-               anum = !anum;
-           if (anum)
-               SETERRNO(0,0);
-           else
-               SETERRNO(EACCES,RMS_PRV);       /* a guess */
-       }
-       return anum;
+            if (instr(s, "cannot make"))
+                SETERRNO(EEXIST,RMS_FEX);
+            else if (instr(s, "existing file"))
+                SETERRNO(EEXIST,RMS_FEX);
+            else if (instr(s, "ile exists"))
+                SETERRNO(EEXIST,RMS_FEX);
+            else if (instr(s, "non-exist"))
+                SETERRNO(ENOENT,RMS_FNF);
+            else if (instr(s, "does not exist"))
+                SETERRNO(ENOENT,RMS_FNF);
+            else if (instr(s, "not empty"))
+                SETERRNO(EBUSY,SS_DEVOFFLINE);
+            else if (instr(s, "cannot access"))
+                SETERRNO(EACCES,RMS_PRV);
+            else
+                SETERRNO(EPERM,RMS_PRV);
+            return 0;
+        }
+        else { /* some mkdirs return no failure indication */
+            Stat_t statbuf;
+            anum = (PerlLIO_stat(save_filename, &statbuf) >= 0);
+            if (PL_op->op_type == OP_RMDIR)
+                anum = !anum;
+            if (anum)
+                SETERRNO(0,0);
+            else
+                SETERRNO(EACCES,RMS_PRV);      /* a guess */
+        }
+        return anum;
     }
     else
-       return 0;
+        return 0;
 }
 #endif
 
@@ -3933,11 +3967,11 @@ S_dooneliner(pTHX_ const char *cmd, const char *filename)
 
 #define TRIMSLASHES(tmps,len,copy) (tmps) = SvPV_const(TOPs, (len)); \
     if ((len) > 1 && (tmps)[(len)-1] == '/') { \
-       do { \
-           (len)--; \
-       } while ((len) > 1 && (tmps)[(len)-1] == '/'); \
-       (tmps) = savepvn((tmps), (len)); \
-       (copy) = TRUE; \
+        do { \
+            (len)--; \
+        } while ((len) > 1 && (tmps)[(len)-1] == '/'); \
+        (tmps) = savepvn((tmps), (len)); \
+        (copy) = TRUE; \
     }
 
 PP(pp_mkdir)
@@ -3963,7 +3997,7 @@ PP(pp_mkdir)
     }
 #endif
     if (copy)
-       Safefree(tmps);
+        Safefree(tmps);
     RETURN;
 }
 
@@ -3982,7 +4016,7 @@ PP(pp_rmdir)
     SETi( dooneliner("rmdir", tmps) );
 #endif
     if (copy)
-       Safefree(tmps);
+        Safefree(tmps);
     RETURN;
 }
 
@@ -3997,18 +4031,17 @@ PP(pp_open_dir)
     IO * const io = GvIOn(gv);
 
     if ((IoIFP(io) || IoOFP(io)))
-       Perl_ck_warner_d(aTHX_ packWARN2(WARN_IO, WARN_DEPRECATED),
-                        "Opening filehandle %"HEKf" also as a directory",
-                            HEKfARG(GvENAME_HEK(gv)) );
+        Perl_croak(aTHX_ "Cannot open %" HEKf " as a dirhandle: it is already open as a filehandle",
+                         HEKfARG(GvENAME_HEK(gv)));
     if (IoDIRP(io))
-       PerlDir_close(IoDIRP(io));
+        PerlDir_close(IoDIRP(io));
     if (!(IoDIRP(io) = PerlDir_open(dirname)))
-       goto nope;
+        goto nope;
 
     RETPUSHYES;
   nope:
     if (!errno)
-       SETERRNO(EBADF,RMS_DIR);
+        SETERRNO(EBADF,RMS_DIR);
     RETPUSHUNDEF;
 #else
     DIE(aTHX_ PL_no_dir_func, "opendir");
@@ -4032,8 +4065,8 @@ PP(pp_readdir)
     IO * const io = GvIOn(gv);
 
     if (!IoDIRP(io)) {
-       Perl_ck_warner(aTHX_ packWARN(WARN_IO),
-                      "readdir() attempted on invalid dirhandle %"HEKf,
+        Perl_ck_warner(aTHX_ packWARN(WARN_IO),
+                       "readdir() attempted on invalid dirhandle %" HEKf,
                             HEKfARG(GvENAME_HEK(gv)));
         goto nope;
     }
@@ -4050,20 +4083,20 @@ PP(pp_readdir)
         if (!(IoFLAGS(io) & IOf_UNTAINT))
             SvTAINTED_on(sv);
         mXPUSHs(sv);
-    } while (gimme == G_ARRAY);
+    } while (gimme == G_LIST);
 
-    if (!dp && gimme != G_ARRAY)
+    if (!dp && gimme != G_LIST)
         RETPUSHUNDEF;
 
     RETURN;
 
   nope:
     if (!errno)
-       SETERRNO(EBADF,RMS_ISI);
-    if (gimme == G_ARRAY)
-       RETURN;
+        SETERRNO(EBADF,RMS_ISI);
+    if (gimme == G_LIST)
+        RETURN;
     else
-       RETPUSHUNDEF;
+        RETPUSHUNDEF;
 #endif
 }
 
@@ -4082,8 +4115,8 @@ PP(pp_telldir)
     IO * const io = GvIOn(gv);
 
     if (!IoDIRP(io)) {
-       Perl_ck_warner(aTHX_ packWARN(WARN_IO),
-                      "telldir() attempted on invalid dirhandle %"HEKf,
+        Perl_ck_warner(aTHX_ packWARN(WARN_IO),
+                       "telldir() attempted on invalid dirhandle %" HEKf,
                             HEKfARG(GvENAME_HEK(gv)));
         goto nope;
     }
@@ -4092,7 +4125,7 @@ PP(pp_telldir)
     RETURN;
   nope:
     if (!errno)
-       SETERRNO(EBADF,RMS_ISI);
+        SETERRNO(EBADF,RMS_ISI);
     RETPUSHUNDEF;
 #else
     DIE(aTHX_ PL_no_dir_func, "telldir");
@@ -4108,8 +4141,8 @@ PP(pp_seekdir)
     IO * const io = GvIOn(gv);
 
     if (!IoDIRP(io)) {
-       Perl_ck_warner(aTHX_ packWARN(WARN_IO),
-                      "seekdir() attempted on invalid dirhandle %"HEKf,
+        Perl_ck_warner(aTHX_ packWARN(WARN_IO),
+                       "seekdir() attempted on invalid dirhandle %" HEKf,
                                 HEKfARG(GvENAME_HEK(gv)));
         goto nope;
     }
@@ -4118,7 +4151,7 @@ PP(pp_seekdir)
     RETPUSHYES;
   nope:
     if (!errno)
-       SETERRNO(EBADF,RMS_ISI);
+        SETERRNO(EBADF,RMS_ISI);
     RETPUSHUNDEF;
 #else
     DIE(aTHX_ PL_no_dir_func, "seekdir");
@@ -4133,16 +4166,16 @@ PP(pp_rewinddir)
     IO * const io = GvIOn(gv);
 
     if (!IoDIRP(io)) {
-       Perl_ck_warner(aTHX_ packWARN(WARN_IO),
-                      "rewinddir() attempted on invalid dirhandle %"HEKf,
+        Perl_ck_warner(aTHX_ packWARN(WARN_IO),
+                       "rewinddir() attempted on invalid dirhandle %" HEKf,
                                 HEKfARG(GvENAME_HEK(gv)));
-       goto nope;
+        goto nope;
     }
     (void)PerlDir_rewind(IoDIRP(io));
     RETPUSHYES;
   nope:
     if (!errno)
-       SETERRNO(EBADF,RMS_ISI);
+        SETERRNO(EBADF,RMS_ISI);
     RETPUSHUNDEF;
 #else
     DIE(aTHX_ PL_no_dir_func, "rewinddir");
@@ -4157,8 +4190,8 @@ PP(pp_closedir)
     IO * const io = GvIOn(gv);
 
     if (!IoDIRP(io)) {
-       Perl_ck_warner(aTHX_ packWARN(WARN_IO),
-                      "closedir() attempted on invalid dirhandle %"HEKf,
+        Perl_ck_warner(aTHX_ packWARN(WARN_IO),
+                       "closedir() attempted on invalid dirhandle %" HEKf,
                                 HEKfARG(GvENAME_HEK(gv)));
         goto nope;
     }
@@ -4166,8 +4199,8 @@ PP(pp_closedir)
     PerlDir_close(IoDIRP(io));
 #else
     if (PerlDir_close(IoDIRP(io)) < 0) {
-       IoDIRP(io) = 0; /* Don't try to close again--coredumps on SysV */
-       goto nope;
+        IoDIRP(io) = 0; /* Don't try to close again--coredumps on SysV */
+        goto nope;
     }
 #endif
     IoDIRP(io) = 0;
@@ -4175,7 +4208,7 @@ PP(pp_closedir)
     RETPUSHYES;
   nope:
     if (!errno)
-       SETERRNO(EBADF,RMS_IFI);
+        SETERRNO(EBADF,RMS_IFI);
     RETPUSHUNDEF;
 #else
     DIE(aTHX_ PL_no_dir_func, "closedir");
@@ -4201,30 +4234,29 @@ PP(pp_fork)
 #endif
     childpid = PerlProc_fork();
     if (childpid == 0) {
-       int sig;
-       PL_sig_pending = 0;
-       if (PL_psig_pend)
-           for (sig = 1; sig < SIG_SIZE; sig++)
-               PL_psig_pend[sig] = 0;
+        int sig;
+        PL_sig_pending = 0;
+        if (PL_psig_pend)
+            for (sig = 1; sig < SIG_SIZE; sig++)
+                PL_psig_pend[sig] = 0;
     }
 #ifdef HAS_SIGPROCMASK
     {
-       dSAVE_ERRNO;
-       sigprocmask(SIG_SETMASK, &oldmask, NULL);
-       RESTORE_ERRNO;
+        dSAVE_ERRNO;
+        sigprocmask(SIG_SETMASK, &oldmask, NULL);
+        RESTORE_ERRNO;
     }
 #endif
     if (childpid < 0)
-       RETPUSHUNDEF;
+        RETPUSHUNDEF;
     if (!childpid) {
 #ifdef PERL_USES_PL_PIDSTATUS
-       hv_clear(PL_pidstatus); /* no kids, so don't wait for 'em */
+        hv_clear(PL_pidstatus);        /* no kids, so don't wait for 'em */
 #endif
     }
     PUSHi(childpid);
     RETURN;
-#else
-#  if (defined(USE_ITHREADS) && defined(PERL_IMPLICIT_SYS)) || defined(__amigaos4__)
+#elif (defined(USE_ITHREADS) && defined(PERL_IMPLICIT_SYS)) || defined(__amigaos4__)
     dSP; dTARGET;
     Pid_t childpid;
 
@@ -4232,12 +4264,11 @@ PP(pp_fork)
     PERL_FLUSHALL_FOR_CHILD;
     childpid = PerlProc_fork();
     if (childpid == -1)
-       RETPUSHUNDEF;
+        RETPUSHUNDEF;
     PUSHi(childpid);
     RETURN;
-#  else
+#else
     DIE(aTHX_ PL_no_func, "fork");
-#  endif
 #endif
 }
 
@@ -4252,9 +4283,9 @@ PP(pp_wait)
         childpid = wait4pid(-1, &argflags, 0);
     else {
         while ((childpid = wait4pid(-1, &argflags, 0)) == -1 &&
-              errno == EINTR) {
-         PERL_ASYNC_CHECK();
-       }
+               errno == EINTR) {
+          PERL_ASYNC_CHECK();
+        }
     }
 #  if defined(USE_ITHREADS) && defined(PERL_IMPLICIT_SYS)
     /* 0 and -1 are both error returns (the former applies to WNOHANG case) */
@@ -4288,9 +4319,9 @@ PP(pp_waitpid)
         result = wait4pid(pid, &argflags, optype);
     else {
         while ((result = wait4pid(pid, &argflags, optype)) == -1 &&
-              errno == EINTR) {
-         PERL_ASYNC_CHECK();
-       }
+               errno == EINTR) {
+          PERL_ASYNC_CHECK();
+        }
     }
 #  if defined(USE_ITHREADS) && defined(PERL_IMPLICIT_SYS)
     /* 0 and -1 are both error returns (the former applies to WNOHANG case) */
@@ -4321,15 +4352,46 @@ PP(pp_system)
     int result;
 # endif
 
+    while (++MARK <= SP) {
+        SV *origsv = *MARK, *copysv;
+        STRLEN len;
+        char *pv;
+        SvGETMAGIC(origsv);
+#if defined(WIN32) || defined(__VMS)
+        /*
+         * Because of a nasty platform-specific variation on the meaning
+         * of arguments to this op, we must preserve numeric arguments
+         * as numeric, not just retain the string value.
+         */
+        if (SvNIOK(origsv) || SvNIOKp(origsv)) {
+            copysv = newSV_type(SVt_PVNV);
+            sv_2mortal(copysv);
+            if (SvPOK(origsv) || SvPOKp(origsv)) {
+                pv = SvPV_nomg(origsv, len);
+                sv_setpvn(copysv, pv, len);
+                SvPOK_off(copysv);
+            }
+            if (SvIOK(origsv) || SvIOKp(origsv))
+                SvIV_set(copysv, SvIVX(origsv));
+            if (SvNOK(origsv) || SvNOKp(origsv))
+                SvNV_set(copysv, SvNVX(origsv));
+            SvFLAGS(copysv) |= SvFLAGS(origsv) &
+                (SVf_IOK|SVf_NOK|SVf_POK|SVp_IOK|SVp_NOK|SVp_POK|
+                    SVf_UTF8|SVf_IVisUV);
+        } else
+#endif
+        {
+            pv = SvPV_nomg(origsv, len);
+            copysv = newSVpvn_flags(pv, len,
+                        (SvFLAGS(origsv) & SVf_UTF8) | SVs_TEMP);
+        }
+        *MARK = copysv;
+    }
+    MARK = ORIGMARK;
+
     if (TAINTING_get) {
-       TAINT_ENV();
-       while (++MARK <= SP) {
-           (void)SvPV_nolen_const(*MARK);      /* stringify for taint check */
-           if (TAINT_get)
-               break;
-       }
-       MARK = ORIGMARK;
-       TAINT_PROPER("system");
+        TAINT_ENV();
+        TAINT_PROPER("system");
     }
     PERL_FLUSHALL_FOR_CHILD;
 #if (defined(HAS_FORK) || defined(__amigaos4__)) && !defined(VMS) && !defined(OS2) || defined(PERL_MICRO)
@@ -4338,17 +4400,17 @@ PP(pp_system)
         struct UserData userdata;
         pthread_t proc;
 #else
-       Pid_t childpid;
+        Pid_t childpid;
 #endif
-       int pp[2];
-       I32 did_pipes = 0;
+        int pp[2];
+        I32 did_pipes = 0;
         bool child_success = FALSE;
 #ifdef HAS_SIGPROCMASK
-       sigset_t newset, oldset;
+        sigset_t newset, oldset;
 #endif
 
-       if (PerlProc_pipe(pp) >= 0)
-           did_pipes = 1;
+        if (PerlProc_pipe_cloexec(pp) >= 0)
+            did_pipes = 1;
 #ifdef __amigaos4__
         amigaos_fork_set_userdata(aTHX_
                                   &userdata,
@@ -4360,75 +4422,73 @@ PP(pp_system)
         child_success = proc > 0;
 #else
 #ifdef HAS_SIGPROCMASK
-       sigemptyset(&newset);
-       sigaddset(&newset, SIGCHLD);
-       sigprocmask(SIG_BLOCK, &newset, &oldset);
-#endif
-       while ((childpid = PerlProc_fork()) == -1) {
-           if (errno != EAGAIN) {
-               value = -1;
-               SP = ORIGMARK;
-               XPUSHi(value);
-               if (did_pipes) {
-                   PerlLIO_close(pp[0]);
-                   PerlLIO_close(pp[1]);
-               }
+        sigemptyset(&newset);
+        sigaddset(&newset, SIGCHLD);
+        sigprocmask(SIG_BLOCK, &newset, &oldset);
+#endif
+        while ((childpid = PerlProc_fork()) == -1) {
+            if (errno != EAGAIN) {
+                value = -1;
+                SP = ORIGMARK;
+                XPUSHi(value);
+                if (did_pipes) {
+                    PerlLIO_close(pp[0]);
+                    PerlLIO_close(pp[1]);
+                }
 #ifdef HAS_SIGPROCMASK
-               sigprocmask(SIG_SETMASK, &oldset, NULL);
+                sigprocmask(SIG_SETMASK, &oldset, NULL);
 #endif
-               RETURN;
-           }
-           sleep(5);
-       }
+                RETURN;
+            }
+            sleep(5);
+        }
         child_success = childpid > 0;
 #endif
-       if (child_success) {
-           Sigsave_t ihand,qhand; /* place to save signals during system() */
-           int status;
+        if (child_success) {
+            Sigsave_t ihand,qhand; /* place to save signals during system() */
+            int status;
 
 #ifndef __amigaos4__
-           if (did_pipes)
-               PerlLIO_close(pp[1]);
+            if (did_pipes)
+                PerlLIO_close(pp[1]);
 #endif
 #ifndef PERL_MICRO
-           rsignal_save(SIGINT,  (Sighandler_t) SIG_IGN, &ihand);
-           rsignal_save(SIGQUIT, (Sighandler_t) SIG_IGN, &qhand);
+            rsignal_save(SIGINT,  (Sighandler_t) SIG_IGN, &ihand);
+            rsignal_save(SIGQUIT, (Sighandler_t) SIG_IGN, &qhand);
 #endif
 #ifdef __amigaos4__
             result = pthread_join(proc, (void **)&status);
 #else
-           do {
-               result = wait4pid(childpid, &status, 0);
-           } while (result == -1 && errno == EINTR);
+            do {
+                result = wait4pid(childpid, &status, 0);
+            } while (result == -1 && errno == EINTR);
 #endif
 #ifndef PERL_MICRO
 #ifdef HAS_SIGPROCMASK
-           sigprocmask(SIG_SETMASK, &oldset, NULL);
-#endif
-           (void)rsignal_restore(SIGINT, &ihand);
-           (void)rsignal_restore(SIGQUIT, &qhand);
-#endif
-           STATUS_NATIVE_CHILD_SET(result == -1 ? -1 : status);
-           do_execfree();      /* free any memory child malloced on fork */
-           SP = ORIGMARK;
-           if (did_pipes) {
-               int errkid;
-               unsigned n = 0;
-               SSize_t n1;
-
-               while (n < sizeof(int)) {
-                   n1 = PerlLIO_read(pp[0],
-                                     (void*)(((char*)&errkid)+n),
-                                     (sizeof(int)) - n);
-                   if (n1 <= 0)
-                       break;
-                   n += n1;
-               }
-               PerlLIO_close(pp[0]);
-               if (n) {                        /* Error */
-                   if (n != sizeof(int))
-                       DIE(aTHX_ "panic: kid popen errno read, n=%u", n);
-                   errno = errkid;             /* Propagate errno from kid */
+            sigprocmask(SIG_SETMASK, &oldset, NULL);
+#endif
+            (void)rsignal_restore(SIGINT, &ihand);
+            (void)rsignal_restore(SIGQUIT, &qhand);
+#endif
+            STATUS_NATIVE_CHILD_SET(result == -1 ? -1 : status);
+            SP = ORIGMARK;
+            if (did_pipes) {
+                int errkid;
+                unsigned n = 0;
+
+                while (n < sizeof(int)) {
+                    const SSize_t n1 = PerlLIO_read(pp[0],
+                                      (void*)(((char*)&errkid)+n),
+                                      (sizeof(int)) - n);
+                    if (n1 <= 0)
+                        break;
+                    n += n1;
+                }
+                PerlLIO_close(pp[0]);
+                if (n) {                       /* Error */
+                    if (n != sizeof(int))
+                        DIE(aTHX_ "panic: kid popen errno read, n=%u", n);
+                    errno = errkid;            /* Propagate errno from kid */
 #ifdef __amigaos4__
                     /* The pipe always has something in it
                      * so n alone is not enough. */
@@ -4437,59 +4497,53 @@ PP(pp_system)
                     {
                         STATUS_NATIVE_CHILD_SET(-1);
                     }
-               }
-           }
-           XPUSHi(STATUS_CURRENT);
-           RETURN;
-       }
+                }
+            }
+            XPUSHi(STATUS_CURRENT);
+            RETURN;
+        }
 #ifndef __amigaos4__
 #ifdef HAS_SIGPROCMASK
-       sigprocmask(SIG_SETMASK, &oldset, NULL);
-#endif
-       if (did_pipes) {
-           PerlLIO_close(pp[0]);
-#if defined(HAS_FCNTL) && defined(F_SETFD) && defined(FD_CLOEXEC)
-           if (fcntl(pp[1], F_SETFD, FD_CLOEXEC) < 0)
-                RETPUSHUNDEF;
-#endif
-       }
-       if (PL_op->op_flags & OPf_STACKED) {
-           SV * const really = *++MARK;
-           value = (I32)do_aexec5(really, MARK, SP, pp[1], did_pipes);
-       }
-       else if (SP - MARK != 1)
-           value = (I32)do_aexec5(NULL, MARK, SP, pp[1], did_pipes);
-       else {
-           value = (I32)do_exec3(SvPVx_nolen(sv_mortalcopy(*SP)), pp[1], did_pipes);
-       }
+        sigprocmask(SIG_SETMASK, &oldset, NULL);
+#endif
+        if (did_pipes)
+            PerlLIO_close(pp[0]);
+        if (PL_op->op_flags & OPf_STACKED) {
+            SV * const really = *++MARK;
+            value = (I32)do_aexec5(really, MARK, SP, pp[1], did_pipes);
+        }
+        else if (SP - MARK != 1)
+            value = (I32)do_aexec5(NULL, MARK, SP, pp[1], did_pipes);
+        else {
+            value = (I32)do_exec3(SvPVx_nolen(sv_mortalcopy(*SP)), pp[1], did_pipes);
+        }
 #endif /* __amigaos4__ */
-       PerlProc__exit(-1);
+        PerlProc__exit(-1);
     }
 #else /* ! FORK or VMS or OS/2 */
     PL_statusvalue = 0;
     result = 0;
     if (PL_op->op_flags & OPf_STACKED) {
-       SV * const really = *++MARK;
-#  if defined(WIN32) || defined(OS2) || defined(__SYMBIAN32__) || defined(__VMS)
-       value = (I32)do_aspawn(really, MARK, SP);
+        SV * const really = *++MARK;
+#  if defined(WIN32) || defined(OS2) || defined(__VMS)
+        value = (I32)do_aspawn(really, MARK, SP);
 #  else
-       value = (I32)do_aspawn(really, (void **)MARK, (void **)SP);
+        value = (I32)do_aspawn(really, (void **)MARK, (void **)SP);
 #  endif
     }
     else if (SP - MARK != 1) {
-#  if defined(WIN32) || defined(OS2) || defined(__SYMBIAN32__) || defined(__VMS)
-       value = (I32)do_aspawn(NULL, MARK, SP);
+#  if defined(WIN32) || defined(OS2) || defined(__VMS)
+        value = (I32)do_aspawn(NULL, MARK, SP);
 #  else
-       value = (I32)do_aspawn(NULL, (void **)MARK, (void **)SP);
+        value = (I32)do_aspawn(NULL, (void **)MARK, (void **)SP);
 #  endif
     }
     else {
-       value = (I32)do_spawn(SvPVx_nolen(sv_mortalcopy(*SP)));
+        value = (I32)do_spawn(SvPVx_nolen(sv_mortalcopy(*SP)));
     }
     if (PL_statusvalue == -1)  /* hint that value must be returned as is */
-       result = 1;
+        result = 1;
     STATUS_NATIVE_CHILD_SET(value);
-    do_execfree();
     SP = ORIGMARK;
     XPUSHi(result ? value : STATUS_CURRENT);
 #endif /* !FORK or VMS or OS/2 */
@@ -4503,32 +4557,32 @@ PP(pp_exec)
     I32 value;
 
     if (TAINTING_get) {
-       TAINT_ENV();
-       while (++MARK <= SP) {
-           (void)SvPV_nolen_const(*MARK);      /* stringify for taint check */
-           if (TAINT_get)
-               break;
-       }
-       MARK = ORIGMARK;
-       TAINT_PROPER("exec");
+        TAINT_ENV();
+        while (++MARK <= SP) {
+            (void)SvPV_nolen_const(*MARK);      /* stringify for taint check */
+            if (TAINT_get)
+                break;
+        }
+        MARK = ORIGMARK;
+        TAINT_PROPER("exec");
     }
 
     PERL_FLUSHALL_FOR_CHILD;
     if (PL_op->op_flags & OPf_STACKED) {
-       SV * const really = *++MARK;
-       value = (I32)do_aexec(really, MARK, SP);
+        SV * const really = *++MARK;
+        value = (I32)do_aexec(really, MARK, SP);
     }
     else if (SP - MARK != 1)
 #ifdef VMS
-       value = (I32)vms_do_aexec(NULL, MARK, SP);
+        value = (I32)vms_do_aexec(NULL, MARK, SP);
 #else
-       value = (I32)do_aexec(NULL, MARK, SP);
+        value = (I32)do_aexec(NULL, MARK, SP);
 #endif
     else {
 #ifdef VMS
-       value = (I32)vms_do_exec(SvPVx_nolen(sv_mortalcopy(*SP)));
+        value = (I32)vms_do_exec(SvPVx_nolen(sv_mortalcopy(*SP)));
 #else
-       value = (I32)do_exec(SvPVx_nolen(sv_mortalcopy(*SP)));
+        value = (I32)do_exec(SvPVx_nolen(sv_mortalcopy(*SP)));
 #endif
     }
     SP = ORIGMARK;
@@ -4553,13 +4607,13 @@ PP(pp_getpgrp)
     dSP; dTARGET;
     Pid_t pgrp;
     const Pid_t pid =
-       (MAXARG < 1) ? 0 : TOPs ? SvIVx(POPs) : ((void)POPs, 0);
+        (MAXARG < 1) ? 0 : TOPs ? SvIVx(POPs) : ((void)POPs, 0);
 
 #ifdef BSD_GETPGRP
     pgrp = (I32)BSD_GETPGRP(pid);
 #else
     if (pid != 0 && pid != PerlProc_getpid())
-       DIE(aTHX_ "POSIX getpgrp can't take an argument");
+        DIE(aTHX_ "POSIX getpgrp can't take an argument");
     pgrp = getpgrp();
 #endif
     XPUSHi(pgrp);
@@ -4578,9 +4632,9 @@ PP(pp_setpgrp)
     pgrp = MAXARG == 2 && (TOPs||POPs) ? POPi : 0;
     if (MAXARG > 0) pid = TOPs ? TOPi : 0;
     else {
-       pid = 0;
-       EXTEND(SP,1);
-       SP++;
+        pid = 0;
+        EXTEND(SP,1);
+        SP++;
     }
 
     TAINT_PROPER("setpgrp");
@@ -4588,9 +4642,9 @@ PP(pp_setpgrp)
     SETi( BSD_SETPGRP(pid, pgrp) >= 0 );
 #else
     if ((pgrp != 0 && pgrp != PerlProc_getpid())
-       || (pid != 0 && pid != PerlProc_getpid()))
+        || (pid != 0 && pid != PerlProc_getpid()))
     {
-       DIE(aTHX_ "setpgrp can't take arguments");
+        DIE(aTHX_ "setpgrp can't take arguments");
     }
     SETi( setpgrp() >= 0 );
 #endif /* USE_BSDPGRP */
@@ -4600,6 +4654,11 @@ PP(pp_setpgrp)
 #endif
 }
 
+/*
+ * The glibc headers typedef __priority_which_t to an enum under C, but
+ * under C++, it keeps it as int. -Wc++-compat doesn't know this, so we
+ * need to explicitly cast it to shut up the warning.
+ */
 #if defined(__GLIBC__) && ((__GLIBC__ == 2 && __GLIBC_MINOR__ >= 3) || (__GLIBC__ > 2))
 #  define PRIORITY_WHICH_T(which) (__priority_which_t)which
 #else
@@ -4642,9 +4701,9 @@ PP(pp_time)
 {
     dSP; dTARGET;
 #ifdef BIG_TIME
-    XPUSHn( time(NULL) );
+    XPUSHn( (NV)time(NULL) );
 #else
-    XPUSHi( time(NULL) );
+    XPUSHu( (UV)time(NULL) );
 #endif
     RETURN;
 }
@@ -4659,26 +4718,24 @@ PP(pp_tms)
     (void)PerlProc_times(&timesbuf);
 
     mPUSHn(((NV)timesbuf.tms_utime)/(NV)PL_clocktick);
-    if (GIMME_V == G_ARRAY) {
-       mPUSHn(((NV)timesbuf.tms_stime)/(NV)PL_clocktick);
-       mPUSHn(((NV)timesbuf.tms_cutime)/(NV)PL_clocktick);
-       mPUSHn(((NV)timesbuf.tms_cstime)/(NV)PL_clocktick);
+    if (GIMME_V == G_LIST) {
+        mPUSHn(((NV)timesbuf.tms_stime)/(NV)PL_clocktick);
+        mPUSHn(((NV)timesbuf.tms_cutime)/(NV)PL_clocktick);
+        mPUSHn(((NV)timesbuf.tms_cstime)/(NV)PL_clocktick);
     }
     RETURN;
-#else
-#   ifdef PERL_MICRO
+#elif defined(PERL_MICRO)
     dSP;
     mPUSHn(0.0);
     EXTEND(SP, 4);
-    if (GIMME_V == G_ARRAY) {
-        mPUSHn(0.0);
-        mPUSHn(0.0);
-        mPUSHn(0.0);
+    if (GIMME_V == G_LIST) {
+         mPUSHn(0.0);
+         mPUSHn(0.0);
+         mPUSHn(0.0);
     }
     RETURN;
-#   else
+#else
     DIE(aTHX_ "times not implemented");
-#   endif
 #endif /* HAS_TIMES */
 }
 
@@ -4702,66 +4759,66 @@ PP(pp_gmtime)
     struct TM *err;
     const char *opname = PL_op->op_type == OP_LOCALTIME ? "localtime" : "gmtime";
     static const char * const dayname[] =
-       {"Sun", "Mon", "Tue", "Wed", "Thu", "Fri", "Sat"};
+        {"Sun", "Mon", "Tue", "Wed", "Thu", "Fri", "Sat"};
     static const char * const monname[] =
-       {"Jan", "Feb", "Mar", "Apr", "May", "Jun",
-        "Jul", "Aug", "Sep", "Oct", "Nov", "Dec"};
+        {"Jan", "Feb", "Mar", "Apr", "May", "Jun",
+         "Jul", "Aug", "Sep", "Oct", "Nov", "Dec"};
 
     if (MAXARG < 1 || (!TOPs && ((void)POPs, 1))) {
-       time_t now;
-       (void)time(&now);
-       when = (Time64_T)now;
+        time_t now;
+        (void)time(&now);
+        when = (Time64_T)now;
     }
     else {
-       NV input = Perl_floor(POPn);
-       const bool pl_isnan = Perl_isnan(input);
-       when = (Time64_T)input;
-       if (UNLIKELY(pl_isnan || when != input)) {
-           /* diag_listed_as: gmtime(%f) too large */
-           Perl_ck_warner(aTHX_ packWARN(WARN_OVERFLOW),
-                          "%s(%.0" NVff ") too large", opname, input);
-           if (pl_isnan) {
-               err = NULL;
-               goto failed;
-           }
-       }
+        NV input = Perl_floor(POPn);
+        const bool pl_isnan = Perl_isnan(input);
+        when = (Time64_T)input;
+        if (UNLIKELY(pl_isnan || when != input)) {
+            /* diag_listed_as: gmtime(%f) too large */
+            Perl_ck_warner(aTHX_ packWARN(WARN_OVERFLOW),
+                           "%s(%.0" NVff ") too large", opname, input);
+            if (pl_isnan) {
+                err = NULL;
+                goto failed;
+            }
+        }
     }
 
     if ( TIME_LOWER_BOUND > when ) {
-       /* diag_listed_as: gmtime(%f) too small */
-       Perl_ck_warner(aTHX_ packWARN(WARN_OVERFLOW),
-                      "%s(%.0" NVff ") too small", opname, when);
-       err = NULL;
+        /* diag_listed_as: gmtime(%f) too small */
+        Perl_ck_warner(aTHX_ packWARN(WARN_OVERFLOW),
+                       "%s(%.0" NVff ") too small", opname, when);
+        err = NULL;
     }
     else if( when > TIME_UPPER_BOUND ) {
-       /* diag_listed_as: gmtime(%f) too small */
-       Perl_ck_warner(aTHX_ packWARN(WARN_OVERFLOW),
-                      "%s(%.0" NVff ") too large", opname, when);
-       err = NULL;
+        /* diag_listed_as: gmtime(%f) too small */
+        Perl_ck_warner(aTHX_ packWARN(WARN_OVERFLOW),
+                       "%s(%.0" NVff ") too large", opname, when);
+        err = NULL;
     }
     else {
-       if (PL_op->op_type == OP_LOCALTIME)
-           err = Perl_localtime64_r(&when, &tmbuf);
-       else
-           err = Perl_gmtime64_r(&when, &tmbuf);
+        if (PL_op->op_type == OP_LOCALTIME)
+            err = Perl_localtime64_r(&when, &tmbuf);
+        else
+            err = Perl_gmtime64_r(&when, &tmbuf);
     }
 
     if (err == NULL) {
-       /* diag_listed_as: gmtime(%f) failed */
-       /* XXX %lld broken for quads */
+        /* diag_listed_as: gmtime(%f) failed */
+        /* XXX %lld broken for quads */
       failed:
-       Perl_ck_warner(aTHX_ packWARN(WARN_OVERFLOW),
-                      "%s(%.0" NVff ") failed", opname, when);
+        Perl_ck_warner(aTHX_ packWARN(WARN_OVERFLOW),
+                       "%s(%.0" NVff ") failed", opname, when);
     }
 
-    if (GIMME_V != G_ARRAY) {  /* scalar context */
+    if (GIMME_V != G_LIST) {   /* scalar context */
         EXTEND(SP, 1);
-       if (err == NULL)
-           RETPUSHUNDEF;
+        if (err == NULL)
+            RETPUSHUNDEF;
        else {
            dTARGET;
            PUSHs(TARG);
-           Perl_sv_setpvf_mg(aTHX_ TARG, "%s %s %2d %02d:%02d:%02d %"IVdf,
+           Perl_sv_setpvf_mg(aTHX_ TARG, "%s %s %2d %02d:%02d:%02d %" IVdf,
                                 dayname[tmbuf.tm_wday],
                                 monname[tmbuf.tm_mon],
                                 tmbuf.tm_mday,
@@ -4772,20 +4829,20 @@ PP(pp_gmtime)
         }
     }
     else {                     /* list context */
-       if ( err == NULL )
-           RETURN;
+        if ( err == NULL )
+            RETURN;
 
         EXTEND(SP, 9);
         EXTEND_MORTAL(9);
         mPUSHi(tmbuf.tm_sec);
-       mPUSHi(tmbuf.tm_min);
-       mPUSHi(tmbuf.tm_hour);
-       mPUSHi(tmbuf.tm_mday);
-       mPUSHi(tmbuf.tm_mon);
-       mPUSHn(tmbuf.tm_year);
-       mPUSHi(tmbuf.tm_wday);
-       mPUSHi(tmbuf.tm_yday);
-       mPUSHi(tmbuf.tm_isdst);
+        mPUSHi(tmbuf.tm_min);
+        mPUSHi(tmbuf.tm_hour);
+        mPUSHi(tmbuf.tm_mday);
+        mPUSHi(tmbuf.tm_mon);
+        mPUSHn(tmbuf.tm_year);
+        mPUSHi(tmbuf.tm_wday);
+        mPUSHi(tmbuf.tm_yday);
+        mPUSHi(tmbuf.tm_isdst);
     }
     RETURN;
 }
@@ -4826,28 +4883,27 @@ PP(pp_alarm)
 PP(pp_sleep)
 {
     dSP; dTARGET;
-    I32 duration;
     Time_t lasttime;
     Time_t when;
 
     (void)time(&lasttime);
     if (MAXARG < 1 || (!TOPs && !POPs))
-       PerlProc_pause();
+        PerlProc_pause();
     else {
-       duration = POPi;
+        const I32 duration = POPi;
         if (duration < 0) {
           /* diag_listed_as: %s() with negative argument */
           Perl_ck_warner_d(aTHX_ packWARN(WARN_MISC),
                            "sleep() with negative argument");
           SETERRNO(EINVAL, LIB_INVARG);
-          XPUSHi(0);
+          XPUSHs(&PL_sv_zero);
           RETURN;
         } else {
           PerlProc_sleep((unsigned int)duration);
         }
     }
     (void)time(&when);
-    XPUSHi(when - lasttime);
+    XPUSHu((UV)(when - lasttime));
     RETURN;
 }
 
@@ -4865,17 +4921,17 @@ PP(pp_shmwrite)
 
     switch (op_type) {
     case OP_MSGSND:
-       value = (I32)(do_msgsnd(MARK, SP) >= 0);
-       break;
+        value = (I32)(do_msgsnd(MARK, SP) >= 0);
+        break;
     case OP_MSGRCV:
-       value = (I32)(do_msgrcv(MARK, SP) >= 0);
-       break;
+        value = (I32)(do_msgrcv(MARK, SP) >= 0);
+        break;
     case OP_SEMOP:
-       value = (I32)(do_semop(MARK, SP) >= 0);
-       break;
+        value = (I32)(do_semop(MARK, SP) >= 0);
+        break;
     default:
-       value = (I32)(do_shmio(op_type, MARK, SP) >= 0);
-       break;
+        value = (I32)(do_shmio(op_type, MARK, SP) >= 0);
+        break;
     }
 
     SP = MARK;
@@ -4897,7 +4953,7 @@ PP(pp_semget)
     const int anum = do_ipcget(PL_op->op_type, MARK, SP);
     SP = MARK;
     if (anum == -1)
-       RETPUSHUNDEF;
+        RETPUSHUNDEF;
     PUSHi(anum);
     RETURN;
 #else
@@ -4914,12 +4970,12 @@ PP(pp_semctl)
     const int anum = do_ipcctl(PL_op->op_type, MARK, SP);
     SP = MARK;
     if (anum == -1)
-       RETPUSHUNDEF;
+        RETPUSHUNDEF;
     if (anum != 0) {
-       PUSHi(anum);
+        PUSHi(anum);
     }
     else {
-       PUSHp(zero_but_true, ZBTLEN);
+        PUSHp(zero_but_true, ZBTLEN);
     }
     RETURN;
 #else
@@ -4935,15 +4991,15 @@ S_space_join_names_mortal(pTHX_ char *const *array)
     SV *target;
 
     if (array && *array) {
-       target = newSVpvs_flags("", SVs_TEMP);
-       while (1) {
-           sv_catpv(target, *array);
-           if (!*++array)
-               break;
-           sv_catpvs(target, " ");
-       }
+        target = newSVpvs_flags("", SVs_TEMP);
+        while (1) {
+            sv_catpv(target, *array);
+            if (!*++array)
+                break;
+            sv_catpvs(target, " ");
+        }
     } else {
-       target = sv_mortalcopy(&PL_sv_no);
+        target = sv_mortalcopy(&PL_sv_no);
     }
     return target;
 }
@@ -4970,70 +5026,70 @@ PP(pp_ghostent)
     EXTEND(SP, 10);
     if (which == OP_GHBYNAME) {
 #ifdef HAS_GETHOSTBYNAME
-       const char* const name = POPpbytex;
-       hent = PerlSock_gethostbyname(name);
+        const char* const name = POPpbytex;
+        hent = PerlSock_gethostbyname(name);
 #else
-       DIE(aTHX_ PL_no_sock_func, "gethostbyname");
+        DIE(aTHX_ PL_no_sock_func, "gethostbyname");
 #endif
     }
     else if (which == OP_GHBYADDR) {
 #ifdef HAS_GETHOSTBYADDR
-       const int addrtype = POPi;
-       SV * const addrsv = POPs;
-       STRLEN addrlen;
-       const char *addr = (char *)SvPVbyte(addrsv, addrlen);
+        const int addrtype = POPi;
+        SV * const addrsv = POPs;
+        STRLEN addrlen;
+        const char *addr = (char *)SvPVbyte(addrsv, addrlen);
 
-       hent = PerlSock_gethostbyaddr(addr, (Netdb_hlen_t) addrlen, addrtype);
+        hent = PerlSock_gethostbyaddr(addr, (Netdb_hlen_t) addrlen, addrtype);
 #else
-       DIE(aTHX_ PL_no_sock_func, "gethostbyaddr");
+        DIE(aTHX_ PL_no_sock_func, "gethostbyaddr");
 #endif
     }
     else
 #ifdef HAS_GETHOSTENT
-       hent = PerlSock_gethostent();
+        hent = PerlSock_gethostent();
 #else
-       DIE(aTHX_ PL_no_sock_func, "gethostent");
+        DIE(aTHX_ PL_no_sock_func, "gethostent");
 #endif
 
 #ifdef HOST_NOT_FOUND
-       if (!hent) {
+        if (!hent) {
 #ifdef USE_REENTRANT_API
 #   ifdef USE_GETHOSTENT_ERRNO
-           h_errno = PL_reentrant_buffer->_gethostent_errno;
+            h_errno = PL_reentrant_buffer->_gethostent_errno;
 #   endif
 #endif
-           STATUS_UNIX_SET(h_errno);
-       }
+            STATUS_UNIX_SET(h_errno);
+        }
 #endif
 
-    if (GIMME_V != G_ARRAY) {
-       PUSHs(sv = sv_newmortal());
-       if (hent) {
-           if (which == OP_GHBYNAME) {
-               if (hent->h_addr)
-                   sv_setpvn(sv, hent->h_addr, hent->h_length);
-           }
-           else
-               sv_setpv(sv, (char*)hent->h_name);
-       }
-       RETURN;
+    if (GIMME_V != G_LIST) {
+        PUSHs(sv = sv_newmortal());
+        if (hent) {
+            if (which == OP_GHBYNAME) {
+                if (hent->h_addr)
+                    sv_setpvn(sv, hent->h_addr, hent->h_length);
+            }
+            else
+                sv_setpv(sv, (char*)hent->h_name);
+        }
+        RETURN;
     }
 
     if (hent) {
-       mPUSHs(newSVpv((char*)hent->h_name, 0));
-       PUSHs(space_join_names_mortal(hent->h_aliases));
-       mPUSHi(hent->h_addrtype);
-       len = hent->h_length;
-       mPUSHi(len);
+        mPUSHs(newSVpv((char*)hent->h_name, 0));
+        PUSHs(space_join_names_mortal(hent->h_aliases));
+        mPUSHi(hent->h_addrtype);
+        len = hent->h_length;
+        mPUSHi(len);
 #ifdef h_addr
-       for (elem = hent->h_addr_list; elem && *elem; elem++) {
-           mXPUSHp(*elem, len);
-       }
+        for (elem = hent->h_addr_list; elem && *elem; elem++) {
+            mXPUSHp(*elem, len);
+        }
 #else
-       if (hent->h_addr)
-           mPUSHp(hent->h_addr, len);
-       else
-           PUSHs(sv_mortalcopy(&PL_sv_no));
+        if (hent->h_addr)
+            mPUSHp(hent->h_addr, len);
+        else
+            PUSHs(sv_mortalcopy(&PL_sv_no));
 #endif /* h_addr */
     }
     RETURN;
@@ -5059,56 +5115,56 @@ PP(pp_gnetent)
 
     if (which == OP_GNBYNAME){
 #ifdef HAS_GETNETBYNAME
-       const char * const name = POPpbytex;
-       nent = PerlSock_getnetbyname(name);
+        const char * const name = POPpbytex;
+        nent = PerlSock_getnetbyname(name);
 #else
         DIE(aTHX_ PL_no_sock_func, "getnetbyname");
 #endif
     }
     else if (which == OP_GNBYADDR) {
 #ifdef HAS_GETNETBYADDR
-       const int addrtype = POPi;
-       const Netdb_net_t addr = (Netdb_net_t) (U32)POPu;
-       nent = PerlSock_getnetbyaddr(addr, addrtype);
+        const int addrtype = POPi;
+        const Netdb_net_t addr = (Netdb_net_t) (U32)POPu;
+        nent = PerlSock_getnetbyaddr(addr, addrtype);
 #else
-       DIE(aTHX_ PL_no_sock_func, "getnetbyaddr");
+        DIE(aTHX_ PL_no_sock_func, "getnetbyaddr");
 #endif
     }
     else
 #ifdef HAS_GETNETENT
-       nent = PerlSock_getnetent();
+        nent = PerlSock_getnetent();
 #else
         DIE(aTHX_ PL_no_sock_func, "getnetent");
 #endif
 
 #ifdef HOST_NOT_FOUND
-       if (!nent) {
+        if (!nent) {
 #ifdef USE_REENTRANT_API
 #   ifdef USE_GETNETENT_ERRNO
-            h_errno = PL_reentrant_buffer->_getnetent_errno;
+             h_errno = PL_reentrant_buffer->_getnetent_errno;
 #   endif
 #endif
-           STATUS_UNIX_SET(h_errno);
-       }
+            STATUS_UNIX_SET(h_errno);
+        }
 #endif
 
     EXTEND(SP, 4);
-    if (GIMME_V != G_ARRAY) {
-       PUSHs(sv = sv_newmortal());
-       if (nent) {
-           if (which == OP_GNBYNAME)
-               sv_setiv(sv, (IV)nent->n_net);
-           else
-               sv_setpv(sv, nent->n_name);
-       }
-       RETURN;
+    if (GIMME_V != G_LIST) {
+        PUSHs(sv = sv_newmortal());
+        if (nent) {
+            if (which == OP_GNBYNAME)
+                sv_setiv(sv, (IV)nent->n_net);
+            else
+                sv_setpv(sv, nent->n_name);
+        }
+        RETURN;
     }
 
     if (nent) {
-       mPUSHs(newSVpv(nent->n_name, 0));
-       PUSHs(space_join_names_mortal(nent->n_aliases));
-       mPUSHi(nent->n_addrtype);
-       mPUSHi(nent->n_net);
+        mPUSHs(newSVpv(nent->n_name, 0));
+        PUSHs(space_join_names_mortal(nent->n_aliases));
+        mPUSHi(nent->n_addrtype);
+        mPUSHi(nent->n_net);
     }
 
     RETURN;
@@ -5135,43 +5191,43 @@ PP(pp_gprotoent)
 
     if (which == OP_GPBYNAME) {
 #ifdef HAS_GETPROTOBYNAME
-       const char* const name = POPpbytex;
-       pent = PerlSock_getprotobyname(name);
+        const char* const name = POPpbytex;
+        pent = PerlSock_getprotobyname(name);
 #else
-       DIE(aTHX_ PL_no_sock_func, "getprotobyname");
+        DIE(aTHX_ PL_no_sock_func, "getprotobyname");
 #endif
     }
     else if (which == OP_GPBYNUMBER) {
 #ifdef HAS_GETPROTOBYNUMBER
-       const int number = POPi;
-       pent = PerlSock_getprotobynumber(number);
+        const int number = POPi;
+        pent = PerlSock_getprotobynumber(number);
 #else
-       DIE(aTHX_ PL_no_sock_func, "getprotobynumber");
+        DIE(aTHX_ PL_no_sock_func, "getprotobynumber");
 #endif
     }
     else
 #ifdef HAS_GETPROTOENT
-       pent = PerlSock_getprotoent();
+        pent = PerlSock_getprotoent();
 #else
-       DIE(aTHX_ PL_no_sock_func, "getprotoent");
+        DIE(aTHX_ PL_no_sock_func, "getprotoent");
 #endif
 
     EXTEND(SP, 3);
-    if (GIMME_V != G_ARRAY) {
-       PUSHs(sv = sv_newmortal());
-       if (pent) {
-           if (which == OP_GPBYNAME)
-               sv_setiv(sv, (IV)pent->p_proto);
-           else
-               sv_setpv(sv, pent->p_name);
-       }
-       RETURN;
+    if (GIMME_V != G_LIST) {
+        PUSHs(sv = sv_newmortal());
+        if (pent) {
+            if (which == OP_GPBYNAME)
+                sv_setiv(sv, (IV)pent->p_proto);
+            else
+                sv_setpv(sv, pent->p_name);
+        }
+        RETURN;
     }
 
     if (pent) {
-       mPUSHs(newSVpv(pent->p_name, 0));
-       PUSHs(space_join_names_mortal(pent->p_aliases));
-       mPUSHi(pent->p_proto);
+        mPUSHs(newSVpv(pent->p_name, 0));
+        PUSHs(space_join_names_mortal(pent->p_aliases));
+        mPUSHi(pent->p_proto);
     }
 
     RETURN;
@@ -5198,48 +5254,48 @@ PP(pp_gservent)
 
     if (which == OP_GSBYNAME) {
 #ifdef HAS_GETSERVBYNAME
-       const char * const proto = POPpbytex;
-       const char * const name = POPpbytex;
-       sent = PerlSock_getservbyname(name, (proto && !*proto) ? NULL : proto);
+        const char * const proto = POPpbytex;
+        const char * const name = POPpbytex;
+        sent = PerlSock_getservbyname(name, (proto && !*proto) ? NULL : proto);
 #else
-       DIE(aTHX_ PL_no_sock_func, "getservbyname");
+        DIE(aTHX_ PL_no_sock_func, "getservbyname");
 #endif
     }
     else if (which == OP_GSBYPORT) {
 #ifdef HAS_GETSERVBYPORT
-       const char * const proto = POPpbytex;
-       unsigned short port = (unsigned short)POPu;
-       port = PerlSock_htons(port);
-       sent = PerlSock_getservbyport(port, (proto && !*proto) ? NULL : proto);
+        const char * const proto = POPpbytex;
+        unsigned short port = (unsigned short)POPu;
+        port = PerlSock_htons(port);
+        sent = PerlSock_getservbyport(port, (proto && !*proto) ? NULL : proto);
 #else
-       DIE(aTHX_ PL_no_sock_func, "getservbyport");
+        DIE(aTHX_ PL_no_sock_func, "getservbyport");
 #endif
     }
     else
 #ifdef HAS_GETSERVENT
-       sent = PerlSock_getservent();
+        sent = PerlSock_getservent();
 #else
-       DIE(aTHX_ PL_no_sock_func, "getservent");
+        DIE(aTHX_ PL_no_sock_func, "getservent");
 #endif
 
     EXTEND(SP, 4);
-    if (GIMME_V != G_ARRAY) {
-       PUSHs(sv = sv_newmortal());
-       if (sent) {
-           if (which == OP_GSBYNAME) {
-               sv_setiv(sv, (IV)PerlSock_ntohs(sent->s_port));
-           }
-           else
-               sv_setpv(sv, sent->s_name);
-       }
-       RETURN;
+    if (GIMME_V != G_LIST) {
+        PUSHs(sv = sv_newmortal());
+        if (sent) {
+            if (which == OP_GSBYNAME) {
+                sv_setiv(sv, (IV)PerlSock_ntohs(sent->s_port));
+            }
+            else
+                sv_setpv(sv, sent->s_name);
+        }
+        RETURN;
     }
 
     if (sent) {
-       mPUSHs(newSVpv(sent->s_name, 0));
-       PUSHs(space_join_names_mortal(sent->s_aliases));
-       mPUSHi(PerlSock_ntohs(sent->s_port));
-       mPUSHs(newSVpv(sent->s_proto, 0));
+        mPUSHs(newSVpv(sent->s_name, 0));
+        PUSHs(space_join_names_mortal(sent->s_aliases));
+        mPUSHi(PerlSock_ntohs(sent->s_port));
+        mPUSHs(newSVpv(sent->s_proto, 0));
     }
 
     RETURN;
@@ -5258,32 +5314,32 @@ PP(pp_shostent)
     switch(PL_op->op_type) {
     case OP_SHOSTENT:
 #ifdef HAS_SETHOSTENT
-       PerlSock_sethostent(stayopen);
+        PerlSock_sethostent(stayopen);
 #else
-       DIE(aTHX_ PL_no_sock_func, PL_op_desc[PL_op->op_type]);
+        DIE(aTHX_ PL_no_sock_func, PL_op_desc[PL_op->op_type]);
 #endif
-       break;
-#ifdef HAS_SETNETENT
+        break;
     case OP_SNETENT:
-       PerlSock_setnetent(stayopen);
+#ifdef HAS_SETNETENT
+        PerlSock_setnetent(stayopen);
 #else
-       DIE(aTHX_ PL_no_sock_func, PL_op_desc[PL_op->op_type]);
+        DIE(aTHX_ PL_no_sock_func, PL_op_desc[PL_op->op_type]);
 #endif
-       break;
+        break;
     case OP_SPROTOENT:
 #ifdef HAS_SETPROTOENT
-       PerlSock_setprotoent(stayopen);
+        PerlSock_setprotoent(stayopen);
 #else
-       DIE(aTHX_ PL_no_sock_func, PL_op_desc[PL_op->op_type]);
+        DIE(aTHX_ PL_no_sock_func, PL_op_desc[PL_op->op_type]);
 #endif
-       break;
+        break;
     case OP_SSERVENT:
 #ifdef HAS_SETSERVENT
-       PerlSock_setservent(stayopen);
+        PerlSock_setservent(stayopen);
 #else
-       DIE(aTHX_ PL_no_sock_func, PL_op_desc[PL_op->op_type]);
+        DIE(aTHX_ PL_no_sock_func, PL_op_desc[PL_op->op_type]);
 #endif
-       break;
+        break;
     }
     RETSETYES;
 }
@@ -5298,60 +5354,60 @@ PP(pp_ehostent)
     switch(PL_op->op_type) {
     case OP_EHOSTENT:
 #ifdef HAS_ENDHOSTENT
-       PerlSock_endhostent();
+        PerlSock_endhostent();
 #else
-       DIE(aTHX_ PL_no_sock_func, PL_op_desc[PL_op->op_type]);
+        DIE(aTHX_ PL_no_sock_func, PL_op_desc[PL_op->op_type]);
 #endif
-       break;
+        break;
     case OP_ENETENT:
 #ifdef HAS_ENDNETENT
-       PerlSock_endnetent();
+        PerlSock_endnetent();
 #else
-       DIE(aTHX_ PL_no_sock_func, PL_op_desc[PL_op->op_type]);
+        DIE(aTHX_ PL_no_sock_func, PL_op_desc[PL_op->op_type]);
 #endif
-       break;
+        break;
     case OP_EPROTOENT:
 #ifdef HAS_ENDPROTOENT
-       PerlSock_endprotoent();
+        PerlSock_endprotoent();
 #else
-       DIE(aTHX_ PL_no_sock_func, PL_op_desc[PL_op->op_type]);
+        DIE(aTHX_ PL_no_sock_func, PL_op_desc[PL_op->op_type]);
 #endif
-       break;
+        break;
     case OP_ESERVENT:
 #ifdef HAS_ENDSERVENT
-       PerlSock_endservent();
+        PerlSock_endservent();
 #else
-       DIE(aTHX_ PL_no_sock_func, PL_op_desc[PL_op->op_type]);
+        DIE(aTHX_ PL_no_sock_func, PL_op_desc[PL_op->op_type]);
 #endif
-       break;
+        break;
     case OP_SGRENT:
 #if defined(HAS_GROUP) && defined(HAS_SETGRENT)
-       setgrent();
+        setgrent();
 #else
-       DIE(aTHX_ PL_no_func, PL_op_desc[PL_op->op_type]);
+        DIE(aTHX_ PL_no_func, PL_op_desc[PL_op->op_type]);
 #endif
-       break;
+        break;
     case OP_EGRENT:
 #if defined(HAS_GROUP) && defined(HAS_ENDGRENT)
-       endgrent();
+        endgrent();
 #else
-       DIE(aTHX_ PL_no_func, PL_op_desc[PL_op->op_type]);
+        DIE(aTHX_ PL_no_func, PL_op_desc[PL_op->op_type]);
 #endif
-       break;
+        break;
     case OP_SPWENT:
 #if defined(HAS_PASSWD) && defined(HAS_SETPWENT)
-       setpwent();
+        setpwent();
 #else
-       DIE(aTHX_ PL_no_func, PL_op_desc[PL_op->op_type]);
+        DIE(aTHX_ PL_no_func, PL_op_desc[PL_op->op_type]);
 #endif
-       break;
+        break;
     case OP_EPWENT:
 #if defined(HAS_PASSWD) && defined(HAS_ENDPWENT)
-       endpwent();
+        endpwent();
 #else
-       DIE(aTHX_ PL_no_func, PL_op_desc[PL_op->op_type]);
+        DIE(aTHX_ PL_no_func, PL_op_desc[PL_op->op_type]);
 #endif
-       break;
+        break;
     }
     EXTEND(SP,1);
     RETPUSHYES;
@@ -5416,7 +5472,7 @@ PP(pp_gpwent)
      * it is only included in special cases.
      *
      * In Digital UNIX/Tru64 if using the getespw*() (which seems to be
-     * be preferred interface, even though also the getprpw*() interface
+     * the preferred interface, even though also the getprpw*() interface
      * is available) one needs to link with -lsecurity -ldb -laud -lm.
      * One also needs to call set_auth_parameters() in main() before
      * doing anything else, whether one is using getespw*() or getprpw*().
@@ -5436,137 +5492,131 @@ PP(pp_gpwent)
     switch (which) {
     case OP_GPWNAM:
       {
-       const char* const name = POPpbytex;
-       pwent  = getpwnam(name);
+        const char* const name = POPpbytex;
+        pwent  = getpwnam(name);
       }
       break;
     case OP_GPWUID:
       {
-       Uid_t uid = POPi;
-       pwent = getpwuid(uid);
+        Uid_t uid = POPi;
+        pwent = getpwuid(uid);
       }
-       break;
+        break;
     case OP_GPWENT:
 #   ifdef HAS_GETPWENT
-       pwent  = getpwent();
+        pwent  = getpwent();
 #ifdef POSIX_BC   /* In some cases pw_passwd has invalid addresses */
-       if (pwent) pwent = getpwnam(pwent->pw_name);
+        if (pwent) pwent = getpwnam(pwent->pw_name);
 #endif
 #   else
-       DIE(aTHX_ PL_no_func, "getpwent");
+        DIE(aTHX_ PL_no_func, "getpwent");
 #   endif
-       break;
+        break;
     }
 
     EXTEND(SP, 10);
-    if (GIMME_V != G_ARRAY) {
-       PUSHs(sv = sv_newmortal());
-       if (pwent) {
-           if (which == OP_GPWNAM)
-               sv_setuid(sv, pwent->pw_uid);
-           else
-               sv_setpv(sv, pwent->pw_name);
-       }
-       RETURN;
+    if (GIMME_V != G_LIST) {
+        PUSHs(sv = sv_newmortal());
+        if (pwent) {
+            if (which == OP_GPWNAM)
+                sv_setuid(sv, pwent->pw_uid);
+            else
+                sv_setpv(sv, pwent->pw_name);
+        }
+        RETURN;
     }
 
     if (pwent) {
-       mPUSHs(newSVpv(pwent->pw_name, 0));
-
-       sv = newSViv(0);
-       mPUSHs(sv);
-       /* If we have getspnam(), we try to dig up the shadow
-        * password.  If we are underprivileged, the shadow
-        * interface will set the errno to EACCES or similar,
-        * and return a null pointer.  If this happens, we will
-        * use the dummy password (usually "*" or "x") from the
-        * standard password database.
-        *
-        * In theory we could skip the shadow call completely
-        * if euid != 0 but in practice we cannot know which
-        * security measures are guarding the shadow databases
-        * on a random platform.
-        *
-        * Resist the urge to use additional shadow interfaces.
-        * Divert the urge to writing an extension instead.
-        *
-        * --jhi */
-       /* Some AIX setups falsely(?) detect some getspnam(), which
-        * has a different API than the Solaris/IRIX one. */
+        mPUSHs(newSVpv(pwent->pw_name, 0));
+
+        sv = newSViv(0);
+        mPUSHs(sv);
+        /* If we have getspnam(), we try to dig up the shadow
+         * password.  If we are underprivileged, the shadow
+         * interface will set the errno to EACCES or similar,
+         * and return a null pointer.  If this happens, we will
+         * use the dummy password (usually "*" or "x") from the
+         * standard password database.
+         *
+         * In theory we could skip the shadow call completely
+         * if euid != 0 but in practice we cannot know which
+         * security measures are guarding the shadow databases
+         * on a random platform.
+         *
+         * Resist the urge to use additional shadow interfaces.
+         * Divert the urge to writing an extension instead.
+         *
+         * --jhi */
+        /* Some AIX setups falsely(?) detect some getspnam(), which
+         * has a different API than the Solaris/IRIX one. */
 #   if defined(HAS_GETSPNAM) && !defined(_AIX)
-       {
-           dSAVE_ERRNO;
-           const struct spwd * const spwent = getspnam(pwent->pw_name);
-                         /* Save and restore errno so that
-                          * underprivileged attempts seem
-                          * to have never made the unsuccessful
-                          * attempt to retrieve the shadow password. */
-           RESTORE_ERRNO;
-           if (spwent && spwent->sp_pwdp)
-               sv_setpv(sv, spwent->sp_pwdp);
-       }
+        {
+            dSAVE_ERRNO;
+            const struct spwd * const spwent = getspnam(pwent->pw_name);
+                          /* Save and restore errno so that
+                           * underprivileged attempts seem
+                           * to have never made the unsuccessful
+                           * attempt to retrieve the shadow password. */
+            RESTORE_ERRNO;
+            if (spwent && spwent->sp_pwdp)
+                sv_setpv(sv, spwent->sp_pwdp);
+        }
 #   endif
 #   ifdef PWPASSWD
-       if (!SvPOK(sv)) /* Use the standard password, then. */
-           sv_setpv(sv, pwent->pw_passwd);
+        if (!SvPOK(sv)) /* Use the standard password, then. */
+            sv_setpv(sv, pwent->pw_passwd);
 #   endif
 
-       /* passwd is tainted because user himself can diddle with it.
-        * admittedly not much and in a very limited way, but nevertheless. */
-       SvTAINTED_on(sv);
+        /* passwd is tainted because user himself can diddle with it.
+         * admittedly not much and in a very limited way, but nevertheless. */
+        SvTAINTED_on(sv);
 
         sv_setuid(PUSHmortal, pwent->pw_uid);
         sv_setgid(PUSHmortal, pwent->pw_gid);
 
-       /* pw_change, pw_quota, and pw_age are mutually exclusive--
-        * because of the poor interface of the Perl getpw*(),
-        * not because there's some standard/convention saying so.
-        * A better interface would have been to return a hash,
-        * but we are accursed by our history, alas. --jhi.  */
+        /* pw_change, pw_quota, and pw_age are mutually exclusive--
+         * because of the poor interface of the Perl getpw*(),
+         * not because there's some standard/convention saying so.
+         * A better interface would have been to return a hash,
+         * but we are accursed by our history, alas. --jhi.  */
 #   ifdef PWCHANGE
-       mPUSHi(pwent->pw_change);
+        mPUSHi(pwent->pw_change);
+#   elif defined(PWQUOTA)
+        mPUSHi(pwent->pw_quota);
+#   elif defined(PWAGE)
+        mPUSHs(newSVpv(pwent->pw_age, 0));
 #   else
-#       ifdef PWQUOTA
-       mPUSHi(pwent->pw_quota);
-#       else
-#           ifdef PWAGE
-       mPUSHs(newSVpv(pwent->pw_age, 0));
-#          else
-       /* I think that you can never get this compiled, but just in case.  */
-       PUSHs(sv_mortalcopy(&PL_sv_no));
-#           endif
-#       endif
+        /* I think that you can never get this compiled, but just in case.  */
+        PUSHs(sv_mortalcopy(&PL_sv_no));
 #   endif
 
-       /* pw_class and pw_comment are mutually exclusive--.
-        * see the above note for pw_change, pw_quota, and pw_age. */
+        /* pw_class and pw_comment are mutually exclusive--.
+         * see the above note for pw_change, pw_quota, and pw_age. */
 #   ifdef PWCLASS
-       mPUSHs(newSVpv(pwent->pw_class, 0));
+        mPUSHs(newSVpv(pwent->pw_class, 0));
+#   elif defined(PWCOMMENT)
+        mPUSHs(newSVpv(pwent->pw_comment, 0));
 #   else
-#       ifdef PWCOMMENT
-       mPUSHs(newSVpv(pwent->pw_comment, 0));
-#      else
-       /* I think that you can never get this compiled, but just in case.  */
-       PUSHs(sv_mortalcopy(&PL_sv_no));
-#       endif
+        /* I think that you can never get this compiled, but just in case.  */
+        PUSHs(sv_mortalcopy(&PL_sv_no));
 #   endif
 
 #   ifdef PWGECOS
-       PUSHs(sv = sv_2mortal(newSVpv(pwent->pw_gecos, 0)));
+        PUSHs(sv = sv_2mortal(newSVpv(pwent->pw_gecos, 0)));
 #   else
-       PUSHs(sv = sv_mortalcopy(&PL_sv_no));
+        PUSHs(sv = sv_mortalcopy(&PL_sv_no));
 #   endif
-       /* pw_gecos is tainted because user himself can diddle with it. */
-       SvTAINTED_on(sv);
+        /* pw_gecos is tainted because user himself can diddle with it. */
+        SvTAINTED_on(sv);
 
-       mPUSHs(newSVpv(pwent->pw_dir, 0));
+        mPUSHs(newSVpv(pwent->pw_dir, 0));
 
-       PUSHs(sv = sv_2mortal(newSVpv(pwent->pw_shell, 0)));
-       /* pw_shell is tainted because user himself can diddle with it. */
-       SvTAINTED_on(sv);
+        PUSHs(sv = sv_2mortal(newSVpv(pwent->pw_shell, 0)));
+        /* pw_shell is tainted because user himself can diddle with it. */
+        SvTAINTED_on(sv);
 
 #   ifdef PWEXPIRE
-       mPUSHi(pwent->pw_expire);
+        mPUSHi(pwent->pw_expire);
 #   endif
     }
     RETURN;
@@ -5586,61 +5636,61 @@ PP(pp_ggrent)
     const struct group *grent;
 
     if (which == OP_GGRNAM) {
-       const char* const name = POPpbytex;
-       grent = (const struct group *)getgrnam(name);
+        const char* const name = POPpbytex;
+        grent = (const struct group *)getgrnam(name);
     }
     else if (which == OP_GGRGID) {
 #if Gid_t_sign == 1
-       const Gid_t gid = POPu;
+        const Gid_t gid = POPu;
 #elif Gid_t_sign == -1
-       const Gid_t gid = POPi;
+        const Gid_t gid = POPi;
 #else
 #  error "Unexpected Gid_t_sign"
 #endif
-       grent = (const struct group *)getgrgid(gid);
+        grent = (const struct group *)getgrgid(gid);
     }
     else
 #ifdef HAS_GETGRENT
-       grent = (struct group *)getgrent();
+        grent = (struct group *)getgrent();
 #else
         DIE(aTHX_ PL_no_func, "getgrent");
 #endif
 
     EXTEND(SP, 4);
-    if (GIMME_V != G_ARRAY) {
-       SV * const sv = sv_newmortal();
+    if (GIMME_V != G_LIST) {
+        SV * const sv = sv_newmortal();
 
-       PUSHs(sv);
-       if (grent) {
-           if (which == OP_GGRNAM)
-               sv_setgid(sv, grent->gr_gid);
-           else
-               sv_setpv(sv, grent->gr_name);
-       }
-       RETURN;
+        PUSHs(sv);
+        if (grent) {
+            if (which == OP_GGRNAM)
+                sv_setgid(sv, grent->gr_gid);
+            else
+                sv_setpv(sv, grent->gr_name);
+        }
+        RETURN;
     }
 
     if (grent) {
-       mPUSHs(newSVpv(grent->gr_name, 0));
+        mPUSHs(newSVpv(grent->gr_name, 0));
 
 #ifdef GRPASSWD
-       mPUSHs(newSVpv(grent->gr_passwd, 0));
+        mPUSHs(newSVpv(grent->gr_passwd, 0));
 #else
-       PUSHs(sv_mortalcopy(&PL_sv_no));
+        PUSHs(sv_mortalcopy(&PL_sv_no));
 #endif
 
         sv_setgid(PUSHmortal, grent->gr_gid);
 
 #if !(defined(_CRAYMPP) && defined(USE_REENTRANT_API))
-       /* In UNICOS/mk (_CRAYMPP) the multithreading
-        * versions (getgrnam_r, getgrgid_r)
-        * seem to return an illegal pointer
-        * as the group members list, gr_mem.
-        * getgrent() doesn't even have a _r version
-        * but the gr_mem is poisonous anyway.
-        * So yes, you cannot get the list of group
-        * members if building multithreaded in UNICOS/mk. */
-       PUSHs(space_join_names_mortal(grent->gr_mem));
+        /* In UNICOS/mk (_CRAYMPP) the multithreading
+         * versions (getgrnam_r, getgrgid_r)
+         * seem to return an illegal pointer
+         * as the group members list, gr_mem.
+         * getgrent() doesn't even have a _r version
+         * but the gr_mem is poisonous anyway.
+         * So yes, you cannot get the list of group
+         * members if building multithreaded in UNICOS/mk. */
+        PUSHs(space_join_names_mortal(grent->gr_mem));
 #endif
     }
 
@@ -5657,7 +5707,7 @@ PP(pp_getlogin)
     char *tmps;
     EXTEND(SP, 1);
     if (!(tmps = PerlProc_getlogin()))
-       RETPUSHUNDEF;
+        RETPUSHUNDEF;
     sv_setpv_mg(TARG, tmps);
     PUSHs(TARG);
     RETURN;
@@ -5678,14 +5728,14 @@ PP(pp_syscall)
     IV retval = -1;
 
     if (TAINTING_get) {
-       while (++MARK <= SP) {
-           if (SvTAINTED(*MARK)) {
-               TAINT;
-               break;
-           }
-       }
-       MARK = ORIGMARK;
-       TAINT_PROPER("syscall");
+        while (++MARK <= SP) {
+            if (SvTAINTED(*MARK)) {
+                TAINT;
+                break;
+            }
+        }
+        MARK = ORIGMARK;
+        TAINT_PROPER("syscall");
     }
 
     /* This probably won't work on machines where sizeof(long) != sizeof(int)
@@ -5693,44 +5743,44 @@ PP(pp_syscall)
      * not likely have syscall implemented either, so who cares?
      */
     while (++MARK <= SP) {
-       if (SvNIOK(*MARK) || !i)
-           a[i++] = SvIV(*MARK);
-       else if (*MARK == &PL_sv_undef)
-           a[i++] = 0;
-       else
-           a[i++] = (unsigned long)SvPV_force_nolen(*MARK);
-       if (i > 15)
-           break;
+        if (SvNIOK(*MARK) || !i)
+            a[i++] = SvIV(*MARK);
+        else if (*MARK == &PL_sv_undef)
+            a[i++] = 0;
+        else
+            a[i++] = (unsigned long)SvPV_force_nolen(*MARK);
+        if (i > 15)
+            break;
     }
     switch (items) {
     default:
-       DIE(aTHX_ "Too many args to syscall");
+        DIE(aTHX_ "Too many args to syscall");
     case 0:
-       DIE(aTHX_ "Too few args to syscall");
+        DIE(aTHX_ "Too few args to syscall");
     case 1:
-       retval = syscall(a[0]);
-       break;
+        retval = syscall(a[0]);
+        break;
     case 2:
-       retval = syscall(a[0],a[1]);
-       break;
+        retval = syscall(a[0],a[1]);
+        break;
     case 3:
-       retval = syscall(a[0],a[1],a[2]);
-       break;
+        retval = syscall(a[0],a[1],a[2]);
+        break;
     case 4:
-       retval = syscall(a[0],a[1],a[2],a[3]);
-       break;
+        retval = syscall(a[0],a[1],a[2],a[3]);
+        break;
     case 5:
-       retval = syscall(a[0],a[1],a[2],a[3],a[4]);
-       break;
+        retval = syscall(a[0],a[1],a[2],a[3],a[4]);
+        break;
     case 6:
-       retval = syscall(a[0],a[1],a[2],a[3],a[4],a[5]);
-       break;
+        retval = syscall(a[0],a[1],a[2],a[3],a[4],a[5]);
+        break;
     case 7:
-       retval = syscall(a[0],a[1],a[2],a[3],a[4],a[5],a[6]);
-       break;
+        retval = syscall(a[0],a[1],a[2],a[3],a[4],a[5],a[6]);
+        break;
     case 8:
-       retval = syscall(a[0],a[1],a[2],a[3],a[4],a[5],a[6],a[7]);
-       break;
+        retval = syscall(a[0],a[1],a[2],a[3],a[4],a[5],a[6],a[7]);
+        break;
     }
     SP = ORIGMARK;
     PUSHi(retval);
@@ -5754,24 +5804,24 @@ fcntl_emulate_flock(int fd, int operation)
 
     switch (operation & ~LOCK_NB) {
     case LOCK_SH:
-       flock.l_type = F_RDLCK;
-       break;
+        flock.l_type = F_RDLCK;
+        break;
     case LOCK_EX:
-       flock.l_type = F_WRLCK;
-       break;
+        flock.l_type = F_WRLCK;
+        break;
     case LOCK_UN:
-       flock.l_type = F_UNLCK;
-       break;
+        flock.l_type = F_UNLCK;
+        break;
     default:
-       errno = EINVAL;
-       return -1;
+        errno = EINVAL;
+        return -1;
     }
     flock.l_whence = SEEK_SET;
     flock.l_start = flock.l_len = (Off_t)0;
 
     res = fcntl(fd, (operation & LOCK_NB) ? F_SETLK : F_SETLKW, &flock);
     if (res == -1 && ((errno == EAGAIN) || (errno == EACCES)))
-       errno = EWOULDBLOCK;
+        errno = EWOULDBLOCK;
     return res;
 }
 
@@ -5817,44 +5867,44 @@ lockf_emulate_flock(int fd, int operation)
     /* flock locks entire file so for lockf we need to do the same     */
     pos = PerlLIO_lseek(fd, (Off_t)0, SEEK_CUR);    /* get pos to restore later */
     if (pos > 0)       /* is seekable and needs to be repositioned     */
-       if (PerlLIO_lseek(fd, (Off_t)0, SEEK_SET) < 0)
-           pos = -1;   /* seek failed, so don't seek back afterwards   */
+        if (PerlLIO_lseek(fd, (Off_t)0, SEEK_SET) < 0)
+            pos = -1;  /* seek failed, so don't seek back afterwards   */
     RESTORE_ERRNO;
 
     switch (operation) {
 
-       /* LOCK_SH - get a shared lock */
-       case LOCK_SH:
-       /* LOCK_EX - get an exclusive lock */
-       case LOCK_EX:
-           i = lockf (fd, F_LOCK, 0);
-           break;
-
-       /* LOCK_SH|LOCK_NB - get a non-blocking shared lock */
-       case LOCK_SH|LOCK_NB:
-       /* LOCK_EX|LOCK_NB - get a non-blocking exclusive lock */
-       case LOCK_EX|LOCK_NB:
-           i = lockf (fd, F_TLOCK, 0);
-           if (i == -1)
-               if ((errno == EAGAIN) || (errno == EACCES))
-                   errno = EWOULDBLOCK;
-           break;
-
-       /* LOCK_UN - unlock (non-blocking is a no-op) */
-       case LOCK_UN:
-       case LOCK_UN|LOCK_NB:
-           i = lockf (fd, F_ULOCK, 0);
-           break;
-
-       /* Default - can't decipher operation */
-       default:
-           i = -1;
-           errno = EINVAL;
-           break;
+        /* LOCK_SH - get a shared lock */
+        case LOCK_SH:
+        /* LOCK_EX - get an exclusive lock */
+        case LOCK_EX:
+            i = lockf (fd, F_LOCK, 0);
+            break;
+
+        /* LOCK_SH|LOCK_NB - get a non-blocking shared lock */
+        case LOCK_SH|LOCK_NB:
+        /* LOCK_EX|LOCK_NB - get a non-blocking exclusive lock */
+        case LOCK_EX|LOCK_NB:
+            i = lockf (fd, F_TLOCK, 0);
+            if (i == -1)
+                if ((errno == EAGAIN) || (errno == EACCES))
+                    errno = EWOULDBLOCK;
+            break;
+
+        /* LOCK_UN - unlock (non-blocking is a no-op) */
+        case LOCK_UN:
+        case LOCK_UN|LOCK_NB:
+            i = lockf (fd, F_ULOCK, 0);
+            break;
+
+        /* Default - can't decipher operation */
+        default:
+            i = -1;
+            errno = EINVAL;
+            break;
     }
 
     if (pos > 0)      /* need to restore position of the handle        */
-       PerlLIO_lseek(fd, pos, SEEK_SET);       /* ignore error here    */
+        PerlLIO_lseek(fd, pos, SEEK_SET);      /* ignore error here    */
 
     return (i);
 }