D14418: Add test for edid parsing and fix reading gamma
Roman Gilg
noreply at phabricator.kde.org
Fri Jul 27 14:22:43 BST 2018
romangg accepted this revision.
romangg added a comment.
This revision is now accepted and ready to land.
> The old code's output was "2.0" instead of "2.2": integer division first, then cast.
Yes, I know. But my train of thoughts was that the original value was multiplied by 100 when the raw edid data was produced, so it should be divisible by 100 again. But the original value was in general already a float, so this is indeed a bug.
REPOSITORY
R110 KScreen Library
REVISION DETAIL
https://phabricator.kde.org/D14418
To: gladhorn, #plasma, romangg
Cc: romangg, plasma-devel, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20180727/40ba46d7/attachment.html>
More information about the Plasma-devel
mailing list