On Wed Aug 14 09:11:42 2013, ikegami@adaelis.com wrote: > On Wed, Aug 14, 2013 at 11:24 AM, bulk88 via RT > <perlbug-followup@perl.org>wrote: > > > On Tue Aug 13 13:46:33 2013, sprout wrote: > > > > > > And I applied that patch, with a slight tweak (with one arg it was > > > leaking ops), as 34b94bc4d. > > > > > > > The patch looks inefficient. Every XSUB call now has a larger overhead. > > > > Why do you think so? Do you think S_ck_dump gets called for every XSUB > call? Isn't it only called for calls Devel::Peek::Dump (and imports > thereof)? > > + CV * const cv = get_cvn_flags("Devel::Peek::Dump", 17, 0); > + cv_set_call_checker(cv, S_ck_dump, (SV *)cv); Whoops, I meant the patch in http://perl5.git.perl.org/perl.git/commit/3455055faace06645b99a6ed63fce90144ab47e1 which claims to be part of this ticket. I didn't compare the SHA1s. -- bulk88 ~ bulk88 at hotmail.com --- via perlbug: queue: perl5 status: resolved https://rt.perl.org:443/rt3/Ticket/Display.html?id=78194Thread Previous | Thread Next