https://bugs.kde.org/show_bug.cgi?id=486067
Nicolas Fella <nicolas.fe...@gmx.de> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
         Resolution|---                         |FIXED
      Latest Commit|https://invent.kde.org/plas |https://invent.kde.org/plas
                   |ma/powerdevil/-/commit/c8f6 |ma/powerdevil/-/commit/b3fa
                   |0bbfdefebfc336a1b6c1bda89a9 |877b4f3665451023ab58e425027
                   |04c105fc6                   |c1d2cf870
             Status|ASSIGNED                    |RESOLVED

--- Comment #33 from Nicolas Fella <nicolas.fe...@gmx.de> ---
Git commit b3fa877b4f3665451023ab58e425027c1d2cf870 by Nicolas Fella.
Committed on 10/02/2025 at 22:39.
Pushed by nicolasfella into branch 'master'.

KeyboardBrightnessController: Drop isValid() check for upower interface

isValid() seems to be racy when upower is just starting up (because we
DBus-activated it earlier in BatteryController).

This causes us to think keyboard brightness control is not available even
when it actually is.

To address this always make a call to the interface. The call will
block until the interface is ready to either give us the correct result
or we know it's not available because we get a relevant DBus error.

As a bonus print a warning message if we get any unexpected errors

M  +16   -16   daemon/controllers/keyboardbrightnesscontroller.cpp

https://invent.kde.org/plasma/powerdevil/-/commit/b3fa877b4f3665451023ab58e425027c1d2cf870

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

Reply via email to