ktouchpadenabler moved to kdereview
faure at kde.org
Wed Jan 4 22:40:26 GMT 2012
On Wednesday 04 January 2012 18:51:44 Albert Astals Cid wrote:
> El Dimecres, 4 de gener de 2012, a les 01:53:13, Christoph Feck va escriure:
> > On Wednesday 04 January 2012 00:28:11 Albert Astals Cid wrote:
> > > My little kded daemon that listens to XF86XK_TouchpadToggle and
> > > enables disables the touchpad accordingly has been moved to
> > > kdereview.
> > >
> > > My plan is moving it to extragear, not really sure if -base or
> > > -utils.
> > >
> > > The code doesn't have a kcm or any kind of configuration since it
> > > is designed to "just work".
> > >
> > > I'd appreciate any review or suggestion over it.
> > I cannot test it because I have no touchpad, but if it is supposed to
> > "just work" without any UI, I suggest to just add it to "khotkeys" or
> > "kaccel" daemon (whichever of them is used for global shortcuts), so
> > that we do not filter global X11 keyboard events twice.
> I don't really see any point in doing that, nothing can be shared between
> them and the existing ktouchpadenabler so instead of one simple codebase
> (166 lines with 20 of headers) you end up adding more complexity to
> existing programs (probably integrating the code in the existing programs
> would be more than 166 lines).
IMHO this isn't about the number of lines of code, but about the runtime
performance (how many process to wake up when pressing a key).
kglobalaccel seems quite suitable indeed, no?
David Faure, faure at kde.org, http://www.davidfaure.fr
Sponsored by Nokia to work on KDE, incl. KDE Frameworks 5
More information about the kde-core-devel