diff options
author | Nicolas Dichtel <nicolas.dichtel@6wind.com> | 2024-02-15 15:36:39 +0100 |
---|---|---|
committer | Robin Jarry <robin@jarry.cc> | 2024-02-15 15:57:00 +0100 |
commit | 46ea87602c092750984307e4f31200ad3b9d7fe4 (patch) | |
tree | c9f024bc1ed0d49776afda592edc5fdbc18d4af3 /stylesets | |
parent | d56e949a339d61da2cc7fc20ce0bf19dadf0095d (diff) | |
download | aerc-46ea87602c092750984307e4f31200ad3b9d7fe4.tar.gz |
store: fix split view blinking
When split view and threading mode are enabled, the message is blinking.
First, since commit ddfa5cac1fe9 ("msgstore: fix deadlock in thread
builder"), the threadCallback is never set to nil: the thread builder
calls it continuously.
After setting it to nil, the message is still blinking once. To avoid this,
don't call the onSelect method (which points to
AccountView.updateSplitView()) from the thread builder: the message is
already displayed.
Let's rewrite Select and selectPriv:
- Select(): it takes the lock and calls the onSelect callback;
- selectPriv: the assumption is that the lock is held. It doesn't call
the onSelect callback. This function is only used by the thread builder.
Fixes: 588be1a28422 ("store: improve cursor position")
Fixes: ddfa5cac1fe9 ("msgstore: fix deadlock in thread builder")
Signed-off-by: Nicolas Dichtel <nicolas.dichtel@6wind.com>
Acked-by: Robin Jarry <robin@jarry.cc>
Diffstat (limited to 'stylesets')
0 files changed, 0 insertions, 0 deletions