<table><tr><td style="">asemke 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/D17946">View Revision</a></tr></table><br /><div><div><p>The patch look ok. Please check the behavior caused by if (symbolsStyle != Symbol::NoSymbols || valuesType != XYCurve::NoValues )  again and land this nice improvement.</p></div></div><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/D17946#inline-116389">View Inline</a><span style="color: #4b4d51; font-weight: bold;">Murmele</span> wrote in <span style="color: #4b4d51; font-weight: bold;">XYCurve.cpp:918</span></div>
<div style="margin: 8px 0; padding: 0 12px; color: #74777D;"><p style="padding: 0; margin: 8px;">Because the logical points are not needed, if no symbols nor values are visible. To save computational power</p></div></div>
<div style="margin: 8px 0; padding: 0 12px;"><p style="padding: 0; margin: 8px;">symbolPointsLogical is also used in updateErrorBars(), updateDropLines() and in updateFilling(). So, even if we don't show any symbols and values, we still need these points for error bars, for the drop lines and for the filling below/above the curve.</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/D17946">https://phabricator.kde.org/D17946</a></div></div><br /><div><strong>To: </strong>Murmele, asemke<br /><strong>Cc: </strong>kde-edu, Murmele, narvaez, apol<br /></div>