<table><tr><td style="">asemke 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/D18279">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/D18279#inline-100994">View Inline</a><span style="color: #4b4d51; font-weight: bold;">AbstractColumn.cpp:547</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"> * Returns the properties hold by this column (no, monotonic increasing, monotonic decreasing,...)</span>
</div><div style="padding: 0 8px; margin: 0 4px; background: rgba(151, 234, 151, .6);"><span style="color: #74777d"> * Will be used in XYCurve to improve the search velocity for the y value for a specific x value</span>
</div><div style="padding: 0 8px; margin: 0 4px; background: rgba(151, 234, 151, .6);"><span style="color: #74777d"> * @return</span>
</div></div></div>
<div style="margin: 8px 0; padding: 0 12px;"><p style="padding: 0; margin: 8px;">We'll need to change this "will be used" to "is used" later.</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/D18279#inline-100998">View Inline</a><span style="color: #4b4d51; font-weight: bold;">ColumnPrivate.cpp:1311</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 class="n">properties</span> <span style="color: #aa2211">=</span> <span class="n">AbstractColumn</span><span style="color: #aa2211">::</span><span class="n">Properties</span><span style="color: #aa2211">::</span><span class="n">No</span><span class="p">;</span>
</div><div style="padding: 0 8px; margin: 0 4px; background: rgba(151, 234, 151, .6);">      <span style="color: #aa4000">return</span><span class="p">;</span>
</div><div style="padding: 0 8px; margin: 0 4px; background: rgba(151, 234, 151, .6);">    <span class="p">}</span>
</div></div></div>
<div style="margin: 8px 0; padding: 0 12px;"><p style="padding: 0; margin: 8px;">can we have this return already in the if-else block before this for-loop? Then we don't need to have this check inside the loop. Also, we should maybe set propertiesAvailable to true if we return for unsupported data types so we don't call this function again and again.</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/D18279#inline-100996">View Inline</a><span style="color: #4b4d51; font-weight: bold;">ColumnPrivate.h:117</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 class="n">mutable</span> <span style="color: #aa4000">bool</span> <span class="n">propertiesAvailable</span><span class="p">{</span><span style="color: #304a96">false</span><span class="p">};</span> <span style="color: #74777d">//is 'properties' already available (true) or needs to be (re-)calculated (false)?</span>
</div><div style="padding: 0 8px; margin: 0 4px; background: rgba(151, 234, 151, .6);">  <span class="n">mutable</span> <span style="color: #aa4000">int</span> <span class="n">properties</span><span class="p">{</span><span class="n">AbstractColumn</span><span style="color: #aa2211">::</span><span class="n">Properties</span><span style="color: #aa2211">::</span><span class="n">No</span><span class="p">};</span> <span style="color: #74777d">// declares the properties of the curve (monotonic increasing/decreasing ...). Speed up algorithms</span>
</div><div style="padding: 0 8px; margin: 0 4px; "><span style="color: #a0a000">private</span><span class="p">:</span>
</div></div></div>
<div style="margin: 8px 0; padding: 0 12px;"><p style="padding: 0; margin: 8px;">int? Why not Properties?</p></div></div></div></div></div><br /><div><strong>REPOSITORY</strong><div><div>R262 LabPlot</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D18279">https://phabricator.kde.org/D18279</a></div></div><br /><div><strong>To: </strong>Murmele, asemke<br /><strong>Cc: </strong>yurchor, kde-edu, Murmele, narvaez, apol<br /></div>