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
2 checks passed
2 checks passed
continuous-integration/appveyor/pr AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
@radare
Copy link
Collaborator

@radare 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
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants