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

--- Comment #10 from Jakob Petsovits <jpe...@petsovits.com> ---
(In reply to Jakob Petsovits from comment #4)
> I can reproduce something similar, except for "Built-in Screen".
>
> (...)
>
> Let's do some debugging to see if this is a bug in the applet or in sending
> D-Bus DisplayAdded/DisplayRemoved signals.

QDBusViewer confirms that PowerDevil emits the correct DisplayRemoved signals,
meaning no issues in the daemon internally related to this. So the next
question was whether the applet will correctly receive these signals and update
the display model.

It turns out that the async nature of checkDisplayNames()/updateDisplayNames()
in ScreenBrightnessControl is what causes this issue. In a nutshell, the change
signals come in fairly quickly, and the use of QCoro means that another call to
queryAndInsertDisplay() may be fired before the previous one had finished.

I've got a fix for this, which bluntly rips out the use of async functions
where display model changes are involved. Which is most of them, really. I'm
sure there's a more efficient fix, but I'll stick to just making it correct for
now.

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

Reply via email to