Revision: 56020
          http://sourceforge.net/p/brlcad/code/56020
Author:   phoenixyjll
Date:     2013-07-12 06:13:29 +0000 (Fri, 12 Jul 2013)
Log Message:
-----------
ON_PolylineCurve::IsArc() always returns false, so using it is not sufficient, 
and the elliptical arc fitting code below will work for this.

Modified Paths:
--------------
    brlcad/trunk/src/libbrep/intersect.cpp

Modified: brlcad/trunk/src/libbrep/intersect.cpp
===================================================================
--- brlcad/trunk/src/libbrep/intersect.cpp      2013-07-12 06:11:33 UTC (rev 
56019)
+++ brlcad/trunk/src/libbrep/intersect.cpp      2013-07-12 06:13:29 UTC (rev 
56020)
@@ -1894,15 +1894,6 @@
        return linecurve;
     }
 
-    // Arc fitting (including circle)
-    ON_Arc arc;
-    if (in->IsArc(&ON_xy_plane, &arc, fitting_tolerance)) {
-       if (delete_curve) delete in;
-       ON_ArcCurve* arccurve = new ON_ArcCurve(arc);
-       arccurve->ChangeDimension(in->Dimension());
-       return arccurve;
-    }
-
     // Conic fitting (ellipse, parabola, hyperbola)
     // It's only meaningful to fit the curve when it's a complex one
     // For a polyline curve, the number of points should not be less than 10.

This was sent by the SourceForge.net collaborative development platform, the 
world's largest Open Source development site.


------------------------------------------------------------------------------
See everything from the browser to the database with AppDynamics
Get end-to-end visibility with application monitoring from AppDynamics
Isolate bottlenecks and diagnose root cause in seconds.
Start your free trial of AppDynamics Pro today!
http://pubads.g.doubleclick.net/gampad/clk?id=48808831&iu=/4140/ostg.clktrk
_______________________________________________
BRL-CAD Source Commits mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/brlcad-commits

Reply via email to