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.

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

Add dij command and expose dbg->reason in there #2857

Closed
wants to merge 2 commits into from

Conversation

earada
Copy link
Contributor

@earada earada commented Jun 28, 2015

Issue #2849

@radare
Copy link
Collaborator

radare commented Jun 28, 2015

Dbg reason should be a struct. An int is not enough to represent all the info we need there.

On 28 Jun 2015, at 17:49, earada notifications@github.com wrote:

Issue #2849

You can view, comment on, or merge this pull request online at:

#2857

Commit Summary

Add dij command and expose dbg->reason in there
File Changes

M libr/core/cmd_debug.c (44)
Patch Links:

https://github.com/radare/radare2/pull/2857.patch
https://github.com/radare/radare2/pull/2857.diff

Reply to this email directly or view it on GitHub.

@earada
Copy link
Contributor Author

earada commented Jun 28, 2015

Which fields are also saved in dbg right now? Could you provide a "dij" output example?

@radare
Copy link
Collaborator

radare commented Jun 28, 2015

see https://pad.nopcode.org/p/tLM4P3CsEx https://pad.nopcode.org/p/tLM4P3CsEx to follow the discussion on this topic

On 28 Jun 2015, at 17:55, earada notifications@github.com wrote:

Which fields are also saved in dbg right now? Could you provide a "dij" output example?


Reply to this email directly or view it on GitHub #2857 (comment).

@radare
Copy link
Collaborator

radare commented Jul 1, 2015

murged. i'll work on the rdebugreason struct and json when i have some time. this was fine to put in master

@radare radare closed this Jul 1, 2015
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.

None yet

2 participants