Review Request 128074: [libkscreen] [Backend Manager] emitBackendReady() only after fetching the config finishes

Kai Uwe Broulik kde at privat.broulik.de
Wed Jun 1 14:39:34 UTC 2016



> On Juni 1, 2016, 2:19 nachm., Sebastian Kügler wrote:
> > I've gone over the handlers for backendReady, and none of them is actually used to request a config then (that's encapsulated in backendmanager), so requesting a config before the backend is announced ready makes sense.

5.6 or better play safe and master?


- Kai Uwe


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/128074/#review96138
-----------------------------------------------------------


On Juni 1, 2016, 1:30 nachm., Kai Uwe Broulik wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/128074/
> -----------------------------------------------------------
> 
> (Updated Juni 1, 2016, 1:30 nachm.)
> 
> 
> Review request for Plasma, Daniel Vrátil and Sebastian Kügler.
> 
> 
> Repository: libkscreen
> 
> 
> Description
> -------
> 
> Without a valid config in the BackendManager, KScreen::Config::canBeApplied always returns false causing KScreen to ignore any stored config on startup sometimes. Wait until we have a config before telling that the backend is ready.
> 
> 
> Diffs
> -----
> 
>   src/backendmanager.cpp 570f575 
> 
> Diff: https://git.reviewboard.kde.org/r/128074/diff/
> 
> 
> Testing
> -------
> 
> I can now start kded5 from console without it falling back to an "ideal" config.
> 
> Didn't do much testing, though. Might have unwanted side-effects.
> 
> 
> Thanks,
> 
> Kai Uwe Broulik
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20160601/5d6ac1c6/attachment.html>


More information about the Plasma-devel mailing list