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)
Sun, Jan 18, 6:09 AM
Unknown Object (File)
Sun, Jan 18, 5:23 AM
Unknown Object (File)
Sun, Jan 18, 1:48 AM
Unknown Object (File)
Sat, Jan 17, 3:32 PM
Unknown Object (File)
Thu, Jan 15, 5:45 PM
Unknown Object (File)
Sat, Jan 3, 5:58 PM
Unknown Object (File)
Dec 20 2025, 1:34 AM
Unknown Object (File)
Dec 9 2025, 3:36 PM
View All 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
rG FreeBSD src repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

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 84639

View Options

sys/ddb/db_lex.c

Loading...