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
=== modified file 'debian/changelog'
--- debian/changelog 2015-07-27 18:07:53 +0000
+++ debian/changelog 2015-08-05 08:17:59 +0000
@@ -1,3 +1,9 @@
1platform-api (3.0.0+15.10.20150730-0ubuntu2) wily; urgency=medium
2
3 * Fix LP:#1478845.
4
5 -- Thomas Voß <thomas.voss@canonical.com> Wed, 29 Jul 2015 09:02:19 +0200
6
1platform-api (3.0.0+15.10.20150727-0ubuntu1) wily; urgency=medium7platform-api (3.0.0+15.10.20150727-0ubuntu1) wily; urgency=medium
28
3 * New rebuild forced.9 * New rebuild forced.
410
=== modified file 'debian/control'
--- debian/control 2015-07-15 19:23:50 +0000
+++ debian/control 2015-08-05 08:17:59 +0000
@@ -14,7 +14,7 @@
14 liburl-dispatcher1-dev,14 liburl-dispatcher1-dev,
15 libdbus-1-dev,15 libdbus-1-dev,
16 libdbus-cpp-dev (>= 4.0.0),16 libdbus-cpp-dev (>= 4.0.0),
17 libubuntu-location-service-dev (>= 1.0.0),17 libubuntu-location-service-dev (>= 3.0.0),
18 libgtest-dev,18 libgtest-dev,
19 libprocess-cpp-dev (>= 0.0.1+14.04.20131212),19 libprocess-cpp-dev (>= 0.0.1+14.04.20131212),
20 libproperties-cpp-dev,20 libproperties-cpp-dev,
2121
=== modified file 'src/ubuntu/application/common/application/location/position_update.cpp'
--- src/ubuntu/application/common/application/location/position_update.cpp 2014-06-23 11:17:42 +0000
+++ src/ubuntu/application/common/application/location/position_update.cpp 2015-08-05 08:17:59 +0000
@@ -65,7 +65,7 @@
65ua_location_position_update_has_altitude(65ua_location_position_update_has_altitude(
66 UALocationPositionUpdate *update)66 UALocationPositionUpdate *update)
67{67{
68 return update->update.value.altitude;68 return update->update.value.altitude ? true : false;
69}69}
7070
71double71double
@@ -79,7 +79,7 @@
79ua_location_position_update_has_horizontal_accuracy(79ua_location_position_update_has_horizontal_accuracy(
80 UALocationPositionUpdate *update)80 UALocationPositionUpdate *update)
81{81{
82 return update->update.value.accuracy.horizontal;82 return update->update.value.accuracy.horizontal ? true : false;
83}83}
8484
85double85double
@@ -93,7 +93,7 @@
93ua_location_position_update_has_vertical_accuracy(93ua_location_position_update_has_vertical_accuracy(
94 UALocationPositionUpdate *update)94 UALocationPositionUpdate *update)
95{95{
96 return update->update.value.accuracy.vertical;96 return update->update.value.accuracy.vertical ? true : false;
97}97}
9898
99double99double
100100
=== modified file 'src/ubuntu/application/common/application/sensors/usensord_service.h'
--- src/ubuntu/application/common/application/sensors/usensord_service.h 2014-02-11 16:00:43 +0000
+++ src/ubuntu/application/common/application/sensors/usensord_service.h 2015-08-05 08:17:59 +0000
@@ -29,7 +29,7 @@
2929
30struct USensorD30struct USensorD
31{31{
32 static std::string& name()32 static const std::string& name()
33 {33 {
34 static std::string s = "com.canonical.usensord";34 static std::string s = "com.canonical.usensord";
35 return s;35 return s;
@@ -37,7 +37,7 @@
3737
38 struct Haptic38 struct Haptic
39 {39 {
40 static std::string name()40 static const std::string& name()
41 {41 {
42 static std::string s = "com.canonical.usensord.haptic";42 static std::string s = "com.canonical.usensord.haptic";
43 return s;43 return s;
@@ -45,7 +45,7 @@
45 45
46 struct Vibrate46 struct Vibrate
47 {47 {
48 static std::string name()48 static const std::string& name()
49 {49 {
50 static std::string s = "Vibrate";50 static std::string s = "Vibrate";
51 return s;51 return s;
@@ -60,7 +60,7 @@
60 60
61 struct VibratePattern61 struct VibratePattern
62 {62 {
63 static std::string name()63 static const std::string& name()
64 {64 {
65 static std::string s = "VibratePattern";65 static std::string s = "VibratePattern";
66 return s;66 return s;

Subscribers

People subscribed via source and target branches