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

2016-06-02 Thread Kai Uwe Broulik
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/128074/ --- (Updated June 2, 2016, 10:40 a.m.) Status -- This change has been ma

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

2016-06-01 Thread Sebastian Kügler
> On June 1, 2016, 2:19 p.m., 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. > > Kai

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

2016-06-01 Thread Kai Uwe Broulik
> 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

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

2016-06-01 Thread Sebastian Kügler
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/128074/#review96138 --- Ship it! I've gone over the handlers for backendReady, and

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

2016-06-01 Thread Kai Uwe Broulik
--- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/128074/ --- Review request for Plasma, Daniel Vrátil and Sebastian Kügler. Repository