X-Git-Url: http://git.vpit.fr/?a=blobdiff_plain;f=Magic.xs;h=38b27dbee18890849f5da63dbe8c02eee4daba76;hb=17a9ff0efce08a362dce5b81091f5be3f345251a;hp=3df5970c6508db8bb94e1e11c8f8e1789dd71ab5;hpb=66e0fff9ba0033da3d616a042d98855eda348f97;p=perl%2Fmodules%2FVariable-Magic.git diff --git a/Magic.xs b/Magic.xs index 3df5970..38b27db 100644 --- a/Magic.xs +++ b/Magic.xs @@ -121,10 +121,19 @@ STATIC SV *vmg_clone(pTHX_ SV *sv, tTHX owner) { #endif /* Applied to dev-5.9 as 25854, integrated to maint-5.8 as 28160 */ -#if !defined(VMG_COMPAT_ARRAY_PUSH_NOLEN) && (VMG_HAS_PERL_MAINT(5, 8, 9, 28160) || VMG_HAS_PERL_MAINT(5, 9, 3, 25854) || VMG_HAS_PERL(5, 10, 0)) -# define VMG_COMPAT_ARRAY_PUSH_NOLEN 1 +#ifndef VMG_COMPAT_ARRAY_PUSH_NOLEN +# if VMG_HAS_PERL_MAINT(5, 8, 9, 28160) || VMG_HAS_PERL_MAINT(5, 9, 3, 25854) || VMG_HAS_PERL(5, 10, 0) +# define VMG_COMPAT_ARRAY_PUSH_NOLEN 1 +# else +# define VMG_COMPAT_ARRAY_PUSH_NOLEN 0 +# endif +#endif + +/* Applied to dev-5.11 as 34908 */ +#if VMG_HAS_PERL_MAINT(5, 11, 0, 34908) +# define VMG_COMPAT_ARRAY_UNSHIFT_NOLEN_VOID 1 #else -# define VMG_COMPAT_ARRAY_PUSH_NOLEN 0 +# define VMG_COMPAT_ARRAY_UNSHIFT_NOLEN_VOID 0 #endif /* Applied to dev-5.9 as 31473 (see #43357), integrated to maint-5.8 as 32542 */ @@ -148,7 +157,6 @@ STATIC void vmg_sv_magicuvar(pTHX_ SV *sv, const char *uf, I32 len) { const MAGIC* mg; sv_magic(sv, NULL, PERL_MAGIC_uvar, uf, len); /* uvar magic has set and get magic, hence this has set SVs_GMG and SVs_SMG. */ - PERL_UNUSED_CONTEXT; if ((mg = SvMAGIC(sv))) { SvRMAGICAL_off(sv); do { @@ -504,12 +512,14 @@ STATIC int vmg_svt_set(pTHX_ SV *sv, MAGIC *mg) { STATIC U32 vmg_svt_len(pTHX_ SV *sv, MAGIC *mg) { SV *svr; - I32 len; + I32 len, has_array; U32 ret; dSP; int count; + has_array = SvTYPE(sv) == SVt_PVAV; + ENTER; SAVETMPS; @@ -517,11 +527,11 @@ STATIC U32 vmg_svt_len(pTHX_ SV *sv, MAGIC *mg) { EXTEND(SP, 3); PUSHs(sv_2mortal(newRV_inc(sv))); PUSHs(mg->mg_obj ? mg->mg_obj : &PL_sv_undef); - if (SvTYPE(sv) == SVt_PVAV) { + if (has_array) { len = av_len((AV *) sv) + 1; mPUSHi(len); } else { - len = 1; + len = 0; PUSHs(&PL_sv_undef); } PUTBACK; @@ -539,7 +549,7 @@ STATIC U32 vmg_svt_len(pTHX_ SV *sv, MAGIC *mg) { FREETMPS; LEAVE; - return ret - 1; + return has_array ? ret - 1 : ret; } STATIC int vmg_svt_clear(pTHX_ SV *sv, MAGIC *mg) { @@ -802,8 +812,8 @@ STATIC SV *vmg_wizard_wiz(pTHX_ SV *wiz) { #if VMG_THREADSAFE #define VMG_CLONE_CB(N) \ - z->cb_ ## N = (w->cb_ ## N) ? newRV_noinc(vmg_clone(SvRV(w->cb_ ## N), \ - w->owner)) \ + z->cb_ ## N = (w->cb_ ## N) ? newRV_inc(vmg_clone(SvRV(w->cb_ ## N), \ + w->owner)) \ : NULL; STATIC MGWIZ *vmg_wizard_clone(pTHX_ const MGWIZ *w) { @@ -868,6 +878,8 @@ BOOT: newCONSTSUB(stash, "VMG_UVAR", newSVuv(VMG_UVAR)); newCONSTSUB(stash, "VMG_COMPAT_ARRAY_PUSH_NOLEN", newSVuv(VMG_COMPAT_ARRAY_PUSH_NOLEN)); + newCONSTSUB(stash, "VMG_COMPAT_ARRAY_UNSHIFT_NOLEN_VOID", + newSVuv(VMG_COMPAT_ARRAY_UNSHIFT_NOLEN_VOID)); newCONSTSUB(stash, "VMG_COMPAT_ARRAY_UNDEF_CLEAR", newSVuv(VMG_COMPAT_ARRAY_UNDEF_CLEAR)); newCONSTSUB(stash, "VMG_COMPAT_SCALAR_LENGTH_NOLEN",