[Merge] lp:~vicamo/qtubuntu-sensors/reimplement-haptics-backend into lp:qtubuntu-sensors

Alfonso Sanchez-Beato alfonso.sanchez-beato at canonical.com
Thu Jan 12 11:42:40 UTC 2017


The proposal to merge lp:~vicamo/qtubuntu-sensors/reimplement-haptics-backend into lp:qtubuntu-sensors has been updated.

Commit Message changed to:

This change rewrites QtFeedbackHapticsEffect backend for ubuntu.

The original implementation is to call APIs in libubuntu-platform-api, which then issue D-Bus calls to usensord. However, it seems in some cases instantiating a D-Bus call in such Qt module may result in segmentation fault. This change rewrites QtFeedbackHapticsEffect backend with QtDBus directly and works around this issue.

This change also tries to conform to QtFeedbackHaptics features definitions. Duration/period is now correctly supported, although this doesn't mean much because usensord does not support variable vibration intensity. It also provides a chance to stop a periodical vibration. The enable/state properties are now correctly set and returned.

For more details, see:
https://code.launchpad.net/~vicamo/qtubuntu-sensors/reimplement-haptics-backend/+merge/300839
-- 
Your team Ubuntu Phablet Team is subscribed to branch lp:qtubuntu-sensors.



More information about the Ubuntu-reviews mailing list