<div dir="ltr"><div>I've fixed the dupes in vtkPlanesIntersection and vtkImageResample.  The latter was nasty.</div><div><a href="https://gitlab.kitware.com/vtk/vtk/merge_requests/2409">https://gitlab.kitware.com/vtk/vtk/merge_requests/2409</a><br></div><div><br></div><div> - David</div><div><br></div><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Jan 20, 2017 at 8:35 AM, Andy Bauer <span dir="ltr"><<a href="mailto:andy.bauer@kitware.com" target="_blank">andy.bauer@kitware.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"><div dir="ltr"><div>Ha, mine was easy! A static const member variable with identical values :)<br><br></div>MR at <a href="https://gitlab.kitware.com/vtk/vtk/merge_requests/2407" target="_blank">https://gitlab.kitware.com/<wbr>vtk/vtk/merge_requests/2407</a>.<br></div><div class="gmail-HOEnZb"><div class="gmail-h5"><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Jan 20, 2017 at 10:21 AM, Andy Bauer <span dir="ltr"><<a href="mailto:andy.bauer@kitware.com" target="_blank">andy.bauer@kitware.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"><div dir="ltr"><div><div><div>Hi Sean,<br><br></div>Thanks for looking at this. I'll take a look at Filters/FlowPaths/vtkAbstractI<wbr>nterpolatedVelocityField.h. <br><br></div>Best,<br></div>Andy<br></div><div class="gmail-m_-6137436482817840770HOEnZb"><div class="gmail-m_-6137436482817840770h5"><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Jan 19, 2017 at 10:13 PM, 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">Hi all,<br>
<br>
I've deliberately un-suppressed cppcheck's duplInheritedMember warning and now we have 24 warnings here:<br>
<br>
<<a href="https://open.cdash.org/viewBuildError.php?type=1&buildid=4730716" rel="noreferrer" target="_blank">https://open.cdash.org/viewBu<wbr>ildError.php?type=1&buildid=47<wbr>30716</a>><br>
<br>
The warning is telling us that various classes have ivars with the same name as an ivar in their superclass. This is a pretty bad code smell IMNSHO.<br>
<br>
But it takes someone who knows the classes well to decide on the best fix...<br>
<br>
The warnings are:<br>
<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>
Common/DataModel/vtkPlanes.h:1<wbr>29: warning: The class 'vtkPlanesIntersection' defines member variable with name 'Plane' also defined in its parent class 'vtkPlanes'.<br>
<br>
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:349<wbr>: warning: The class 'vtkCompositeDataPipeline' defines member variable with name 'UpdateExtentRequest' also defined in its parent class 'vtkStreamingDemandDrivenPipel<wbr>ine'.<br>
<br>
Filters/FlowPaths/vtkAbstractI<wbr>nterpolatedVelocityField.h:211<wbr>: warning: The class 'vtkCompositeInterpolatedVeloc<wbr>ityField' defines member variable with name 'TOLERANCE_SCALE' also defined in its parent class 'vtkAbstractInterpolatedVeloci<wbr>tyField'.<br>
<br>
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>
Imaging/Core/vtkImageReslice.h<wbr>:510: warning: The class 'vtkImageResample' defines member variable with name 'OutputSpacing' also defined in its parent class 'vtkImageReslice'.<br>
<br>
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>
Interaction/Widgets/vtkBiDimen<wbr>sionalRepresentation.h:233: warning: The class 'vtkBiDimensionalRepresentatio<wbr>n2D' defines member variable with name 'P1World' also defined in its parent class 'vtkBiDimensionalRepresentatio<wbr>n'.<br>
<br>
Interaction/Widgets/vtkBiDimen<wbr>sionalRepresentation.h:234: warning: The class 'vtkBiDimensionalRepresentatio<wbr>n2D' defines member variable with name 'P2World' also defined in its parent class 'vtkBiDimensionalRepresentatio<wbr>n'.<br>
<br>
Interaction/Widgets/vtkBiDimen<wbr>sionalRepresentation.h:235: warning: The class 'vtkBiDimensionalRepresentatio<wbr>n2D' defines member variable with name 'P3World' also defined in its parent class 'vtkBiDimensionalRepresentatio<wbr>n'.<br>
<br>
Interaction/Widgets/vtkBiDimen<wbr>sionalRepresentation.h:236: warning: The class 'vtkBiDimensionalRepresentatio<wbr>n2D' defines member variable with name 'P4World' also defined in its parent class 'vtkBiDimensionalRepresentatio<wbr>n'.<br>
<br>
Interaction/Widgets/vtkBiDimen<wbr>sionalRepresentation.h:237: warning: The class 'vtkBiDimensionalRepresentatio<wbr>n2D' defines member variable with name 'P21World' also defined in its parent class 'vtkBiDimensionalRepresentatio<wbr>n'.<br>
<br>
Interaction/Widgets/vtkBiDimen<wbr>sionalRepresentation.h:238: warning: The class 'vtkBiDimensionalRepresentatio<wbr>n2D' defines member variable with name 'P43World' also defined in its parent class 'vtkBiDimensionalRepresentatio<wbr>n'.<br>
<br>
Interaction/Widgets/vtkBiDimen<wbr>sionalRepresentation.h:239: warning: The class 'vtkBiDimensionalRepresentatio<wbr>n2D' defines member variable with name 'T21' also defined in its parent class 'vtkBiDimensionalRepresentatio<wbr>n'.<br>
<br>
Interaction/Widgets/vtkBiDimen<wbr>sionalRepresentation.h:240: warning: The class 'vtkBiDimensionalRepresentatio<wbr>n2D' defines member variable with name 'T43' also defined in its parent class 'vtkBiDimensionalRepresentatio<wbr>n'.<br>
<br>
Interaction/Widgets/vtkBiDimen<wbr>sionalRepresentation.h:241: warning: The class 'vtkBiDimensionalRepresentatio<wbr>n2D' defines member variable with name 'CenterWorld' also defined in its parent class 'vtkBiDimensionalRepresentatio<wbr>n'.<br>
<br>
Interaction/Widgets/vtkBiDimen<wbr>sionalRepresentation.h:242: warning: The class 'vtkBiDimensionalRepresentatio<wbr>n2D' defines member variable with name 'StartEventPositionWorld' also defined in its parent class 'vtkBiDimensionalRepresentatio<wbr>n'.<br>
<br>
Rendering/Core/vtkColorTransfe<wbr>rFunction.h:398: warning: The class 'vtkDiscretizableColorTransfer<wbr>Function' defines member variable with name 'BuildTime' also defined in its parent class 'vtkColorTransferFunction'.<br>
<br>
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></blockquote></div></div></div></div></blockquote></div></div></div></div></blockquote></div></div></div>