Skip to content

[WIP] Linenumbers/Addresses#70

Draft
fabianfreyer wants to merge 5 commits into
weggli-rs:mainfrom
fabianfreyer:linenumbers
Draft

[WIP] Linenumbers/Addresses#70
fabianfreyer wants to merge 5 commits into
weggli-rs:mainfrom
fabianfreyer:linenumbers

Conversation

@fabianfreyer
Copy link
Copy Markdown
Contributor

This is a draft (on top of #39 for now, although that's just a demo of flexibility for now) PR for adding outputting of linenumbers.

image

image

While this does break existing workflows, there are two main arguments
for doing this:
1. it is a more a common workflow, that the path is kept constant while
   the query is iterated upon. Having the query as the last positional
   argument makes it easier to go up in history and edit the query.
2. it sets things up to for other interactive input methods for queries.
For now this just decompiles the whole binary indiscriminately, without
prioritising based on query identifiers. But as a proof-of-concept, it
works.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant