<div dir="ltr"><div><div>Hi Sean,<br><br></div>I volunteer to look at #3. For some reason I missed this email thread -- thanks for your persistence!<br><br></div>Best,<br><div>Andy<br></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Dec 3, 2014 at 12:08 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:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Thanks David for fixing #2, and thanks Dan Lipsa for doing #3.<br>
<br>
Anyone know vtkPolyLine.cxx to look at #1?<br>
<br>
And to add a #4:<br>
<br>
Common/ExecutionModel/vtkCompositeDataPipeline.cxx:1047:5: Value stored to 'outputPort' is never read<br>
<br>
The whole block of code seems pointless because nothing actually uses 'outputPort' after setting it.<br>
<br>
// Save the information about COMPOSITE_INDICES() as needed in the data<br>
// object.<br>
int outputPort = 0;<br>
if(request->Has(FROM_OUTPUT_PORT()))<br>
  {<br>
  outputPort = request->Get(FROM_OUTPUT_PORT());<br>
  outputPort = (outputPort >= 0 ? outputPort : 0);<br>
  }<br>
<br>
Thanks,<br>
<br>
Sean<br>
<br>
<br>
On Fri, 21 Nov 2014 10:20:54 -0700, David Gobbi said:<br>
<br>
>Hi Sean,<br>
><br>
>I can fix vtkCellPicker.cxx by making the code able to handle the<br>
>case that property is null.<br>
><br>
> - David<br>
><br>
><br>
>On Fri, Nov 21, 2014 at 9:46 AM, Sean McBride <<a href="mailto:sean@rogue-research.com">sean@rogue-research.com</a>><br>
>wrote:<br>
><br>
>> Hi all,<br>
>><br>
>> Occasionally, I try to fix some of the hundreds of clang static analyzer<br>
>> warnings in VTK.  Some are easy, some are false positives, and some are<br>
>> real problems that I don't know how to fix.  I'd like to appeal to you all<br>
>> to help fix these 3:<br>
>><br>
>> 1)<br>
>> file: Common/DataModel/vtkPolyLine.cxx<br>
>> method: vtkPolyLine::GenerateSlidingNormals<br>
>> lines: 227-237<br>
>> problem: dead code.  The "if ( largeRotation )" branch sets "theta" then<br>
>> nothing uses it.<br>
>><br>
>> 2)<br>
>> file: Rendering/Core/vtkCellPicker.cxx<br>
>> method: vtkCellPicker::IntersectVolumeWithLine<br>
>> line: 691<br>
>> problem: the 'property' pointer is initialized to null, and if the<br>
>> subsequent 'if' is not entered, it's certainly still null at line 691 and<br>
>> dereferenced.<br>
>><br>
>> 3)<br>
>> file: Common/DataModel/vtkKdTree.cxx<br>
>> line: 3188<br>
>> method: vtkKdTree::GenerateRepresentationWholeSpace<br>
>> problem: the first time through the 'for' loop always does "1 << -1".  You<br>
>> can't left shift by a negative count.<br>
>><br>
>> I don't want to blindly delete dead code that maybe shouldn't be dead, or<br>
>> just throw in null checks, so hopefully someone knows this code!<br>
<br>
<br>
_______________________________________________<br>
Powered by <a href="http://www.kitware.com" target="_blank">www.kitware.com</a><br>
<br>
Visit other Kitware open-source projects at <a href="http://www.kitware.com/opensource/opensource.html" target="_blank">http://www.kitware.com/opensource/opensource.html</a><br>
<br>
Follow this link to subscribe/unsubscribe:<br>
<a href="http://public.kitware.com/mailman/listinfo/vtk-developers" target="_blank">http://public.kitware.com/mailman/listinfo/vtk-developers</a><br>
<br>
</blockquote></div><br></div>