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