Merge lp:~vanvugt/mir/reverse-scene-traversal into lp:mir
Proposed by
Daniel van Vugt
Status: | Merged |
---|---|
Approved by: | Alexandros Frantzis |
Approved revision: | no longer in the source branch. |
Merged at revision: | 1138 |
Proposed branch: | lp:~vanvugt/mir/reverse-scene-traversal |
Merge into: | lp:mir |
Diff against target: |
187 lines (+113/-0) 6 files modified
include/server/mir/compositor/scene.h (+5/-0) include/server/mir/surfaces/surface_stack.h (+2/-0) src/server/surfaces/surface_stack.cpp (+18/-0) tests/unit-tests/compositor/test_default_display_buffer_compositor.cpp (+4/-0) tests/unit-tests/compositor/test_multi_threaded_compositor.cpp (+2/-0) tests/unit-tests/surfaces/test_surface_stack.cpp (+82/-0) |
To merge this branch: | bzr merge lp:~vanvugt/mir/reverse-scene-traversal |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Alexandros Frantzis (community) | Approve | ||
Alan Griffiths | Abstain | ||
Kevin DuBois (community) | Approve | ||
PS Jenkins bot (community) | continuous-integration | Approve | |
Review via email: mp+190315@code.launchpad.net |
Commit message
Add support for traversing the Scene from front surface to back.
This is required for occlusion detection at least (coming soon).
Description of the change
Prerequisite for fixing bug 1227739.
To post a comment you must log in.
43 + for ( auto layer = layers_ by_depth. rbegin( ) by_depth. rend()
44 + ; layer != layers_
45 + ; ++layer
46 + )
Although I don't find anything in our coding standard about this, we tend to break lines after punctuation marks.
Use std::for_each() instead?
49 + for (auto it = surfaces.rbegin(); it != surfaces.rend(); ++it)
50 + {
Use std::for_each() instead?