GSOC 2026: Adding Cutter Diff; Basic Function diffing and HexDiffWidget

[NewtronReal]

Your checklist for this pull request

• I've read the guidelines for contributing to this repository
• I made sure to follow the project's coding style
• I've updated the documentation with the relevant information (if needed)
• I've used AI tools to generate fully or partially these code changes and I'm sure the changes are not copyrighted by somebody else.

Detailed description

Summary

This PR introduces a way to expose rz-diff features to Cutter as a new tool Cutter Diff. The Cutter Diff window can be opened by File->Cutter Diff. In the opened window user will be able to diff two new files(Can also diff with current open file at main cutter window). User can set analysis levels and compare logic then can run the bindiff for getting diff thread. It also introduces CutterDiff which will be used anlogouse to CutterCore in the CutterDiff window which exposes the rz-diff features as API. CutterDiff will be creating individual instances for each Cutter Diff window which allows user to diff multiple files to the same opened cutter main bin.

Pending work
Comparing to the main core file is also pending.
LineDiff..

The project was becoming so large that i was afraid reviewer might not be able to review it all at once so i opened this draft.

Added Widgets

CutterDiff // manages two cores initialization resource management and apis
BinDiff // runs as an external thread and provides basic binary diffing result
HexDiff // widget analogous to HexDump with diffing features implemented
DiffLoadWindow
DiffWaitWindow
CutterDiffWindow //The main window for cutter diff can be opened by File->Cutter Diff(should be moved to windows or tools?)
MemoryDiffData // manages the loading and storing of binary data from the core

class DiffFileContext
{
public:
    DiffFile file;
    uint64_t startAddress;
    std::unique_ptr<MemoryDiffData> data;
    QRectF addrArea;
    QRectF itemArea;
    QRectF asciiArea;
};

BasicDiffCursor //unnecessary and will be removed

HexDiffCursor
HexDiffSelection
LineDiffWidget

If added all the widget it will get too long so i listed only the important widgets to be noted.

