Re: accelerated stepping

--Xananews.1.2.3

Richard Foley wrote:

> On Friday 29 August 2008 19:28:08 Heiko Eifeldt wrote:
> > 
> > To Richard:
> > Afterwards I realized, $DB::single is to be used as a bitmask.
> > So it would be 8 instead of 3, since 4 is already taken.
> > 
> Details, details ;-)
> 
> > The only difference should be the execution of grep/map/sort/...
> > 
> > I either use 's' to do small steps or use 'n' with the intention to
> > do bigger steps. But currently I cannot get this behaviour. It is
> > not exactly what is documented, but this is what my expectation is
> > (silly me).
> > <heretic  mode off>
> > 
> If you mean:
> 
> 1. n <- next step over everything (including grep/map/sort).
> 
> 2. s <- step into everything (including grep/map/sort).
> 
> 3. forget nn and N.
> 
> Then I would think this would be (mostly very) intuitive change, and
> the behaviour (most) people would expect from the debugger, most of
> the time.  You'd have to check for unwarranted side effects of
> course, such that blocks other than single-line grep, map and sort,
> remain unaffected, but otherwise it seems to me to be a good idea.

Ok, here is what I did, patch is against Perl 5.10.0. 
Please review, thanks. My simple tests worked so far.

Greetings, Heiko
-- 


--Xananews.1.2.3
Content-Type: Application/Octet-Stream; name=accelerated_n_perldb.pl.diff
Content-Transfer-Encoding: base64

