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

Overlay-aware backtrace #103

Draft
wants to merge 5 commits into
base: main
Choose a base branch
from
Draft

Overlay-aware backtrace #103

wants to merge 5 commits into from

Conversation

bates64
Copy link
Owner

@bates64 bates64 commented Oct 23, 2024

  • fix PANIC_MSG
  • add global variables to backtrace symbol table
  • use script name in evt infinite loop check
  • address2symbol: only consider symbols in loaded segments

@bates64
Copy link
Owner Author

bates64 commented Oct 23, 2024

Draft PR because there is an issue with object names (I think?). To reproduce add Loop(0) EndLoop to a script and observe that the reported script name is incorrect - in particular it provides the script before it in RAM

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
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@bates64