Test plan (required)
Build Options:
CUTTER_ASAN=ON
CUTTER_USE_BUNDLED_RIZIN=OFF(use https://github.com/NewtronReal/rizin/tree/diffopstringify instead)
Compile and run. Open a file in cutter(an option for opening the Cutter Diff directly is also to be added). Select File A and File B. Select analysis level and compare logic. open the file change the tabs use the functions panel to navigate along the functions(HexDiff)

Screencast_20260620_224325.mp4

[wargio]

rebase on dev

[notxvilka]

Please fix clang-tidy warnings

[NewtronReal]

Think it's ready for review. Waiting for all checks to be green.

[wargio]

what happens when i open 2 big files? do i have a "waiting" dialog ?

[NewtronReal]

what happens when i open 2 big files? do i have a "waiting" dialog ?

Yes you will have a waiting diolog until the function comparison is finished. This is actually not ideal for thos who just want to diff the hexdump so I was thinking of adding some options to choose what all diffing to be done in the initial options menu. Ex for those who just want to view the hexdump diff may be the unckeck the function comparison.

[PremadeS]

Not going into the code details yet.
Here are the issues I found while testing, some of them could be because they aren't implemented yet, in that case this comment can serve as a checklist

1- "Diff files" option/action should be in "Tools" not in "File"

2- There should be an option to set current file as either "File 1" or "File 2"

3- Also, it would be better to allow user to manually type the filepath aswell - for "File 1" and "File 2" - (same as InitialFileDialog)

4- Not sure why "Match Results" have an odd greyish background

• Also better to remove the initial padding that Qt automatically puts in treeview (the space on the left side of entries)

5- "Functions B" shows functions for "File 1" and "Functions A" shows functions for "File 2" for some reason

6- Why name them "File 1" and "File 2" in the initial dialog and then name them "Functions A" and "Functions B"? I think it would be better to name the binary itself instead of "Functions A" if the binary name is too long then you can put and max character limit on it and do string elision based on that (replace last X chars with ...), same with "INFO A" and "INFO B" and any other occurences

7- Function offsets should be in hex not decimal

• Also copying function address through context menu should also copy in hex (just so it's consistent with disassembly)

8- It would be nice if when clicking on a any function entry in "Functions A/B" would switch to Hex Diff and seek to the function offset, same with entries in Match Results, Added and Removed tabs

9- IMO the "Select which files to compare" should be shown when opening "Diff files" through the "tools" menu, seeing and empty widget and clicking "File->Diff new Files" is a lot more work, however "File->Diff new Files" option should be there and it should pre-fill with currently opened binary paths when user does "File-> Diff new Files"

10- When diffing I used "diff -1" as "File -1" and "diff-2" as "File-2" - it correctly showed that "sym.mult" was removed in "diff-2" but if I diff them in opposite order (meaning diff-1 as File-2 and vice versa) it doesn't show that "sym.mult" was Added, both "Added" and "Removed" tabs become empty. I could be misinterpreting something or it could be a rizin issue (will double check this later)

• For convenience "diff-1' and "diff-2" are attached at the end

11- When diffing a large file, the waiting dialog blocks the cutter main window as well, meaning user can't use the main cutter app when diffing is happening (if running it in a separate thread we have to make sure whether the diff functions change rizin's internal state or not, if they do then core has to be locked meaning the user can't use the main cutter app properly because no rizin function can be called till diffing is done - I will check more on this and provide an update)

12- Even though I like the appearance of waiting dialog, it doesn't seem consistent with cutter theme. You can leave it as it is for now - we can discuss the appearance of it later

Binaries:
diff-1.zip
diff-2.zip

[wargio]

10- When diffing I used "diff -1" as "File -1" and "diff-2" as "File-2" - it correctly showed that "sym.mult" was removed in "diff-2" but if I diff them in opposite order (meaning diff-1 as File-2 and vice versa) it doesn't show that "sym.mult" was Added, both "Added" and "Removed" tabs become empty. I could be misinterpreting something or it could be a rizin issue (will double check this later)

This sounds like a bug. let me also check rizin

Edit:

On rizin side, looks correct.

$ rz-diff -t command -e scr.color=0 -0 izzq diff-1 diff-2

--- diff-1
+++ diff-2
@@ -12,7 +12,7 @@
 0x51b 12 11 GLIBC_2.2.5
 0x1031 4 4 PTE1
 0x10cb 4 4 u3UH
-0x2040 4 4 \e\f\a\b
+0x2038 4 4 \e\f\a\b
 ---------- 27 26 GCC: (GNU) 16.1.1 20260430
 ---------- 8 7 Scrt1.o
 ---------- 10 9 __abi_tag
@@ -23,7 +23,7 @@
 ---------- 39 38 __do_global_dtors_aux_fini_array_entry
 ---------- 12 11 frame_dummy
 ---------- 31 30 __frame_dummy_init_array_entry
----------- 11 10 diff-1.cpp
+---------- 11 10 diff-2.cpp
 ---------- 10 9 crtendS.o
 ---------- 14 13 __FRAME_END__
 ---------- 19 18 __GNU_EH_FRAME_HDR
@@ -34,7 +34,6 @@
 ---------- 27 26 __cxa_finalize@GLIBC_2.2.5
 ---------- 5 4 main
 ---------- 13 12 __dso_handle
----------- 9 8 _Z4multv
 ---------- 6 5 _fini
 ---------- 29 28 __libc_start_main@GLIBC_2.34
 ---------- 6 5 _init

$ rz-diff -t command -e scr.color=0 -0 izzq diff-2 diff-1

--- diff-2
+++ diff-1
@@ -12,7 +12,7 @@
 0x51b 12 11 GLIBC_2.2.5
 0x1031 4 4 PTE1
 0x10cb 4 4 u3UH
-0x2038 4 4 \e\f\a\b
+0x2040 4 4 \e\f\a\b
 ---------- 27 26 GCC: (GNU) 16.1.1 20260430
 ---------- 8 7 Scrt1.o
 ---------- 10 9 __abi_tag
@@ -23,7 +23,7 @@
 ---------- 39 38 __do_global_dtors_aux_fini_array_entry
 ---------- 12 11 frame_dummy
 ---------- 31 30 __frame_dummy_init_array_entry
----------- 11 10 diff-2.cpp
+---------- 11 10 diff-1.cpp
 ---------- 10 9 crtendS.o
 ---------- 14 13 __FRAME_END__
 ---------- 19 18 __GNU_EH_FRAME_HDR
@@ -34,6 +34,7 @@
 ---------- 27 26 __cxa_finalize@GLIBC_2.2.5
 ---------- 5 4 main
 ---------- 13 12 __dso_handle
+---------- 9 8 _Z4multv
 ---------- 6 5 _fini
 ---------- 29 28 __libc_start_main@GLIBC_2.34
 ---------- 6 5 _init

[NewtronReal]

10- When diffing I used "diff -1" as "File -1" and "diff-2" as "File-2" - it correctly showed that "sym.mult" was removed in "diff-2" but if I diff them in opposite order (meaning diff-1 as File-2 and vice versa) it doesn't show that "sym.mult" was Added, both "Added" and "Removed" tabs become empty. I could be misinterpreting something or it could be a rizin issue (will double check this later)

This sounds like a bug. let me also check rizin

Edit:

On rizin side, looks correct.

$ rz-diff -t command -e scr.color=0 -0 izzq diff-1 diff-2

--- diff-1
+++ diff-2
@@ -12,7 +12,7 @@
 0x51b 12 11 GLIBC_2.2.5
 0x1031 4 4 PTE1
 0x10cb 4 4 u3UH
-0x2040 4 4 \e\f\a\b
+0x2038 4 4 \e\f\a\b
 ---------- 27 26 GCC: (GNU) 16.1.1 20260430
 ---------- 8 7 Scrt1.o
 ---------- 10 9 __abi_tag
@@ -23,7 +23,7 @@
 ---------- 39 38 __do_global_dtors_aux_fini_array_entry
 ---------- 12 11 frame_dummy
 ---------- 31 30 __frame_dummy_init_array_entry
----------- 11 10 diff-1.cpp
+---------- 11 10 diff-2.cpp
 ---------- 10 9 crtendS.o
 ---------- 14 13 __FRAME_END__
 ---------- 19 18 __GNU_EH_FRAME_HDR
@@ -34,7 +34,6 @@
 ---------- 27 26 __cxa_finalize@GLIBC_2.2.5
 ---------- 5 4 main
 ---------- 13 12 __dso_handle
----------- 9 8 _Z4multv
 ---------- 6 5 _fini
 ---------- 29 28 __libc_start_main@GLIBC_2.34
 ---------- 6 5 _init

$ rz-diff -t command -e scr.color=0 -0 izzq diff-2 diff-1

--- diff-2
+++ diff-1
@@ -12,7 +12,7 @@
 0x51b 12 11 GLIBC_2.2.5
 0x1031 4 4 PTE1
 0x10cb 4 4 u3UH
-0x2038 4 4 \e\f\a\b
+0x2040 4 4 \e\f\a\b
 ---------- 27 26 GCC: (GNU) 16.1.1 20260430
 ---------- 8 7 Scrt1.o
 ---------- 10 9 __abi_tag
@@ -23,7 +23,7 @@
 ---------- 39 38 __do_global_dtors_aux_fini_array_entry
 ---------- 12 11 frame_dummy
 ---------- 31 30 __frame_dummy_init_array_entry
----------- 11 10 diff-2.cpp
+---------- 11 10 diff-1.cpp
 ---------- 10 9 crtendS.o
 ---------- 14 13 __FRAME_END__
 ---------- 19 18 __GNU_EH_FRAME_HDR
@@ -34,6 +34,7 @@
 ---------- 27 26 __cxa_finalize@GLIBC_2.2.5
 ---------- 5 4 main
 ---------- 13 12 __dso_handle
+---------- 9 8 _Z4multv
 ---------- 6 5 _fini
 ---------- 29 28 __libc_start_main@GLIBC_2.34
 ---------- 6 5 _init

    added = new DiffMismatchModel(&listDel, this);
    removed = new DiffMismatchModel(&listAdd, this);

Problem in the cutter part only will be fixing it.

[NewtronReal]

Not going into the code details yet. Here are the issues I found while testing, some of them could be because they aren't implemented yet, in that case this comment can serve as a checklist

1- "Diff files" option/action should be in "Tools" not in "File"
Done
2- There should be an option to set current file as either "File 1" or "File 2"
Done
3- Also, it would be better to allow user to manually type the filepath aswell - for "File 1" and "File 2" - (same as InitialFileDialog)
Done(Although not quite like InitialFileDialog)
4- Not sure why "Match Results" have an odd greyish background

* Also better to remove the initial padding that Qt automatically puts in treeview (the space on the left side of entries)

Needed to add colors gui.match.partial and gui.match.perfect on rizin side the pr is still pending review.
rizin:#6614

5- "Functions B" shows functions for "File 1" and "Functions A" shows functions for "File 2" for some reason

Fixed.

6- Why name them "File 1" and "File 2" in the initial dialog and then name them "Functions A" and "Functions B"? I think it would be better to name the binary itself instead of "Functions A" if the binary name is too long then you can put and max character limit on it and do string elision based on that (replace last X chars with ...), same with "INFO A" and "INFO B" and any other occurences

Done

7- Function offsets should be in hex not decimal

* Also copying function address through context menu should also copy in hex (just so it's consistent with disassembly)

Done

8- It would be nice if when clicking on a any function entry in "Functions A/B" would switch to Hex Diff and seek to the function offset, same with entries in Match Results, Added and Removed tabs

Done(on double click it seeks to the function offset and shows hexdiff)

9- IMO the "Select which files to compare" should be shown when opening "Diff files" through the "tools" menu, seeing and empty widget and clicking "File->Diff new Files" is a lot more work, however "File->Diff new Files" option should be there and it should pre-fill with currently opened binary paths when user does "File-> Diff new Files"

Added(The CutterDiffWindows are now on independent(ie parent = nullptr) and deletes on close.)

10- When diffing I used "diff -1" as "File -1" and "diff-2" as "File-2" - it correctly showed that "sym.mult" was removed in "diff-2" but if I diff them in opposite order (meaning diff-1 as File-2 and vice versa) it doesn't show that "sym.mult" was Added, both "Added" and "Removed" tabs become empty. I could be misinterpreting something or it could be a rizin issue (will double check this later)

* For convenience "diff-1' and "diff-2" are attached at the end

This is a rizin side issue where they match every function of A to B means a function in matches multiple times in A(Atleast that's what i think it is am currently checking how analysis_match_functions work in rizin and soon will be fixing it from the qt side.)

11- When diffing a large file, the waiting dialog blocks the cutter main window as well, meaning user can't use the main cutter app when diffing is happening (if running it in a separate thread we have to make sure whether the diff functions change rizin's internal state or not, if they do then core has to be locked meaning the user can't use the main cutter app properly because no rizin function can be called till diffing is done - I will check more on this and provide an update)

Similar fix to that of CutterDiffWindow made it parentless and deleteonclose. Cores are locked in CutterDiff the cores are independent of the CutterCore rzCore. But still used a RAII for future purposes on assumption of other diffing operations may require multiple threads running at same time.

12- Even though I like the appearance of waiting dialog, it doesn't seem consistent with cutter theme. You can leave it as it is for now - we can discuss the appearance of it later

Left as it was for now.

Binaries: diff-1.zip diff-2.zip

[NewtronReal]

Initial work on LineDiff had started which requires rizin#6614 as well.

[wargio]

reorder the variables in the constructor

[PremadeS]

Only updates Line and graph diff if function is selected from "Match Results" tab, Doesn't update them via seeking or selecting from side "Functions tabs"

Also clicking on a function from the left functions tab switches to HexDiff everytime, instead or current tab

[NewtronReal]

Only updates Line and graph diff if function is selected from "Match Results" tab, Doesn't update them via seeking or selecting from side "Functions tabs"

Also clicking on a function from the left functions tab switches to HexDiff everytime, instead or current tab

Oh that thing is not changed after adding the CutterDiffItem. I'll change it in the CutterDiffWindow that when the function is clicked the current CutterDiffItem will be changed to the corresponding function address.

[PremadeS]

Some changes:

Text related:

1. In "DiffLoadDialog" change "Select which file to compare" to "Select files to compare"
2. In "DiffLoadDialog" change File 1(required) to File A, same for File 2. Using 1/2 suffix in filenames in "DiffLoadDialog" while main diff window uses A/B seems inconsistent. Use the A/B scheme everywhere imo
3. When clicking select on File 1, the dialog title shows Select File 2, change to Select File A
4. In "DiffLoadDialog" when clicking OK after leaving one of the filepaths empty, it shows "Select a file for diffing" with the title "Empty FileA" change to "Please provide a valid path for File A", with the title "Invalid File Path"
5. In "DiffLoadDialog" when clicking OK after providing an invalid file path or providing a file with insufficient permissions it should say "Cannot open file "
6. In the left-side Functions widget, it's better to name the first column "Name" instead of "Function Name", would same some space
7. In the DiffWatDialog it says, "original file" and "modified file" - change to File A and B aswell
8. In "LineDiff" it says Unified/Split/Left/Right while in "GraphDiff" it is "Unified/Split/Original/Modified". Better to name them File A / File B instead of Left/Right or Original/Modified

Others:

1. In "DiffLoadDialog" when clicking OK after providing a file with insufficient permissions it still opens it and shows invalid data instead of refusing to open and displaying error message. In rz-diff it shows this:
   ERROR: rz-diff: error, cannot open file '<FILENAME>'
2. In "DiffLoadDialog" when clicking OK after providing an empty filepath, it shows error message and closes the "DiffLoadDialog aswell". It should keep the "DiffLoadDialog" open imo, so the user can try again
3. Color scheme looks a bit dark in light theme imo (Same with graphs):

Also the percentage text is not visible in light theme until the blue background of progress bar reaches it (the percentage is white on white background)

4. "Shift" buttons in hexviews look a bit odd. They take too much space imo. Any ideas for this? (maybe move them in the Info container and name it something generic?). Also in the buttons "File A" ha spacing while "FileB" doesn't
5. Add a bit of space between "Info A" and "Info B" container, they seem too crammed together, also use "Info A/B" instead of all caps (INFO A/B)
6. Whats with the empty space at the bottom of the DiffWindow?

7) Clicking "Seek To" shifts the hex widget positions, is that intentional? 8) Remove the left padding before every item from both tree views in left functions widget 9) Too much spacing/padding at the bottom of "GraphDiff" while on "LineDiff" it seems fine.

Some additions:

1. It would be nice to make the two hexwidgets adjustable aswell, similar to the horizontal/vertical split views in Line/Graph Diff
2. Double clicking on any function in the left functions widget, Match results, Added and Removed should switch to the last used memory diff widget (Line, graph, hex)
3. Add context menus in left functions, added and removed widgets. Also separate each action category. Currently in "Match results context menu" "Copy Address" appears in between "Line Diff" and "Graph Diff". It would also be better to put Line Diff, Hex Diff and Graph Diff in a submenu, maybe call it "Show In" (See the context menu of Functions Widget in mainwindow). Context menu should have Copy (which copies entire line) and "Copy Address"
4. Saving the widget layout upon closing would also be a nice addition imo.
5. Add closable quick filter in all item based widgets
6. Make all treeviews sortable by column

-) Maybe add the option to undo / redo seek?

