Skip to content

Add tip about debug overhead#42

Merged
rocky merged 2 commits intoTrepan-Debuggers:masterfrom
dev590t:feature/documentation
Jun 26, 2022
Merged

Add tip about debug overhead#42
rocky merged 2 commits intoTrepan-Debuggers:masterfrom
dev590t:feature/documentation

Conversation

@dev590t
Copy link
Copy Markdown

@dev590t dev590t commented Jun 26, 2022

No description provided.

Copy link
Copy Markdown
Collaborator

@rocky rocky left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM - there are small changes I'll make down the line, but for now this is good.

I will be out of town for a little more than a week.

@rocky rocky merged commit bdbdff5 into Trepan-Debuggers:master Jun 26, 2022
@dev590t
Copy link
Copy Markdown
Author

dev590t commented Jun 26, 2022

yeah, I close the issue #40

@rocky
Copy link
Copy Markdown
Collaborator

rocky commented Sep 23, 2024

@dev590t Due to work in using this debugger for use in the open source Mathematica project, Mathics3, I have been working on this project recently, and have fixed several problems in the slowness around using "finish", "next", and "continue".

I have also gone over the document to explain the situation better. There is still some slowness compared to pdb. I am not completely sure what exactly is wrong or different. But the code is faster (and has fewer bugs in hangs).

@dev590t
Copy link
Copy Markdown
Author

dev590t commented Sep 23, 2024

nice!

@rocky
Copy link
Copy Markdown
Collaborator

rocky commented Sep 23, 2024

nice!

Nice, but it would be even better to figure out where the additional overhead is still coming from in trepan3k that isn't in pdb. Some callbacks to the debugger occur that it feels like should not be happening.

What's weird is that this code has way more information about the program at its disposal that pdb doesn't have. How to make even better use of it is still a mystery.

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.

3 participants