[vtk-developers] Bug in vtkGraph::RemoveVertexInternal

[Jeff Baumes]

Fixed now:

commit 9c30d5b965c6c7273ceda6e5f3af2d2691b0b1cf
Author: Jeffrey Baumes <jeff.baumes at kitware.com>
Date:   Mon Jul 5 16:45:39 2010 -0400

    Copying graph point correctly.

    Thanks to Nasztanovics Ferenc for pointing out this bug.

diff --git a/Filtering/vtkGraph.cxx b/Filtering/vtkGraph.cxx
index cda39ba..f9e8999 100644
--- a/Filtering/vtkGraph.cxx
+++ b/Filtering/vtkGraph.cxx
@@ -1550,7 +1550,7 @@ void vtkGraph::RemoveVertexInternal(vtkIdType v, bool
dire
   if (this->Points)
     {
     double x[3];
-    this->Points->GetPoint(lv);
+    this->Points->GetPoint(lv, x);
     this->Points->SetPoint(v, x);
     this->Points->SetNumberOfPoints(lv);
     }

Jeff

On Fri, Jul 2, 2010 at 4:09 AM, Nasztanovics Ferenc <naszta at naszta.hu>wrote:

> Now in code:
>
>  if (this->Points)
>    {
>    double x[3];
>    this->Points->GetPoint(lv);
>    this->Points->SetPoint(v, x);
>    this->Points->SetNumberOfPoints(lv);
>    }
>
> Should change to:
>
>  if (this->Points)
>    {
>    double x[3];
>    this->Points->GetPoint(lv,x);
>    this->Points->SetPoint(v, x);
>    this->Points->SetNumberOfPoints(lv);
>    }
>
> Best regards,
>
> Naszta
> _______________________________________________
> Powered by www.kitware.com
>
> Visit other Kitware open-source projects at
> http://www.kitware.com/opensource/opensource.html
>
> Follow this link to subscribe/unsubscribe:
> http://www.vtk.org/mailman/listinfo/vtk-developers
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://public.kitware.com/pipermail/vtk-developers/attachments/20100705/7647b975/attachment.html>