On Tue, 30 Apr 2013 17:35:01 +0200, Aaron J. Seigo <ase...@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...

Reply via email to