Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Auxtools update #233

Merged
merged 5 commits into from
Dec 9, 2020
Merged

Auxtools update #233

merged 5 commits into from
Dec 9, 2020

Conversation

willox
Copy link
Contributor

@willox willox commented Nov 29, 2020 •
edited
Loading

This fulfils some auxtool debugger requests you had.
Depends on: willox/auxtools@b3fb2f7.

  1. All connection modes except for BACKGROUND wait for the DAP client to be configured before continuing.
  2. stddef.dm contents are sent to the debug client
  3. disassemble eval command works (with the added benefit of being able to disassemble procs that aren't currently running)

The updated auxtools also does some other stuff you wanted:

  1. src/usr moved to arguments
  2. your PR Fall back to Value(tag, type) if to_string returns nothing willox/auxtools#11
  3. arguments with no formal parameter in a proc that is being debugged should show up

@willox
Copy link
Contributor Author

willox commented Nov 29, 2020 •
edited
Loading

wait i forgot to do linux
k done

@SpaceManiac
Copy link
Owner

SpaceManiac commented Dec 9, 2020 •
edited
Loading

  • it would be good if the #dis output included the original opcodes instead of DebugBreaks
  • #dis output for OutputFormat instruction appears to print nonsense rather than the actual format string
  • Arguments should display always so src/usr show, not just when there's >0 actual arguments

@SpaceManiac SpaceManiac merged commit 00edac1 into SpaceManiac:master Dec 9, 2020
@SpaceManiac SpaceManiac added this to the suite v1.7 milestone Dec 10, 2020
j-awn pushed a commit to j-awn/SpacemanDMM that referenced this pull request Apr 1, 2021
@willox @j-awn
Add #dis, wait-for-configure, other tweaks to Auxtools (SpaceManiac…
dc2db27 
…#233)

This fulfils some auxtool debugger requests you had.

1) All connection modes except for `BACKGROUND` wait for the DAP client 
   to be configured before continuing.
2) stddef.dm contents are sent to the debug client
3) disassemble eval command works (with the added benefit of being able 
   to disassemble procs that aren't currently running)

The updated auxtools also does some other stuff you wanted:

1) src/usr moved to arguments
2) your PR willox/auxtools#11
3) arguments with no formal parameter in a proc that is being debugged 
   should show up
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
suite v1.7
Development

Successfully merging this pull request may close these issues.
2 participants
@willox @SpaceManiac