[Differential] [Commented On] D3617: [Touchpad KCM] New KWin Wayland version

2016-12-30 Thread Sebastian Kügler
sebas added a comment. Clarified your questions. My remaining comments are all non-critical so as long as you consider them, it's up to you if you change things or not. Just wanted to make sure it's not an oversight there. INLINE COMMENTS > ExclGroupBox.qml:53 > +id

[Differential] [Commented On] D3617: [Touchpad KCM] New KWin Wayland version

2016-12-27 Thread subdiff (Roman Gilg)
subdiff added inline comments. INLINE COMMENTS > sebas wrote in kwinwaylandtouchpad.h:36 > I wonder if it wouldn't be neater if the features would be an enum, and you'd > have methods to check for supported and enabled features. Have you thought of > this? Haven't thought about it. The structu

[Differential] [Commented On] D3617: [Touchpad KCM] New KWin Wayland version

2016-12-26 Thread Martin Gräßlin
graesslin added inline comments. INLINE COMMENTS > subdiff wrote in kwinwaylandbackend.cpp:84 > Are you sure? The foreach keyword is still listed in the official Qt docu. > What's the best alternative? A normal for-loop? With upcounting integer or > iterator? https://www.kdab.com/goodbye-q_for

[Differential] [Commented On] D3617: [Touchpad KCM] New KWin Wayland version

2016-12-25 Thread subdiff (Roman Gilg)
subdiff added inline comments. INLINE COMMENTS > knambiar wrote in kwinwaylandbackend.cpp:72 > Just wondering whether these calls are synchronous and may cause delay in > startup. > Just wondering whether these calls are synchronous and may cause delay in > startup. I assume so. Should be no

[Differential] [Commented On] D3617: [Touchpad KCM] New KWin Wayland version

2016-12-08 Thread knambiar (Rajeesh K Nambiar)
knambiar added a comment. A query about the dbus calls in findTouchpads(). INLINE COMMENTS > kwinwaylandbackend.cpp:72 > +QStringList devicesSysNames; > +QVariant reply = m_deviceManager->property("devicesSysNames"); > +if (reply.isValid()) { Just wondering whether these calls ar