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

Support all output formats for the refs graph (agr) #10078

Merged
merged 1 commit into from May 12, 2018

Conversation

cyanpencil
Copy link
Contributor

@cyanpencil cyanpencil commented May 11, 2018

Now agr supports every output format.

Other minor things:

  • Adjusted aggj json output (sometimes there was a trailing comma)
  • Removed e graph.layout = 1 from agr* (horizontal layout has broken edges... for now, better stick to the normal one)
  • Fixed a bug when changing layout with = in the interactive view that would crash if the graph was a custom one (aggv or aggi)
  • Small refactoring to make some function names more clear
  • Running agr without argument now defaults to using core->offset rather than using offset 0x0 (which is much more intuitive imho)

commit adjust
@radare radare merged commit 4364aa3 into radareorg:master May 12, 2018
@radare
Copy link
Collaborator

radare commented May 12, 2018

Use more descriptive commit messages next time. Not just in here

SakiiR pushed a commit to SakiiR/radare2 that referenced this pull request Jul 1, 2019
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