D26986: WIP-NotWorking: Try to fix flicker of screen while locked and detaching screens
Tomaz Canabrava
noreply at phabricator.kde.org
Wed Jan 29 12:39:34 GMT 2020
tcanabrava added a comment.
In D26986#602373 <https://phabricator.kde.org/D26986#602373>, @davidedmundson wrote:
> This is definitely an area for improvement.
>
> This code was initially written before QGuiApplication::screenRemoved existed, which is why it's like this.
> I think this solution is overly complex, you have the pointer to what changed in screenRemoved, we then discard that information and then go through loops to effectively retrieve that information we originally had.
> With a refactor that could be cleaner.
I'll try to refactor.
> As for your initial bug you're trying to fix:
> You're still in a race to see if kscreenlocker can update before kwin renders something, this might make kscreenlocker faster, but it won't resolve it being a race.
I'll poke vlad to see how to proceed here.
REPOSITORY
R133 KScreenLocker
REVISION DETAIL
https://phabricator.kde.org/D26986
To: tcanabrava
Cc: davidedmundson, plasma-devel, Orage, LeGast00n, The-Feren-OS-Dev, jraleigh, zachus, fbampaloukas, GB_2, ragreen, ZrenBot, ngraham, alexeymin, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20200129/f87894d8/attachment.html>
More information about the Plasma-devel
mailing list