Merge lp:~alan-griffiths/miral/fix-1645284 into lp:miral

Proposed by Alan Griffiths on 2016-11-28
Status: Merged
Approved by: Gerry Boland on 2016-11-29
Approved revision: 458
Merged at revision: 457
Proposed branch: lp:~alan-griffiths/miral/fix-1645284
Merge into: lp:miral
Diff against target: 12 lines (+2/-1)
1 file modified
miral/active_outputs.cpp (+2/-1)
To merge this branch: bzr merge lp:~alan-griffiths/miral/fix-1645284
Reviewer Review Type Date Requested Status
Gerry Boland (community) 2016-11-28 Approve on 2016-11-29
Review via email: mp+311943@code.launchpad.net

Commit message

[libmiral] Register ActiveOutputsMonitor with the "display configuration observer registrar" before server init starts

To post a comment you must log in.
Gerry Boland (gerboland) wrote :

Ok.
Mir API disappointing to require this in the preinit

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'miral/active_outputs.cpp'
2--- miral/active_outputs.cpp 2016-11-28 09:19:40 +0000
3+++ miral/active_outputs.cpp 2016-11-28 14:52:28 +0000
4@@ -107,7 +107,8 @@
5 #if MIR_SERVER_VERSION < MIR_VERSION_NUMBER(0, 26, 0)
6 server.override_the_display_configuration_report([this]{ return self; });
7 #else
8- server.the_display_configuration_observer_registrar()->register_interest(self);
9+ server.add_pre_init_callback([this, &server]
10+ { server.the_display_configuration_observer_registrar()->register_interest(self); });
11 #endif
12 }
13

Subscribers

People subscribed via source and target branches