Page MenuHomeFreeBSD
Differential D28725

ddb: just move cursor when the lexer backs up
ClosedPublic
Actions

Authored by rlibby on Feb 17 2021, 6:56 AM.
Tags
None
Referenced Files
Unknown Object (File)
Sat, Sep 21, 10:55 PM
Unknown Object (File)
Wed, Sep 18, 7:19 AM
Unknown Object (File)
Thu, Sep 12, 10:29 AM
Unknown Object (File)
Thu, Sep 12, 10:29 AM
Unknown Object (File)
Thu, Sep 12, 10:24 AM
Unknown Object (File)
Thu, Sep 12, 10:24 AM
Unknown Object (File)
Thu, Sep 12, 7:48 AM
Unknown Object (File)
Mon, Sep 9, 3:29 AM
View All 60 Files
Subscribers
imp
rgrimes

Details

Reviewers
markj
Commits
rGd8404b7ec36d: ddb: just move cursor when the lexer backs up
Summary

Get rid of db_look_char because it's not compatible with db_get_line().
This fixes the following issue:

db> script lockinfo=show alllocks
db> run lockinfo
db:0:lockinfo> how alllocks
No such command; use "help" to list available commands

Reported by: markj

Test Plan
db> script lockinfo=show alllocks
db> script lockinfo

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 37095
Build 33984: arc lint + arc unit

Event Timeline

rlibby created this revision.Feb 17 2021, 6:56 AM
Herald added a subscriber: imp. · View Herald TranscriptFeb 17 2021, 6:56 AM
rlibby requested review of this revision.Feb 17 2021, 6:56 AM
Harbormaster completed remote builds in B37086: Diff 84068.Feb 17 2021, 6:56 AM
markj accepted this revision.Feb 17 2021, 4:16 PM

Thank you!

sys/ddb/db_lex.c
113

Convert to an ANSI definition while here?

This revision is now accepted and ready to land.Feb 17 2021, 4:16 PM
rlibby updated this revision to Diff 84094.Feb 17 2021, 4:28 PM

markj feedback: ANSI-fy while here

This revision now requires review to proceed.Feb 17 2021, 4:28 PM
Harbormaster completed remote builds in B37095: Diff 84094.Feb 17 2021, 4:28 PM
markj accepted this revision.Feb 17 2021, 4:37 PM
This revision is now accepted and ready to land.Feb 17 2021, 4:37 PM
Closed by commit rGd8404b7ec36d: ddb: just move cursor when the lexer backs up (authored by rlibby). · Explain WhyFeb 25 2021, 12:00 AM
This revision was automatically updated to reflect the committed changes.
rlibby added a commit: rGd8404b7ec36d: ddb: just move cursor when the lexer backs up.

Revision Contents
Changeset List

PathSize
sys/
ddb/
27 lines

Diff 84094

View Options

sys/ddb/db_lex.c

Loading...