Merge lp:~loic.molinari/ubuntu-ui-toolkit/ubuntu-ui-toolkit-unused-variables-removal into lp:ubuntu-ui-toolkit/staging

[Loïc Molinari]

Removed unused variables breaking debug builds.

Removed unused variables breaking debug builds.

[PS Jenkins bot (ps-jenkins)]

FAILED: Continuous integration, rev:1384
http://jenkins.qa.ubuntu.com/job/ubuntu-sdk-team-ubuntu-ui-toolkit-staging-ci/1377/
Executed test runs:
    UNSTABLE: http://jenkins.qa.ubuntu.com/job/generic-deb-autopilot-vivid-touch/885
    FAILURE: http://jenkins.qa.ubuntu.com/job/ubuntu-sdk-team-ubuntu-ui-toolkit-staging-vivid-amd64-ci/104/console
    SUCCESS: http://jenkins.qa.ubuntu.com/job/ubuntu-sdk-team-ubuntu-ui-toolkit-staging-vivid-armhf-ci/107
        deb: http://jenkins.qa.ubuntu.com/job/ubuntu-sdk-team-ubuntu-ui-toolkit-staging-vivid-armhf-ci/107/artifact/work/output/*zip*/output.zip
    SUCCESS: http://jenkins.qa.ubuntu.com/job/ubuntu-sdk-team-ubuntu-ui-toolkit-staging-vivid-i386-ci/104
    UNSTABLE: http://jenkins.qa.ubuntu.com/job/generic-deb-autopilot-runner-vivid-mako/777
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-vivid-armhf/883
        deb: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-vivid-armhf/883/artifact/work/output/*zip*/output.zip
    SUCCESS: http://s-jenkins.ubuntu-ci:8080/job/touch-flash-device/17244

Click here to trigger a rebuild:
http://s-jenkins.ubuntu-ci:8080/job/ubuntu-sdk-team-ubuntu-ui-toolkit-staging-ci/1377/rebuild

review: Needs Fixing (continuous-integration)

[Zsombor Egri (zsombi)]

This is weird... The original setup does pass on my env, and even on CI did... Thanks of the fix!

review: Approve

[PS Jenkins bot (ps-jenkins)]

review: Approve (continuous-integration)

=== modified file 'modules/Ubuntu/Components/plugin/uclistitem.cpp'
--- modules/Ubuntu/Components/plugin/uclistitem.cpp	2015-01-16 14:52:17 +0000
+++ modules/Ubuntu/Components/plugin/uclistitem.cpp	2015-01-20 16:59:31 +0000
@@ -414,7 +414,6 @@
 {
     setHighlighted(false);
     // initiate rebinding only if there were actions tugged
-    Q_Q(UCListItem);
     if (!UCActionPanel::isConnected(leadingPanel) &&
         !UCActionPanel::isConnected(trailingPanel)) {
         return;

=== modified file 'modules/Ubuntu/Components/plugin/ucviewitemsattached.cpp'
--- modules/Ubuntu/Components/plugin/ucviewitemsattached.cpp	2015-01-06 06:11:00 +0000
+++ modules/Ubuntu/Components/plugin/ucviewitemsattached.cpp	2015-01-20 16:59:31 +0000
@@ -75,7 +75,6 @@
 void UCViewItemsAttachedPrivate::clearChangesList()
 {
     // clear property change objects
-    Q_Q(UCViewItemsAttached);
     Q_FOREACH(PropertyChange *change, changes) {
         // deleting PropertyChange will restore the saved property
         // to its original binding/value