<table><tr><td style="">Murmele 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/D23419">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/D23419#inline-132363">View Inline</a><span style="color: #4b4d51; font-weight: bold;">asemke</span> wrote in <span style="color: #4b4d51; font-weight: bold;">Project.cpp:268</span></div>
<div style="margin: 8px 0; padding: 0 12px; color: #74777D;"><p style="padding: 0; margin: 8px;">where is the pointer column defined?</p></div></div>
<div style="margin: 8px 0; padding: 0 12px;"><p style="padding: 0; margin: 8px;">at the beginning of the function:</p>

<p style="padding: 0; margin: 8px;">void Project::descriptionChanged(const AbstractAspect* aspect) {<br />
        if (isLoading())</p>

<div class="remarkup-code-block" style="margin: 12px 0;" data-code-lang="text" data-sigil="remarkup-code-block"><pre class="remarkup-code" style="font: 11px/15px "Menlo", "Consolas", "Monaco", monospace; padding: 12px; margin: 0; background: rgba(71, 87, 120, 0.08);">              return;</pre></div>

<p style="padding: 0; margin: 8px;">if (this != aspect) {</p>

<div class="remarkup-code-block" style="margin: 12px 0;" data-code-lang="text" data-sigil="remarkup-code-block"><pre class="remarkup-code" style="font: 11px/15px "Menlo", "Consolas", "Monaco", monospace; padding: 12px; margin: 0; background: rgba(71, 87, 120, 0.08);">              const AbstractColumn* column = dynamic_cast<const AbstractColumn*>(aspect);</pre></div></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/D23419#inline-132362">View Inline</a><span style="color: #4b4d51; font-weight: bold;">asemke</span> wrote in <span style="color: #4b4d51; font-weight: bold;">Spreadsheet.cpp:908</span></div>
<div style="margin: 8px 0; padding: 0 12px; color: #74777D;"><p style="padding: 0; margin: 8px;">where is this signal consumed?</p></div></div>
<div style="margin: 8px 0; padding: 0 12px;"><p style="padding: 0; margin: 8px;">In macrosXYCurve.h<br />
it is connected to ColumnAboutToBeRemoved</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/D23419#inline-132366">View Inline</a><span style="color: #4b4d51; font-weight: bold;">asemke</span> wrote in <span style="color: #4b4d51; font-weight: bold;">XYCurve.h:136</span></div>
<div style="margin: 8px 0; padding: 0 12px; color: #74777D;"><p style="padding: 0; margin: 8px;">do I see it correctly that with this patch the connection of a column with a curve is lost if the column is renamed? At the moment it's possible to rename the columns (for example from "my great y-data" to "my great and fancy y-data") without affecting the curves - is this not possible anymore since we work now with your patch with names and not with the pointers anymore?</p></div></div>
<div style="margin: 8px 0; padding: 0 12px;"><p style="padding: 0; margin: 8px;">This makes the call of reset() from the column. This signal is connected to ColumnAboutToBeRemoved in XYCurve<br />
see macrosXYCurve.h</p>

<p style="padding: 0; margin: 8px;">template instead of a macro would be better, but I don't know how to create one in this case</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/D23419">https://phabricator.kde.org/D23419</a></div></div><br /><div><strong>To: </strong>Murmele, asemke<br /><strong>Cc: </strong>yurchor, kde-edu, narvaez, apol<br /></div>