Merge lp:~thomas-voss/platform-api/fix-1478845-take-2 into lp:platform-api

Proposed by Thomas Voß
Status: Merged
Approved by: kevin gunn
Approved revision: 313
Merged at revision: 311
Proposed branch: lp:~thomas-voss/platform-api/fix-1478845-take-2
Merge into: lp:platform-api
Diff against target: 97 lines (+14/-8)
4 files modified
debian/changelog (+6/-0)
debian/control (+1/-1)
src/ubuntu/application/common/application/location/position_update.cpp (+3/-3)
src/ubuntu/application/common/application/sensors/usensord_service.h (+4/-4)
To merge this branch: bzr merge lp:~thomas-voss/platform-api/fix-1478845-take-2
Reviewer Review Type Date Requested Status
kevin gunn (community) Approve
PS Jenkins bot continuous-integration Needs Fixing
Ubuntu Phablet Team Pending
Review via email: mp+267002@code.launchpad.net

Commit message

Make sure that const references are returned instead of temporary objects.
Bump build dep version for location-service to 3.
Fix LP:#1478845.

Description of the change

Make sure that const references are returned instead of temporary objects.
Bump build dep version for location-service to 3.
Fix LP:#1478845.

To post a comment you must log in.
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
kevin gunn (kgunn72) wrote :

via testing this definitely fixes the bug 1481279
it was 100% reliable bug corrected with this change

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'debian/changelog'
2--- debian/changelog 2015-07-27 18:07:53 +0000
3+++ debian/changelog 2015-08-05 08:17:59 +0000
4@@ -1,3 +1,9 @@
5+platform-api (3.0.0+15.10.20150730-0ubuntu2) wily; urgency=medium
6+
7+ * Fix LP:#1478845.
8+
9+ -- Thomas Voß <thomas.voss@canonical.com> Wed, 29 Jul 2015 09:02:19 +0200
10+
11 platform-api (3.0.0+15.10.20150727-0ubuntu1) wily; urgency=medium
12
13 * New rebuild forced.
14
15=== modified file 'debian/control'
16--- debian/control 2015-07-15 19:23:50 +0000
17+++ debian/control 2015-08-05 08:17:59 +0000
18@@ -14,7 +14,7 @@
19 liburl-dispatcher1-dev,
20 libdbus-1-dev,
21 libdbus-cpp-dev (>= 4.0.0),
22- libubuntu-location-service-dev (>= 1.0.0),
23+ libubuntu-location-service-dev (>= 3.0.0),
24 libgtest-dev,
25 libprocess-cpp-dev (>= 0.0.1+14.04.20131212),
26 libproperties-cpp-dev,
27
28=== modified file 'src/ubuntu/application/common/application/location/position_update.cpp'
29--- src/ubuntu/application/common/application/location/position_update.cpp 2014-06-23 11:17:42 +0000
30+++ src/ubuntu/application/common/application/location/position_update.cpp 2015-08-05 08:17:59 +0000
31@@ -65,7 +65,7 @@
32 ua_location_position_update_has_altitude(
33 UALocationPositionUpdate *update)
34 {
35- return update->update.value.altitude;
36+ return update->update.value.altitude ? true : false;
37 }
38
39 double
40@@ -79,7 +79,7 @@
41 ua_location_position_update_has_horizontal_accuracy(
42 UALocationPositionUpdate *update)
43 {
44- return update->update.value.accuracy.horizontal;
45+ return update->update.value.accuracy.horizontal ? true : false;
46 }
47
48 double
49@@ -93,7 +93,7 @@
50 ua_location_position_update_has_vertical_accuracy(
51 UALocationPositionUpdate *update)
52 {
53- return update->update.value.accuracy.vertical;
54+ return update->update.value.accuracy.vertical ? true : false;
55 }
56
57 double
58
59=== modified file 'src/ubuntu/application/common/application/sensors/usensord_service.h'
60--- src/ubuntu/application/common/application/sensors/usensord_service.h 2014-02-11 16:00:43 +0000
61+++ src/ubuntu/application/common/application/sensors/usensord_service.h 2015-08-05 08:17:59 +0000
62@@ -29,7 +29,7 @@
63
64 struct USensorD
65 {
66- static std::string& name()
67+ static const std::string& name()
68 {
69 static std::string s = "com.canonical.usensord";
70 return s;
71@@ -37,7 +37,7 @@
72
73 struct Haptic
74 {
75- static std::string name()
76+ static const std::string& name()
77 {
78 static std::string s = "com.canonical.usensord.haptic";
79 return s;
80@@ -45,7 +45,7 @@
81
82 struct Vibrate
83 {
84- static std::string name()
85+ static const std::string& name()
86 {
87 static std::string s = "Vibrate";
88 return s;
89@@ -60,7 +60,7 @@
90
91 struct VibratePattern
92 {
93- static std::string name()
94+ static const std::string& name()
95 {
96 static std::string s = "VibratePattern";
97 return s;

Subscribers

People subscribed via source and target branches