[PremadeS]

Opening the same file as File A and File B, shows same functions in Added and Removed tabs.
I don't think it's a rizin issue, will verify in a bit

Binary: libssl_1.0.1f.so.zip

Use this as both File A and File B

[NewtronReal]

Some changes:

Text related:

1. In "DiffLoadDialog"  change "Select which file to compare" to "Select files to compare"

Done

2. In "DiffLoadDialog" change File 1(required) to File A, same for File 2. Using 1/2  suffix in filenames in "DiffLoadDialog" while main diff window uses A/B seems inconsistent. Use the A/B scheme everywhere imo

Done

3. When clicking select on File 1, the dialog title shows Select File 2, change to Select File A

Done

4. In "DiffLoadDialog" when clicking OK after leaving one of the filepaths empty, it shows "Select a file for diffing" with the title "Empty FileA" change to "Please provide a valid path for File A", with the title "Invalid File Path"

Done

5. In "DiffLoadDialog" when clicking OK after providing an invalid file path or providing a file with insufficient permissions it should say "Cannot open file "

Done

6. In the left-side Functions widget, it's better to name the first column "Name" instead of "Function Name", would same some space

Done

7. In the DiffWatDialog it says, "original file" and "modified file" - change to File A and B aswell

Done

8. In "LineDiff" it says Unified/Split/Left/Right while in "GraphDiff" it is "Unified/Split/Original/Modified". Better to name them File A / File B instead of Left/Right or Original/Modified

