Revision: 56809
          http://sourceforge.net/p/brlcad/code/56809
Author:   starseeker
Date:     2013-08-13 19:31:54 +0000 (Tue, 13 Aug 2013)
Log Message:
-----------
Add a note to check out the diamond inheritance fix recently made to stepcode - 
may relate to this edge_loop issue.

Modified Paths:
--------------
    brlcad/trunk/src/conv/step/ON_Brep.cpp

Modified: brlcad/trunk/src/conv/step/ON_Brep.cpp
===================================================================
--- brlcad/trunk/src/conv/step/ON_Brep.cpp      2013-08-13 19:23:28 UTC (rev 
56808)
+++ brlcad/trunk/src/conv/step/ON_Brep.cpp      2013-08-13 19:31:54 UTC (rev 
56809)
@@ -317,12 +317,14 @@
                }
        }
 
-       // loop topology.  STEP defines loops with 3D edge curves, but 
OpenNURBS describes ON_BrepLoops with 
+       // loop topology.  STEP defines loops with 3D edge curves, but 
OpenNURBS describes ON_BrepLoops with
        // 2d trim curves.  So for a given loop, we need to interate over the 
trims, for each trim get the
        // index of its corresponding edge, and add that edge to the _edge_list 
for the loop.
        //
        // TODO - for some reason _edge_list is initialized to a NULL pointer 
in the ObjCreate process, and
-       // it's protected so I can't manually repair it afterward.
+       // it's protected so I can't manually repair it afterward.  Looks like 
this is supposed to inherit the
+       // initialization of _edge_list from SdaiPath, but is only inheriting 
from SdaiLoop.  May be related to
+       // the recent proposed patch by Mark in stepcode to enable diamond 
inheritance - will have to experiment.
        /*
        for (int i = 0; i < brep->m_L.Count(); ++i) {
                ON_BrepLoop *loop= &(brep->m_L[i]);

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


------------------------------------------------------------------------------
Get 100% visibility into Java/.NET code with AppDynamics Lite!
It's a free troubleshooting tool designed for production.
Get down to code-level detail for bottlenecks, with <2% overhead. 
Download for free and get started troubleshooting in minutes. 
http://pubads.g.doubleclick.net/gampad/clk?id=48897031&iu=/4140/ostg.clktrk
_______________________________________________
BRL-CAD Source Commits mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/brlcad-commits

Reply via email to