Mir

Merge lp:~afrantzis/mir/ci-failure-experiment-1 into lp:mir

Proposed by Alexandros Frantzis
Status: Rejected
Rejected by: Alexandros Frantzis
Proposed branch: lp:~afrantzis/mir/ci-failure-experiment-1
Merge into: lp:mir
Diff against target: 16 lines (+2/-2)
1 file modified
src/platforms/android/server/android_buffer_allocator.cpp (+2/-2)
To merge this branch: bzr merge lp:~afrantzis/mir/ci-failure-experiment-1
Reviewer Review Type Date Requested Status
PS Jenkins bot (community) continuous-integration Approve
Alexandros Frantzis (community) Disapprove
Review via email: mp+278344@code.launchpad.net

Commit message

DO NOT MERGE, experimental

android: Don't close the gralloc device

Description of the change

DO NOT MERGE, experimental

android: Don't close the gralloc device

To post a comment you must log in.
Revision history for this message
Alexandros Frantzis (afrantzis) :
review: Disapprove
3123. By Alexandros Frantzis

Avoid build error

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

Unmerged revisions

3123. By Alexandros Frantzis

Avoid build error

3122. By Alexandros Frantzis

Don't close the gralloc device

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'src/platforms/android/server/android_buffer_allocator.cpp'
2--- src/platforms/android/server/android_buffer_allocator.cpp 2015-08-07 15:55:22 +0000
3+++ src/platforms/android/server/android_buffer_allocator.cpp 2015-11-23 17:10:59 +0000
4@@ -38,10 +38,10 @@
5 {
6 struct AllocDevDeleter
7 {
8- void operator()(alloc_device_t* t)
9+ void operator()(alloc_device_t* /*t*/)
10 {
11 /* android takes care of delete for us */
12- t->common.close((hw_device_t*)t);
13+ //t->common.close((hw_device_t*)t);
14 }
15 };
16 }

Subscribers

People subscribed via source and target branches