From kde-core-devel  Thu May 02 14:00:59 2013
From: "Martin Briza" <mbriza () redhat ! com>
Date: Thu, 02 May 2013 14:00:59 +0000
To: kde-core-devel
Subject: Re: libs/kworkspace/kdisplaymanager.cpp mess
Message-Id: <op.wwgmjxzffxu3g2 () dhcp-24-242 ! brq ! redhat ! com>
X-MARC-Message: https://marc.info/?l=kde-core-devel&m=136753195531284

On Thu, 02 May 2013 13:12:55 +0200, Martin Briza <mbriza@redhat.com> wrote:
> On Tue, 30 Apr 2013 17:33:57 +0200, Aaron J. Seigo <aseigo@kde.org>  
> wrote:
>> On Thursday, April 25, 2013 15:11:25 Martin Briza wrote:
>>>   - The KDisplayManager class is used only on a few places so  
>>> replacing its
>>> constructions with calls to the factory will be easy.
>>
>> i don't think KDisplayManager's public API needs to be changed in any  
>> way.
> The API itself wouldn't change. I'm just proposing usage of a singleton  
> factory class to have only one instance of the KDisplayManager class  
> instead of constructing the object all over. But say the word and I  
> implement it as you suggested below.

Started implementing it as you described, I'll let the singleton backend  
stay in KDisplayManager.
So far it seems there will be null (!defined(Q_WS_X11)), basic  
(legacy/current), systemd and maybe CK backends.
I'll push when I'm sure everything is okay into a new branch for you to  
see.