Merge lp://staging/~mir-team/mir/fix-1675331 into lp://staging/mir
Proposed by
Daniel van Vugt
| Status: | Merged |
|---|---|
| Approved by: | Daniel van Vugt |
| Approved revision: | no longer in the source branch. |
| Merged at revision: | 4108 |
| Proposed branch: | lp://staging/~mir-team/mir/fix-1675331 |
| Merge into: | lp://staging/mir |
| Diff against target: |
70 lines (+14/-8) 2 files modified
src/platforms/mesa/server/kms/real_kms_display_configuration.cpp (+11/-7) src/platforms/mesa/server/kms/real_kms_display_configuration.h (+3/-1) |
| To merge this branch: | bzr merge lp://staging/~mir-team/mir/fix-1675331 |
| Related bugs: |
| Reviewer | Review Type | Date Requested | Status |
|---|---|---|---|
| Alan Griffiths | Approve | ||
| Cemil Azizoglu (community) | Approve | ||
| Mir CI Bot | continuous-integration | Approve | |
|
Review via email:
|
|||
Commit message
Don't use zero (mir_display_
(LP: #1675331)
Description of the change
The offending class is not under test at all, so that's why there
is no regression test right now. If that concerns anyone we could just
temporarily revert revision 4103 instead. Either way, I don't want to
be responsible for regression testing this one.
To post a comment you must log in.
Revision history for this message
| Daniel van Vugt (vanvugt) wrote | # |
Revision history for this message
| Mir CI Bot (mir-ci-bot) wrote | # |
review:
Approve
(continuous-integration)
Preview Diff
[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
On the other hand, it looks like the only user of mir_display_