diff options
| author | blasty <peter@haxx.in> | 2026-07-23 21:39:08 +0200 |
|---|---|---|
| committer | blasty <peter@haxx.in> | 2026-07-23 21:39:33 +0200 |
| commit | 1417d81081e7fec0ef044619a8790802c3c66255 (patch) | |
| tree | 471bf78b46de8f290c58af26f50cc34064d75f3c /docs/TUI_DRIVING_BLUEPRINT.md | |
| parent | views: highlight all occurrences of the token under the cursor (diff) | |
| download | ida-tui-1417d81081e7fec0ef044619a8790802c3c66255.tar.gz ida-tui-1417d81081e7fec0ef044619a8790802c3c66255.tar.xz ida-tui-1417d81081e7fec0ef044619a8790802c3c66255.zip | |
listing: restore full-segment search load (find every match)
Revert the "search only the loaded portion" shortcut from f5aaf50 and go back to
a single, guarded background load_all before matching, so a search finds every
occurrence in the segment (not just whatever streamed in so far). The guard
(_search_loading + queued callbacks) shares ONE load across per-keystroke
updates instead of spawning/cancelling a worker each keypress.
The wedge that shortcut was meant to avoid turned out to be unrelated to the
search load: it is a pre-existing navigation bug (opening a function right after
scrolling to the bottom of another leaves the listing cursor out of range and
'/' fails to focus the search input, so typed chars leak into the view as
listing verbs, one of which is 'u'=undefine). Reproduces at 0c097f5, before any
of this work; tracked separately.
Verified in isolation: search 3.1s, matches found; search/mouse/func_banners/
listing_view/continuous_view/decomp_nav/follow_xrefs 45/0.
Diffstat (limited to 'docs/TUI_DRIVING_BLUEPRINT.md')
0 files changed, 0 insertions, 0 deletions
