On Mon, Jun 28, 2010 at 12:35, Tim Bunce <[email protected]> wrote: > On Mon, Jun 28, 2010 at 10:28:25AM -0600, Alex Hunsaker wrote: >> >> > p.p.s. I'll tweak NYTProf to pre-extend PL_endav enough to reduce the >> > risk of this happening. >> >> I thought we might need to check that sv is not NULL in resolve_sub_to_cv(). > > It gets called with garbage, so checking it's not NULL wouldn't help much.
My thinking was: 1: Some of the times I have triggered it, it gets passed NULL 2: if we extend the array, there might be a larger chance of getting called with NULL instead of a bogus value Yeah, I might just be flat out wrong about #2. Not to mention I've no idea what to do when you get a NULL; that and it *is* a bug with mod_perl. So uh, sorry for badgering you :-) > I offered that example as something you could use to file a bug report > against mod_perl without involving NYTProf. Any behaviour other than > "it works" counts as a useful bug report test case :) Point taken. Sorry for the noise. I do _try_ to not be a pain. :) Thank you for the time you spent looking and adding a workaround for this. [ That goes for you as well, Nicholas. Err well I guess mainly the looking, not so much the workaround part :-) ] -- You've received this message because you are subscribed to the Devel::NYTProf Development User group. Group hosted at: http://groups.google.com/group/develnytprof-dev Project hosted at: http://perl-devel-nytprof.googlecode.com CPAN distribution: http://search.cpan.org/dist/Devel-NYTProf To post, email: [email protected] To unsubscribe, email: [email protected]
