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

--- Comment #4 from Jakob Petsovits <jpe...@petsovits.com> ---
(In reply to Tim from comment #3)
> Shouldn't the initial value of the batteryConservationMode be set to the
> value read by the getconservationmode job similar to the line above?
> 
> https://github.com/KDE/powerdevil/blob/master/kcmodule/profiles/ExternalServiceSettings.cpp#L105

So that it reads like this:

setBatteryConservationMode(m_savedBatteryConservationMode);

which was just written by setSavedBatteryConservationMode() with the job
result, instead of

setBatteryConservationMode(m_batteryConservationMode);

which is merely the pre-existing value as it was initialized earlier? I think
that's an extremely promising attempt at a fix.

Fabian, could you test this change and submit an MR if it fixes the bug?

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

Reply via email to