Merge lp:~vanvugt/mir/fix-1407558 into lp:mir
Proposed by
Daniel van Vugt
Status: | Merged |
---|---|
Approved by: | Alan Griffiths |
Approved revision: | no longer in the source branch. |
Merged at revision: | 2193 |
Proposed branch: | lp:~vanvugt/mir/fix-1407558 |
Merge into: | lp:mir |
Prerequisite: | lp:~vanvugt/mir/wrap-the-rest |
Diff against target: |
48 lines (+9/-6) 3 files modified
src/platforms/mesa/real_kms_display_configuration.cpp (+6/-3) src/utils/out.c (+1/-1) tests/unit-tests/graphics/mesa/test_display_configuration.cpp (+2/-2) |
To merge this branch: | bzr merge lp:~vanvugt/mir/fix-1407558 |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
PS Jenkins bot (community) | continuous-integration | Approve | |
Alan Griffiths | Approve | ||
Review via email:
|
Commit message
Report vertical refresh rates to two decimal places so that one can tell
the difference between common frequencies such as 59.94 vs 60.00Hz.
(LP: #1407558)
I'm kind of undecided about this. Although slightly on the side of
preferring two decimal places because that's the precision most commonly
used in the industry. If people do round then they tend to round to the
nearest whole number. Never one decimal place.
To post a comment you must log in.
Ok