Grouping nodes - Githubissues

radare commented 9 years ago

Merging two or more nodes from the ascii art graph would be very useful, the idea behind this feature consists in the following:

Add a key to toggle node compactness.
We want to have more than one group
Maybe we can just tag each block with a keyword and assign a random color to every tag to print the block box.
Then add a key to group/ungroup all blocks with the same tag as the currently selected one.
We can automatically add those tags to entry/exit/math/loop/...
Graph should be relayouted after the grouping operation happens.
Grouped nodes can have more than one in and out edges

ret2libc commented 9 years ago

I think we should delay this. I won't be able to do it soon.

radare commented 9 years ago

Moved to 1.0

radare commented 7 years ago

We can use the 'T' and 'F' keys to merge the TRUE and FALSE nodes to the current one. and maybe use U to undo the operation

alvarofe commented 7 years ago

'U' is already use for redo seek

radare commented 7 years ago

F was also taken for the trackflags thing :(

On 19 Dec 2016, at 00:27, Álvaro Felipe Melchor notifications@github.com wrote:

'U' is already use for redo seek

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub, or mute the thread.

radare commented 7 years ago

I have an idea for this and will be easier than i though... but not today.moving to 1.2

XVilka commented 6 years ago

@cyanpencil please check this one too, if you have an idea how to implement this quickly. If not - then I think you already have more than enough.

radare commented 6 years ago

maybe good to have some feedback from @xarkes because that may be good to be supported in Cutter too. We can manually merge basic blocks in the analysis information. so maybe this is a solution, but this will affect more things than just the way we display the graph. So i would prefer to make this thing just happen in visual, and be able to save/restore graph layouts (and merged/grouped) nodes)

ret2libc commented 4 years ago

This issue has been moved from radareorg/radare2 to radareorg/ideas as we are trying to clean our backlog and this issue has probably been created a long while ago. This is an effort to help contributors understand what are the actionable items they can work on, prioritize issues better and help users find active/duplicated issues more easily. If this is not an enhancement/improvement/general idea but a bug, feel free to ask for re-transfer to main repo. Thanks for your understanding and contribution with this issue.

XVilka commented 4 years ago

@karliss you might be interested in this issue long term.

radareorg / ideas

Grouping nodes #41