LS0tIHBlcmw1ZGIucGwub3JnCTIwMDctMTItMTggMTA6NDc6MDcuMDAwMDAwMDAwICswMTAwDQor
KysgcGVybDVkYi5wbAkyMDA4LTA4LTMwIDIyOjE1OjQ2LjkyMTg3NTAwMCArMDIwMA0KQEAgLTI2
NDMsNyArMjY0Myw3IEBADQogZGVtYXJjYXRpb24nIGFib3ZlIHdoaWNoIGNvbW1hbmRzIGNhbiBi
ZSBlbnRlcmVkIGFueXRpbWUsIGFuZCBiZWxvdyB3aGljaA0KIHRoZXkgY2FuJ3QuDQogDQotPWhl
YWQ0IEM8bj4gLSBzaW5nbGUgc3RlcCwgYnV0IGRvbid0IHRyYWNlIGRvd24gaW50byBzdWJzDQor
PWhlYWQ0IEM8bj4gLSBzaW5nbGUgc3RlcCwgYnV0IGRvbid0IHRyYWNlIGRvd24gaW50byBzdWJz
LCBhbmQgbm90IGV2ZW4gaW50byBncmVwL21hcC9zb3J0DQogDQogRG9uZSBieSBzZXR0aW5nIEM8
JHNpbmdsZT4gdG8gMiwgd2hpY2ggZm9yY2VzIHN1YnMgdG8gZXhlY3V0ZSBzdHJhaWdodCB0aHJv
dWdoDQogd2hlbiBlbnRlcmVkIChzZWUgQzxEQjo6c3ViPikuIFdlIGFsc28gc2F2ZSB0aGUgQzxu
PiBjb21tYW5kIGluIEM8JGxhc3RzdGVwPiwNCkBAIC0yNjU1LDkgKzI2NTUsNDEgQEANCiAgICAg
ICAgICAgICAgICAgJGNtZCA9fiAvXm4kLyAmJiBkbyB7DQogICAgICAgICAgICAgICAgICAgICBl
bmRfcmVwb3J0KCksIG5leHQgQ01EIGlmICRmaW5pc2hlZCBhbmQgJGxldmVsIDw9IDE7DQogDQot
ICAgICAgICAgICAgICAgICAgICAjIFNpbmdsZSBzdGVwLCBidXQgZG9uJ3QgZW50ZXIgc3Vicy4N
Ci0gICAgICAgICAgICAgICAgICAgICRzaW5nbGUgPSAyOw0KKyAgICAgICAgICAgICAgICAgICAg
aWYgKCAgICRkYmxpbmVbJGxpbmVdID1+IG17XGJncmVwXGJ9eG1zDQorICAgICAgICAgICAgICAg
ICAgICAgICAgfHwgJGRibGluZVskbGluZV0gPX4gbXtcYm1hcFxifXhtcw0KKyAgICAgICAgICAg
ICAgICAgICAgICAgIHx8ICRkYmxpbmVbJGxpbmVdID1+IG17XGJzb3J0XGJ9eG1zDQorICAgICAg
ICAgICAgICAgICAgICAgICApIHsNCisgICAgICAgICAgICAgICAgICAgICAgICAgICAgIyBTY2Fu
IGZvcndhcmQgdG8gdGhlIG5leHQgZXhlY3V0YWJsZSBsaW5lDQorICAgICAgICAgICAgICAgICAg
ICAgICAgICAgICRpID0gJGxpbmU7DQorICAgICAgICAgICAgICAgICAgICAgICAgICAgICsrJGk7
DQogDQorICAgICAgICAgICAgICAgICAgICAgICAgICAgICRtYXggPSAkI2RibGluZTsNCisgICAg
ICAgICAgICAgICAgICAgICAgICAgICAgKyskaSB3aGlsZSAkZGJsaW5lWyRpXSA9PSAwICYmICRp
IDwgJG1heDsNCisgICAgICAgICAgICAgICAgICAgICAgICAgICAgIyBDaGVjayBpZiB3ZSB3b3Vs
ZCBsZWF2ZSB0aGUgc3ViDQorICAgICAgICAgICAgICAgICAgICAgICAgICAgIG15IEBzdWJsYXN0
ID0gc3BsaXQgbXstfXhtcywgJHN1Ynskc3VifTsNCisgICAgICAgICAgICAgICAgICAgICAgICAg
ICAgaWYgKCRzdWIgZXEgJycgfHwgJGkgPD0gJHN1Ymxhc3RbLTFdKSB7DQorICAgICAgICAgICAg
ICAgICAgICAgICAgICAgICAgICAjIE5leHQgbGluZSBpcyBpbiBjdXJyZW50IHN1Yg0KKw0KKyAg
ICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgIyBCcmVha2FibGU/DQorICAgICAgICAgICAg
ICAgICAgICAgICAgICAgICAgICBpZiAoICRkYmxpbmVbJGldID09IDAgKSB7DQorICAgICAgICAg
ICAgICAgICAgICAgICAgICAgICAgICAgICAgcHJpbnQgJE9VVCAiTGluZSAkaSBub3QgYnJlYWth
YmxlLlxuIjsNCisgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICBuZXh0IENNRDsN
CisgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgIH0NCisNCisgICAgICAgICAgICAgICAg
ICAgICAgICAgICAgICAgICMgU2V0IHVwIHRoZSBvbmUtdGltZS1icmVhayBzaWdpbC4NCisgICAg
ICAgICAgICAgICAgICAgICAgICAgICAgICAgICRkYmxpbmV7JGl9ID1+IHMvKCR8XDApLzs5JDEv
OyAgIyBhZGQgb25lLXRpbWUtb25seSBiLnAuDQorDQorICAgICAgICAgICAgICAgICAgICAgICAg
ICAgICAgICAjIENsZWFyIHNpbmdsZSBzdGVwcGluZyBmbGFncw0KKyAgICAgICAgICAgICAgICAg
ICAgICAgICAgICAgICAgJERCOjpzaW5nbGUgJj0gfjM7CQ0KKyAgICAgICAgICAgICAgICAgICAg
ICAgICAgICB9IGVsc2Ugew0KKyAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgIyBBYm91
dCB0byBsZWF2ZSB0aGUgc3ViLi4uDQorICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAj
IFR1cm4gb24gc3RhY2sgdHJhY2luZyBvbmUgdXAuDQorICAgICAgICAgICAgICAgICAgICAgICAg
ICAgICAgICAkc3RhY2tbIC0xIF0gfD0gMTsNCisgICAgICAgICAgICAgICAgICAgICAgICAgICAg
fQ0KKyAgICAgICAgICAgICAgICAgICAgfSBlbHNlIHsNCisgICAgICAgICAgICAgICAgICAgICAg
ICAjIFNpbmdsZSBzdGVwLCBidXQgZG9uJ3QgZW50ZXIgc3Vicy4NCisgICAgICAgICAgICAgICAg
ICAgICAgICAkc2luZ2xlID0gMjsNCisgICAgICAgICAgICAgICAgICAgIH0NCiAgICAgICAgICAg
ICAgICAgICAgICMgU2F2ZSBmb3IgZW1wdHkgY29tbWFuZCAocmVwZWF0IGxhc3QpLg0KICAgICAg
ICAgICAgICAgICAgICAgJGxhc3RzdGVwID0gJGNtZDsNCiAgICAgICAgICAgICAgICAgICAgIGxh
c3QgQ01EOw0K

--Xananews.1.2.3--