summaryrefslogtreecommitdiff
path: root/mg.c
diff options
context:
space:
mode:
authorAaron Crane <arc@cpan.org>2017-10-13 17:02:25 +0200
committerAaron Crane <arc@cpan.org>2017-10-21 16:53:07 +0100
commit2abc681d27f72f8a4ad94049b1ae16171521dd33 (patch)
tree9c2c7d6279e74f1e1656922958bb3801392672fd /mg.c
parent059fd8e704532aaa45601f3f14cfbecd491672dd (diff)
downloadperl-2abc681d27f72f8a4ad94049b1ae16171521dd33.tar.gz
mg.c: simplify cpp conditionals
Diffstat (limited to 'mg.c')
-rw-r--r--mg.c46
1 files changed, 13 insertions, 33 deletions
diff --git a/mg.c b/mg.c
index 8c0c1b5dcd..a9f50b38f5 100644
--- a/mg.c
+++ b/mg.c
@@ -2808,17 +2808,13 @@ Perl_magic_set(pTHX_ SV *sv, MAGIC *mg)
if (*(mg->mg_ptr+1) == '\0') {
#ifdef VMS
set_vaxc_errno(SvIV(sv));
-#else
-# ifdef WIN32
+#elif defined(WIN32)
SetLastError( SvIV(sv) );
-# else
-# ifdef OS2
+#elif defined(OS2)
os2_setsyserrno(SvIV(sv));
-# else
+#else
/* will anyone ever use this? */
SETERRNO(SvIV(sv), 4);
-# endif
-# endif
#endif
}
else if (strEQ(mg->mg_ptr + 1, "NCODING") && SvOK(sv))
@@ -3088,26 +3084,22 @@ Perl_magic_set(pTHX_ SV *sv, MAGIC *mg)
}
#ifdef HAS_SETRUID
PERL_UNUSED_RESULT(setruid(new_uid));
-#else
-#ifdef HAS_SETREUID
+#elif defined(HAS_SETREUID)
PERL_UNUSED_RESULT(setreuid(new_uid, (Uid_t)-1));
-#else
-#ifdef HAS_SETRESUID
+#elif defined(HAS_SETRESUID)
PERL_UNUSED_RESULT(setresuid(new_uid, (Uid_t)-1, (Uid_t)-1));
#else
if (new_uid == PerlProc_geteuid()) { /* special case $< = $> */
-#ifdef PERL_DARWIN
+# ifdef PERL_DARWIN
/* workaround for Darwin's setuid peculiarity, cf [perl #24122] */
if (new_uid != 0 && PerlProc_getuid() == 0)
PERL_UNUSED_RESULT(PerlProc_setuid(0));
-#endif
+# endif
PERL_UNUSED_RESULT(PerlProc_setuid(new_uid));
} else {
Perl_croak(aTHX_ "setruid() not implemented");
}
#endif
-#endif
-#endif
break;
}
case '>':
@@ -3121,11 +3113,9 @@ Perl_magic_set(pTHX_ SV *sv, MAGIC *mg)
}
#ifdef HAS_SETEUID
PERL_UNUSED_RESULT(seteuid(new_euid));
-#else
-#ifdef HAS_SETREUID
+#elif defined(HAS_SETREUID)
PERL_UNUSED_RESULT(setreuid((Uid_t)-1, new_euid));
-#else
-#ifdef HAS_SETRESUID
+#elif defined(HAS_SETRESUID)
PERL_UNUSED_RESULT(setresuid((Uid_t)-1, new_euid, (Uid_t)-1));
#else
if (new_euid == PerlProc_getuid()) /* special case $> = $< */
@@ -3134,8 +3124,6 @@ Perl_magic_set(pTHX_ SV *sv, MAGIC *mg)
Perl_croak(aTHX_ "seteuid() not implemented");
}
#endif
-#endif
-#endif
break;
}
case '(':
@@ -3149,11 +3137,9 @@ Perl_magic_set(pTHX_ SV *sv, MAGIC *mg)
}
#ifdef HAS_SETRGID
PERL_UNUSED_RESULT(setrgid(new_gid));
-#else
-#ifdef HAS_SETREGID
+#elif defined(HAS_SETREGID)
PERL_UNUSED_RESULT(setregid(new_gid, (Gid_t)-1));
-#else
-#ifdef HAS_SETRESGID
+#elif defined(HAS_SETRESGID)
PERL_UNUSED_RESULT(setresgid(new_gid, (Gid_t)-1, (Gid_t) -1));
#else
if (new_gid == PerlProc_getegid()) /* special case $( = $) */
@@ -3162,8 +3148,6 @@ Perl_magic_set(pTHX_ SV *sv, MAGIC *mg)
Perl_croak(aTHX_ "setrgid() not implemented");
}
#endif
-#endif
-#endif
break;
}
case ')':
@@ -3232,11 +3216,9 @@ Perl_magic_set(pTHX_ SV *sv, MAGIC *mg)
}
#ifdef HAS_SETEGID
PERL_UNUSED_RESULT(setegid(new_egid));
-#else
-#ifdef HAS_SETREGID
+#elif defined(HAS_SETREGID)
PERL_UNUSED_RESULT(setregid((Gid_t)-1, new_egid));
-#else
-#ifdef HAS_SETRESGID
+#elif defined(HAS_SETRESGID)
PERL_UNUSED_RESULT(setresgid((Gid_t)-1, new_egid, (Gid_t)-1));
#else
if (new_egid == PerlProc_getgid()) /* special case $) = $( */
@@ -3245,8 +3227,6 @@ Perl_magic_set(pTHX_ SV *sv, MAGIC *mg)
Perl_croak(aTHX_ "setegid() not implemented");
}
#endif
-#endif
-#endif
break;
}
case ':':