Merge lp:~mir-team/unity-system-compositor/cookie-entropy-workaround into lp:unity-system-compositor

Proposed by Brandon Schaefer
Status: Merged
Approved by: Alexandros Frantzis
Approved revision: 275
Merged at revision: 277
Proposed branch: lp:~mir-team/unity-system-compositor/cookie-entropy-workaround
Merge into: lp:unity-system-compositor
Diff against target: 13 lines (+3/-1)
1 file modified
src/server.cpp (+3/-1)
To merge this branch: bzr merge lp:~mir-team/unity-system-compositor/cookie-entropy-workaround
Reviewer Review Type Date Requested Status
PS Jenkins bot (community) continuous-integration Approve
Alexandros Frantzis (community) Approve
Review via email: mp+285402@code.launchpad.net

Commit message

Set the size of the vector to the expected size internally. This is just a workaround until we fix some things in mir!

Description of the change

Set the size of the vector to the expected size internally. This is just a workaround until we fix some things in mir!

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
Alexandros Frantzis (afrantzis) wrote :

OK as a temporary fix, as long as we remember to update this when we ship the Mir fix.

review: Approve
Revision history for this message
PS Jenkins bot (ps-jenkins) :
review: Approve (continuous-integration)

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'src/server.cpp'
2--- src/server.cpp 2016-01-27 14:58:39 +0000
3+++ src/server.cpp 2016-02-08 19:17:37 +0000
4@@ -110,7 +110,9 @@
5
6 std::vector<uint8_t> serialize() const override
7 {
8- return std::vector<uint8_t>();
9+ // FIXME Workaround until mir fixes its internal message bliting
10+ // This size needs to equal mir/src/include/cookie/mir/cookie/blob.h::default_blob_size
11+ return std::vector<uint8_t>(29);
12 }
13 };
14

Subscribers

People subscribed via source and target branches