<table><tr><td style="">cfeck added inline comments.
</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/D12388">View Revision</a></tr></table><br /><div><strong>INLINE COMMENTS</strong><div><div style="margin: 6px 0 12px 0;"><div style="border: 1px solid #C7CCD9; border-radius: 3px;"><div style="padding: 0; background: #F7F7F7; border-color: #e3e4e8; border-style: solid; border-width: 0 0 1px 0; margin: 0;"><div style="color: #74777d; background: #eff2f4; padding: 6px 8px; overflow: hidden;"><a style="float: right; text-decoration: none;" href="https://phabricator.kde.org/D12388#inline-62835">View Inline</a><span style="color: #4b4d51; font-weight: bold;">outputconfiguration.h:211</span></div>
<div style="font: 11px/15px "Menlo", "Consolas", "Monaco", monospace; white-space: pre-wrap; clear: both; padding: 4px 0; margin: 0;"><div style="padding: 0 8px; margin: 0 4px; background: rgba(151, 234, 151, .6);"><span style="color: #74777d">     */</span>
</div><div style="padding: 0 8px; margin: 0 4px; background: rgba(151, 234, 151, .6);">    <span style="color: #aa4000">void</span> <span style="color: #004012">setColorCurves</span><span class="p">(</span><span class="n">OutputDevice</span> <span style="color: #aa2211">*</span><span class="n">outputdevice</span><span class="p">,</span> <span class="n">QVector</span><span style="color: #aa2211"><</span><span class="n">quint16</span><span style="color: #aa2211">></span> <span class="n">red</span><span class="p">,</span> <span class="n">QVector</span><span style="color: #aa2211"><</span><span class="n">quint16</span><span style="color: #aa2211">></span> <span class="n">green</span><span class="p">,</span> <span class="n">QVector</span><span style="color: #aa2211"><</span><span class="n">quint16</span><span style="color: #aa2211">></span> <span class="n">blue</span><span class="p">);</span>
</div></div></div>
<div style="margin: 8px 0; padding: 0 12px;"><p style="padding: 0; margin: 8px;">The documentation could state how many elements need to be in the vectors. Ideally, every component could have any number of elements, and if there are too few, the other elements are interpolated.</p>

<p style="padding: 0; margin: 8px;">But it is probably simpler to just state "The number of elements in each vector must be 256 or 1024, depending on the depth of the framebuffer (24 bits or 30 bits)."</p></div></div><br /><div style="border: 1px solid #C7CCD9; border-radius: 3px;"><div style="padding: 0; background: #F7F7F7; border-color: #e3e4e8; border-style: solid; border-width: 0 0 1px 0; margin: 0;"><div style="color: #74777d; background: #eff2f4; padding: 6px 8px; overflow: hidden;"><a style="float: right; text-decoration: none;" href="https://phabricator.kde.org/D12388#inline-62839">View Inline</a><span style="color: #4b4d51; font-weight: bold;">outputconfiguration_interface.cpp:217</span></div>
<div style="font: 11px/15px "Menlo", "Consolas", "Monaco", monospace; white-space: pre-wrap; clear: both; padding: 4px 0; margin: 0;"><div style="padding: 0 8px; margin: 0 4px; background: rgba(151, 234, 151, .6);">        <span style="color: #aa4000">while</span><span class="p">((</span><span style="color: #aa4000">char</span><span style="color: #aa2211">*</span><span class="p">)</span><span class="n">pos</span> <span style="color: #aa2211"><</span> <span class="p">(</span><span style="color: #aa4000">char</span><span style="color: #aa2211">*</span><span class="p">)</span><span class="n">array</span><span style="color: #aa2211">-></span><span class="n">data</span> <span style="color: #aa2211">+</span> <span class="n">array</span><span style="color: #aa2211">-></span><span class="n">size</span><span class="p">)</span> <span class="p">{</span>
</div><div style="padding: 0 8px; margin: 0 4px; background: rgba(151, 234, 151, .6);">            <span style="color: #74777d">// TODO: check value not bigger than size (i.e. for 8bit display < 255)?</span>
</div><div style="padding: 0 8px; margin: 0 4px; background: rgba(151, 234, 151, .6);">            <span class="n">v</span><span style="color: #aa2211">-></span><span class="n">append</span><span class="p">(</span><span style="color: #aa2211">*</span><span class="n">pos</span><span class="p">);</span>
</div></div></div>
<div style="margin: 8px 0; padding: 0 12px;"><p style="padding: 0; margin: 8px;">X11 uses 16 bit values even for 24 bit screens, because the actual values that are sent to the DAC can have higher precision than the screen pixmap. I doubt that Wayland has reduced the precision, so I don't think checking values here is right.</p></div></div></div></div></div><br /><div><strong>REPOSITORY</strong><div><div>R127 KWayland</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D12388">https://phabricator.kde.org/D12388</a></div></div><br /><div><strong>To: </strong>romangg, Frameworks<br /><strong>Cc: </strong>cfeck, michaelh, bruns<br /></div>