Done

Others:

1. In "DiffLoadDialog" when clicking OK after providing a file with insufficient permissions it still opens it and shows invalid data instead of refusing to open and displaying error message. In `rz-diff` it shows this:
   `ERROR: rz-diff: error, cannot open file '<FILENAME>'`

Done. Added cannot open file message box.

2. In "DiffLoadDialog" when clicking OK after providing an empty filepath, it shows error message and closes the "DiffLoadDialog aswell". It should keep the "DiffLoadDialog" open imo, so the user can try again

Done

3. Color scheme looks a bit dark in light theme imo (Same with graphs):

Done Alpha composed with the background.

Also the percentage text is not visible in light theme until the blue background of progress bar reaches it (the percentage is white on white background)

Done. By alpha composing

4. "Shift" buttons in hexviews look a bit odd. They take too much space imo. Any ideas for this? (maybe move them in the Info container and name it something generic?). Also in the buttons "File A" ha spacing while "FileB" doesn't

This along with 7 was taken as a feature from rizin where you are able to seek between bytes using the buttons. May be we can replace the buttons with keyboard shortcuts. This feature works with startAddressA, transpose(relative distance to B's address), and the cursors current position. The shift buttons changes the transpose and startAddressA. May be we can use a Numberbox and let the user set the transpose and startAddressA themself instead. may be we can give two numberboxes instead.

5. Add a bit of space between "Info A" and "Info B" container, they seem too crammed together, also use "Info A/B" instead of all caps (INFO A/B)

6. Whats with the empty space at the bottom of the DiffWindow?

<img alt="GSOC_Diff_Space_Bottom" width="2000" height="1125" src="https://private-user-images.githubusercontent.com/128969155/650918634-0483cb9c-722b-4878-8464-2223251f7718.png?jwt=eyJ0eXAiOiJKV1QiLCJhbGciOiJIUzI1NiJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3ODk1ODkyNjYsIm5iZiI6MTc4OTU4ODk2NiwicGF0aCI6Ii8xMjg5NjkxNTUvNjUwOTE4NjM0LTA0ODNjYjljLTcyMmItNDg3OC04NDY0LTIyMjMyNTFmNzcxOC5wbmc_WC1BbXotQWxnb3JpdGhtPUFXUzQtSE1BQy1TSEEyNTYmWC1BbXotQ3JlZGVudGlhbD1BS0lBVkNPRFlMU0E1M1BRSzRaQSUyRjIwMjYwOTE2JTJGdXMtZWFzdC0xJTJGczMlMkZhd3M0X3JlcXVlc3QmWC1BbXotRGF0ZT0yMDI2MDkxNlQyMDAyNDZaJlgtQW16LUV4cGlyZXM9MzAwJlgtQW16LVNpZ25hdHVyZT1jMzYyMzhhZjE2YjY0YTQyMGQ0ZmQ4YjIwYTUxOGMyZWUxNDNmNjQ0NzQwNGJkNjQ3NzM5MTdmNmIxNjlhZjk5JlgtQW16LVNpZ25lZEhlYWRlcnM9aG9zdCZyZXNwb25zZS1jb250ZW50LXR5cGU9aW1hZ2UlMkZwbmcifQ._Adhinex4ZgP3lb1j0S34P0ozOyA-jQ8_jHJQDBGeJE"

Done

7. Clicking "Seek To" shifts the hex widget positions, is that intentional?
   Seek and align feature is intentional which is now removed from the PR until the seek address work is complete. This lets the user align the hexdiffviews. It felt a bit complex to understand and use for the user. So i think this can be added as a check box called auto align where they'll set the values for transposeA and transposeB number boxes as discused

8. Remove the left padding before every item from both tree views in left functions widget 9) Too much spacing/padding at the bottom of "GraphDiff" while on "LineDiff" it seems fine.

