<table><tr><td style="">gladhorn added a comment.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D14418">View Revision</a></tr></table><br /><div><div><blockquote style="border-left: 3px solid #8C98B8;
          color: #6B748C;
          font-style: italic;
          margin: 4px 0 12px 0;
          padding: 8px 12px;
          background-color: #F8F9FC;">
<div style="font-style: normal;
          padding-bottom: 4px;">In <a href="https://phabricator.kde.org/D14418#299028" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">D14418#299028</a>, <a href="https://phabricator.kde.org/p/romangg/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;">@romangg</a> wrote:</div>
<div style="margin: 0;
          padding: 0;
          border: 0;
          color: rgb(107, 116, 140);"><p>I'm not sure if you could really say that the current code is with a bug, since <tt style="background: #ebebeb; font-size: 13px;">data[GCM_EDID_OFFSET_GAMMA]</tt> should be dividable by 100. I.e. the result is again an integer and nothing gets truncated. But we can still go with the updated code, which looks nicer. Maybe one could add a comment why divide by 100 and add 1.</p>

<p>In regards to the parser test in the edid test:</p>

<ul class="remarkup-list">
<li class="remarkup-list-item">How did you choose the four brands/models to test?</li>
<li class="remarkup-list-item">Pls order the rows alphabetically by brand/model name.</li>
</ul></div>
</blockquote>

<p>The old code's output was "2.0" instead of "2.2": integer division first, then cast.</p>

<p>I randomly took the four edid's that were somewhat different from each other that I had available (I had a few more, but they were pretty much the same, so I chose different manufacturers).</p>

<p>I don't see what sorting alphabetically gains us.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R110 KScreen Library</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D14418">https://phabricator.kde.org/D14418</a></div></div><br /><div><strong>To: </strong>gladhorn, Plasma, romangg<br /><strong>Cc: </strong>plasma-devel, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart<br /></div>