Mir

Merge lp:~raof/mir/fix-clanger into lp:~mir-team/mir/trunk

Proposed by Chris Halse Rogers
Status: Merged
Approved by: Alan Griffiths
Approved revision: no longer in the source branch.
Merged at revision: 809
Proposed branch: lp:~raof/mir/fix-clanger
Merge into: lp:~mir-team/mir/trunk
Diff against target: 10 lines (+1/-0)
1 file modified
3rd_party/android-deps/std/properties.h (+1/-0)
To merge this branch: bzr merge lp:~raof/mir/fix-clanger
Reviewer Review Type Date Requested Status
Alan Griffiths Approve
PS Jenkins bot (community) continuous-integration Approve
Robert Ancell Approve
Review via email: mp+172956@code.launchpad.net

Commit message

Fix clearly-broken code, for which clang produced deliberately broken output

Description of the change

Fix unit-tests when built with clang.

Actually, this also fixes unit-tests when built with g++, but
we obviously never actually checked the return value from
property_get().

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
Robert Ancell (robert-ancell) :
review: Approve
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Approve (continuous-integration)
Revision history for this message
Alan Griffiths (alan-griffiths) wrote :

LGTM

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file '3rd_party/android-deps/std/properties.h'
2--- 3rd_party/android-deps/std/properties.h 2013-03-13 04:54:15 +0000
3+++ 3rd_party/android-deps/std/properties.h 2013-07-04 06:14:26 +0000
4@@ -42,6 +42,7 @@
5 value[0] = '\0';
6 len = 0;
7 }
8+ return len;
9 }
10 }
11

Subscribers

People subscribed via source and target branches