Re^5: RFC: Accelerated stepping (p5p)

Thanks,

I am trying to consider corner cases just now. The weak part in my patch is the correct detection of grep/map/sort, I think.

Example:

$c++; $d++; $e++; print 'map';
[download]

would currently not stop four times when single stepping with my modified 'n' command, which is not yet ok.

BTW: Where can I see your patch?

Comment on Re^5: RFC: Accelerated stepping (p5p) Download Code

Replies are listed 'Best First'.
Re^6: RFC: Accelerated stepping (p5p) by tye (Sage) on Sep 04, 2008 at 07:24 UTC
The "proper" way to implement what you are talking about is to base your actions on the parsed opnode tree, which probably means the patch belongs in Perl's own source code (or where-ever the existing 'n' vs 's' behavior difference is already implemented for subroutine calls). As for my patch, it was reported to p5p so should be in their e-mail archive (I believe there are at least two web servers that provide access to archives of p5p e-mails -- one says something like "basic navigation is coming RSN" and has said that for several years and I just find a lack of basic navigation quite frustrating; meanwhile, the other seems to be quite difficult to find a URL to for some bizarre reason or is harder to search or something so people rarely produce URLs to it... or, more to the point, I don't have even close to handy the information I would need to be able to confidently find a reference that would lead me to where my patch is available in even close to the amount of time it even took me to write this rediculously long-winded paragraph. Sorry. You could also try rt://perl, but I suspect that could easily turn into a rather large time sink while easily not turning up the desired results and may not even be in use any more for p5p-filed bugs against Perl.) A quick google for terms I know I used in the e-mail also failed to turn it up. :} Update: My patch, http://rt.perl.org/rt3//Public/Bug/Display.html?id=57016, got applied today, 2008-09-07 (more than a month since the last activity on it, just FYI). Thanks. - tye	[reply]

Replies are listed 'Best First'.

Re^6: RFC: Accelerated stepping (p5p)
by tye (Sage) on Sep 04, 2008 at 07:24 UTC

The "proper" way to implement what you are talking about is to base your actions on the parsed opnode tree, which probably means the patch belongs in Perl's own source code (or where-ever the existing 'n' vs 's' behavior difference is already implemented for subroutine calls).

As for my patch, it was reported to p5p so should be in their e-mail archive (I believe there are at least two web servers that provide access to archives of p5p e-mails -- one says something like "basic navigation is coming RSN" and has said that for several years and I just find a lack of basic navigation quite frustrating; meanwhile, the other seems to be quite difficult to find a URL to for some bizarre reason or is harder to search or something so people rarely produce URLs to it... or, more to the point, I don't have even close to handy the information I would need to be able to confidently find a reference that would lead me to where my patch is available in even close to the amount of time it even took me to write this rediculously long-winded paragraph. Sorry. You could also try rt://perl, but I suspect that could easily turn into a rather large time sink while easily not turning up the desired results and may not even be in use any more for p5p-filed bugs against Perl.) A quick google for terms I know I used in the e-mail also failed to turn it up. :}

Update: My patch, http://rt.perl.org/rt3//Public/Bug/Display.html?id=57016, got applied today, 2008-09-07 (more than a month since the last activity on it, just FYI). Thanks.

- tye

[reply]