diff options
| author | Danilo M. <danix@danix.xyz> | 2026-08-08 10:16:25 +0200 |
|---|---|---|
| committer | Danilo M. <danix@danix.xyz> | 2026-08-08 10:16:25 +0200 |
| commit | bb3e119a55345e997efe80ec275e67acf7a04851 (patch) | |
| tree | 2d8ce8eca66685e34efe225804b2678fb169f833 /src/htmlbuilder.h | |
| parent | e0595d421c2327ae44ec7af3940ec6ef3d966eae (diff) | |
| download | qtmaildir-bb3e119a55345e997efe80ec275e67acf7a04851.tar.gz qtmaildir-bb3e119a55345e997efe80ec275e67acf7a04851.zip | |
feat(worker): load a thread as a reply tree with per-message depth
loadThread could not be extended to do this. It walks
notmuch_query_search_messages, and a message obtained that way returns NULL from
notmuch_message_get_replies (notmuch.h:1617-1628), so that walk cannot produce
reply depth at all. The tree comes from notmuch_thread_get_toplevel_messages
instead, and the pane keeps the flat list it wants.
walkReplies takes raw notmuch_message_t*, against this file's rule that every
handle is RAII-owned. Messages reached through a thread are freed with it
(notmuch.h:1637), so an NmMessage wrapper would destroy memory the thread frees
again. The NmThread in the caller is what keeps them alive.
Every message in the thread gets a node regardless of the query: the list is
where the reply count is read, and hiding unmatched replies would make that
count disagree with the rows under it.
Both tests mutation-checked. Flattening depth fails the depth assertion, and
skipping the thread walk fails it too, so neither passes against the two
mistakes the notmuch API invites.
Diffstat (limited to 'src/htmlbuilder.h')
0 files changed, 0 insertions, 0 deletions
