<div dir="ltr"><div><div><div>The issue with the vtkScatterPlotMatrix class is interesting.<br><br></div>The ivar being reported as a duplicate is not, in fact, a duplicate: both the parent class vtkChartMatrix and the child vtkScatterPlotMatrix define an internal class named PIMPL. And the ivar "Private" is an instance of that class.<br><br></div>I was not aware of the use of private implementations that PIMPL represents within the VTK context.<br><br>Now, I could rename the child class' PIMPL class or its instance.<br><br></div><div>I'd like to have some advice on this. Renaming the parents' does not seem a better idea. <br></div><div><br></div>BTW, I'm interested in knowing about the (need of/convenience) PIMPL classes. Is the use of such PIMPL classes documented somewhere?<br><div><div><br></div><div>Thanks,<br></div><div>JON HAITZ<br><br clear="all"></div><div><div><div><div><div class="gmail_signature"><div dir="ltr"><div><div dir="ltr"><div><div dir="ltr"><div><div dir="ltr"><div dir="ltr"><table style="color:rgb(0,0,0);font-family:"times new roman";font-size:medium" cellspacing="0" cellpadding="0" border="0" align="center"><tbody><tr><td></td></tr></tbody></table><pre style="color:rgb(0,0,0);white-space:pre-wrap">--<br><br></pre><div><div dir="ltr"></div></div></div></div></div></div></div></div></div></div></div></div>
<br><div class="gmail_quote">---------- Forwarded message ----------<br>From: <b class="gmail_sendername">Jon Haitz Legarreta</b> <span dir="ltr"><<a href="mailto:jhlegarreta@vicomtech.org">jhlegarreta@vicomtech.org</a>></span><br>Date: 5 May 2017 at 16:30<br>Subject: Re: [vtk-developers] help with cppcheck duplInheritedMember warnings<br>To: Sean McBride <<a href="mailto:sean@rogue-research.com">sean@rogue-research.com</a>><br>Cc: VTK Developers <<a href="mailto:vtk-developers@vtk.org">vtk-developers@vtk.org</a>><br><br><br><div dir="ltr">You're welcome.<br><br>I'll adopt the vtkPlotSurface, <span class="gmail-m_-5057617569639123648gmail-im gmail-m_-5057617569639123648gmail-HOEnZb">vtkScatterPlotMatrix</span>, and vtkBiDimensionalRepresentation<wbr>2D (there was a member in this last that escaped my patch set) class issues.<br><br>JON HAITZ<br clear="all"><div class="gmail_extra"><div><div class="gmail-m_-5057617569639123648gmail_signature"><div dir="ltr"><div><div dir="ltr"><div><div dir="ltr"><div><div dir="ltr"><div dir="ltr"><table style="color:rgb(0,0,0);font-family:"times new roman";font-size:medium" cellspacing="0" cellpadding="0" border="0" align="center"><tbody><tr><td></td></tr></tbody></table><pre style="color:rgb(0,0,0);white-space:pre-wrap">--<br><br></pre><div><div dir="ltr"></div></div></div></div></div></div></div></div></div></div></div></div><div><div class="gmail-h5">
<br><div class="gmail_quote">On 5 May 2017 at 16:09, Sean McBride <span dir="ltr"><<a href="mailto:sean@rogue-research.com" target="_blank">sean@rogue-research.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><span class="gmail-m_-5057617569639123648gmail-">On Wed, 3 May 2017 09:25:09 -0600, David Gobbi said:<br>
<br>
>I don't think suppressing them is a good idea, redefined member variables<br>
>are strongly indicative of real bugs.<br>
<br>
</span>Thanks David & Jon!  We are down to 10 now:<br>
<span class="gmail-m_-5057617569639123648gmail-im gmail-m_-5057617569639123648gmail-HOEnZb"><br>
Charts/Core/vtkPlot3D.h:152: warning: The class 'vtkPlotSurface' defines member variable with name 'Colors' also defined in its parent class 'vtkPlot3D'.<br>
<br>
Charts/Core/vtkChartMatrix.h:1<wbr>55: warning: The class 'vtkScatterPlotMatrix' defines member variable with name 'Private' also defined in its parent class 'vtkChartMatrix'.<br>
<br>
</span><span class="gmail-m_-5057617569639123648gmail-im gmail-m_-5057617569639123648gmail-HOEnZb">Common/DataModel/vtkGenericCel<wbr>lTessellator.h:211: warning: The class 'vtkSimpleCellTessellator' defines member variable with name 'DataSet' also defined in its parent class 'vtkGenericCellTessellator'.<br>
<br>
Common/ExecutionModel/vtkStrea<wbr>mingDemandDrivenPipeline.h:<wbr>349: warning: The class 'vtkCompositeDataPipeline' defines member variable with name 'UpdateExtentRequest' also defined in its parent class 'vtkStreamingDemandDrivenPipel<wbr>ine'.<br>
<br>
</span><span class="gmail-m_-5057617569639123648gmail-im gmail-m_-5057617569639123648gmail-HOEnZb">IO/SQL/vtkTableToDatabaseWrite<wbr>r.h:75: warning: The class 'vtkTableToSQLiteWriter' defines member variable with name 'Input' also defined in its parent class 'vtkTableToDatabaseWriter'.<br>
<br>
</span><span class="gmail-m_-5057617569639123648gmail-im gmail-m_-5057617569639123648gmail-HOEnZb">Interaction/Widgets/vtkBiDimen<wbr>sionalRepresentation.h:213: warning: The class 'vtkBiDimensionalRepresentatio<wbr>n2D' defines member variable with name 'Modifier' also defined in its parent class 'vtkBiDimensionalRepresentatio<wbr>n'.<br>
<br>
</span><div class="gmail-m_-5057617569639123648gmail-HOEnZb"><div class="gmail-m_-5057617569639123648gmail-h5">Rendering/OpenGL/vtkStandardPo<wbr>lyDataPainter.h:86: warning: The class 'vtkHardwareSelectionPolyDataP<wbr>ainter' defines member variable with name 'TotalCells' also defined in its parent class 'vtkStandardPolyDataPainter'.<br>
<br>
Rendering/OpenGL/vtkXRenderWin<wbr>dowInteractor.h:198: warning: The class 'vtkXRenderWindowTclInteractor<wbr>' defines member variable with name 'Internal' also defined in its parent class 'vtkXRenderWindowInteractor'.<br>
<br>
Views/Infovis/vtkRenderView.h:<wbr>289: warning: The class 'vtkGraphLayoutView' defines member variable with name 'Interacting' also defined in its parent class 'vtkRenderView'.<br>
<br>
Views/Infovis/vtkRenderedRepre<wbr>sentation.h:96: warning: The class 'vtkRenderedTreeAreaRepresenta<wbr>tion' defines member variable with name 'Implementation' also defined in its parent class 'vtkRenderedRepresentation'.<br>
<br>
<br>
<br>
______________________________<wbr>_________________<br>
Powered by <a href="http://www.kitware.com" rel="noreferrer" target="_blank">www.kitware.com</a><br>
<br>
Visit other Kitware open-source projects at <a href="http://www.kitware.com/opensource/opensource.html" rel="noreferrer" target="_blank">http://www.kitware.com/opensou<wbr>rce/opensource.html</a><br>
<br>
Search the list archives at: <a href="http://markmail.org/search/?q=vtk-developers" rel="noreferrer" target="_blank">http://markmail.org/search/?q=<wbr>vtk-developers</a><br>
<br>
Follow this link to subscribe/unsubscribe:<br>
<a href="http://public.kitware.com/mailman/listinfo/vtk-developers" rel="noreferrer" target="_blank">http://public.kitware.com/mail<wbr>man/listinfo/vtk-developers</a><br>
<br>
</div></div></blockquote></div><br></div></div></div></div>
</div><br></div><br></div></div></div></div>