diff options
| author | blasty <peter@haxx.in> | 2026-07-26 14:36:12 +0200 |
|---|---|---|
| committer | blasty <peter@haxx.in> | 2026-07-26 14:36:12 +0200 |
| commit | 95fc4ca72e0b840d333740b19715a87d9c631223 (patch) | |
| tree | 4e7c78a42e3888c961f793e854005f5ab5a61150 | |
| parent | app: delete DisasmView, superseded by the unified listing (diff) | |
| download | ida-tui-95fc4ca72e0b840d333740b19715a87d9c631223.tar.gz ida-tui-95fc4ca72e0b840d333740b19715a87d9c631223.tar.xz ida-tui-95fc4ca72e0b840d333740b19715a87d9c631223.zip | |
TODO: DisasmView removed
| -rw-r--r-- | TODO | 11 |
1 files changed, 0 insertions, 11 deletions
@@ -90,14 +90,3 @@ before trusting a failure that appeared without a code change. Coverage for "an edit must not move the view" lives in tests/test_blob_ui.py, which builds its own throwaway binary and can mutate freely. - -## DisasmView looks like dead code - -idatui/app.py defines DisasmView (Line-based, function-scoped) but the app never -instantiates or queries it — the unified ListingView replaced it, and even the -test harness's `Ctx.dis` returns ListingView. DisasmModel is still used, but only -to compute indices in _do_edit_item, never to render. - -Worth confirming and deleting: it's ~180 lines carrying its own rendering, -search and cursor logic that no longer runs, and it misled me into thinking -assembly highlighting needed doing twice. |