https://bugs.kde.org/show_bug.cgi?id=488713

Vlad Zahorodnii <vlad.zahorod...@kde.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
      Latest Commit|https://invent.kde.org/plas |https://invent.kde.org/plas
                   |ma/kdeplasma-addons/-/commi |ma/kdeplasma-addons/-/commi
                   |t/1245968618afff3a0bba5e2c6 |t/06d0cd9cd918e52155804b256
                   |088e77f0c9f8aff             |23c5b9cf4d5c6cb

--- Comment #55 from Vlad Zahorodnii <vlad.zahorod...@kde.org> ---
Git commit 06d0cd9cd918e52155804b25623c5b9cf4d5c6cb by Vlad Zahorodnii.
Committed on 28/09/2024 at 16:34.
Pushed by vladz into branch 'Plasma/6.2'.

runners/converter: Fix thread affinity of currency update timer

Amends 8f164924d06015db13a8b703fdfa1258d7f8d718.

The runner lives in a separate thread, but the currency update timer
lives on the main thread. Furthermore, the update timer is owned by
the runner, which is not okay because its thread affinity is different.
It also means that the event loop on the main thread may not properly
clean up its timer list.

In order to fix the issue, this change makes the update timer a child
of the runner object. With that, both will have the same thread affinity.


(cherry picked from commit 1245968618afff3a0bba5e2c6088e77f0c9f8aff)

Co-authored-by: Vlad Zahorodnii <vlad.zahorod...@kde.org>

M  +4    -3    runners/converter/converterrunner.cpp
M  +1    -1    runners/converter/converterrunner.h

https://invent.kde.org/plasma/kdeplasma-addons/-/commit/06d0cd9cd918e52155804b25623c5b9cf4d5c6cb

-- 
You are receiving this mail because:
You are watching all bug changes.

Reply via email to