X-Git-Url: http://git.shadowcat.co.uk/gitweb/gitweb.cgi?a=blobdiff_plain;f=xs-src%2FMouseUtil.xs;h=3ec7d8cf4a4bc7a737ee9f96a7d349855cb1d8a4;hb=952a7aee6593ddddfbcbcc9c9ccd0be318ef998c;hp=5b01e404c0d56e99c0f45b6451c0d2e1954fa2e3;hpb=b3cd4c14b73eb0160d29f7d76828de0060c0b522;p=gitmo%2FMouse.git diff --git a/xs-src/MouseUtil.xs b/xs-src/MouseUtil.xs index 5b01e40..3ec7d8c 100644 --- a/xs-src/MouseUtil.xs +++ b/xs-src/MouseUtil.xs @@ -112,34 +112,39 @@ mouse_throw_error(SV* const metaobject, SV* const data /* not used */, const cha mPUSHs(newSVpvs("depth")); mPUSHi(-1); } - PUTBACK; - - call_method("throw_error", G_VOID); + if(SvOK(metaobject)) { + call_method("throw_error", G_VOID); + } + else { + call_pv("Mouse::Util::throw_error", G_VOID); + } croak("throw_error() did not throw the error (%"SVf")", message); } } -/* workaround RT #69939 */ +/* workaround Perl-RT #69939 */ I32 mouse_call_sv_safe(pTHX_ SV* const sv, I32 const flags) { - assert( (flags & G_EVAL) == 0 ); + I32 count; + ENTER; + /* Don't do SAVETMPS */ - if(!PL_in_eval) { - I32 count; - SAVESPTR(ERRSV); - ERRSV = sv_newmortal(); + SAVESPTR(ERRSV); + ERRSV = sv_newmortal(); - count = Perl_call_sv(aTHX_ sv, flags | G_EVAL); + count = Perl_call_sv(aTHX_ sv, flags | G_EVAL); - if(sv_true(ERRSV)){ - croak(NULL); /* rethrow */ - } - return count; - } - else { - return Perl_call_sv(aTHX_ sv, flags); + if(sv_true(ERRSV)){ + SV* const err = sv_mortalcopy(ERRSV); + LEAVE; + sv_setsv(ERRSV, err); + croak(NULL); /* rethrow */ } + + LEAVE; + + return count; } void @@ -318,7 +323,7 @@ mouse_install_sub(pTHX_ GV* const gv, SV* const code_ref) { if(GvCVu(gv)){ /* delete *slot{gv} to work around "redefine" warning */ SvREFCNT_dec(GvCV(gv)); - GvCV(gv) = NULL; + GvCV_set(gv, NULL); } sv_setsv_mg((SV*)gv, code_ref); /* *gv = $code_ref */ @@ -346,7 +351,7 @@ mouse_install_sub(pTHX_ GV* const gv, SV* const code_ref) { } } - CvGV(cv) = gv; + CvGV_set(cv, gv); CvANON_off(cv); } }