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