Some additions:

1. It would be nice to make the two hexwidgets adjustable aswell, similar to the horizontal/vertical split views in Line/Graph Diff

the hexDiffView is a single QtCanvas not 2 widget that we can simply add a spliter to make it adjustable. I think that's a work for another PR since the entire painting logic has to be transferred to two seperate widgets with their own painter and synchronizing object. Also i think this feature aligns itself with the existing HexDumpView widget for setting row length as multiple of 2 byte pairing etc.

2. Double clicking on any function in the left functions widget, Match results, Added and Removed should switch to the last used memory diff widget (Line, graph, hex)

3. Add context menus in left functions, added and removed widgets. Also separate each action category. Currently in "Match results context menu" "Copy Address" appears in between "Line Diff" and "Graph Diff". It would also be better to put Line Diff, Hex Diff and Graph Diff in a submenu, maybe call it "Show In" (See the context menu of Functions Widget in mainwindow). Context menu should have Copy (which copies entire line) and "Copy Address"

Done. Grouping them is left.

4. Saving the widget layout upon closing would also be a nice addition imo.

Implemented and being tested.

5. Add closable quick filter in all item based widgets

Done.

6. Make all treeviews sortable by column

Done.

-) Maybe add the option to undo / redo seek?
Implemented partially can seek through <Memory widget, diffItem> pairs. May be extend to <Memory widget, diffItem, transposeA, tranposeB> pairs in future and seek through graphblocks and difflines as well?

[NewtronReal]

For cutterside sorting the code is left commented till the rizin submodule is updated to the latest. Also rz_analysis_match_blocks has the same issue.