libs/kworkspace/kdisplaymanager.cpp mess

Martin Briza mbriza at redhat.com
Thu May 2 12:18:13 BST 2013


On Tue, 30 Apr 2013 17:35:01 +0200, Aaron J. Seigo <aseigo at 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.
>
> oh, btw.. the LightDM code there is also shared by SDDM. calling it
> "LightDM" is a bit of a misnomer now as a result. might be better to  
> call it
> something related to the DBus interface it implements?
>

Yes, I actually like this approach to DM interfacing a lot.
I'd like to stick to something related to the interface's name as you  
wrote, but "org.freedesktop.DisplayManager" is a tad long and calling it  
just "DisplayManager" is too general...




More information about the kde-core-devel mailing list