Revision: 73930
          http://sourceforge.net/p/brlcad/code/73930
Author:   starseeker
Date:     2019-09-14 21:32:44 +0000 (Sat, 14 Sep 2019)
Log Message:
-----------
Got perf on a run - note the hot spot.

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

Modified: brlcad/trunk/src/libbrep/cdt_mesh.cpp
===================================================================
--- brlcad/trunk/src/libbrep/cdt_mesh.cpp       2019-09-14 16:34:14 UTC (rev 
73929)
+++ brlcad/trunk/src/libbrep/cdt_mesh.cpp       2019-09-14 21:32:44 UTC (rev 
73930)
@@ -1474,6 +1474,11 @@
     return (long)(normals.size() - 1);
 }
 
+// TODO - perf reports a huge percentage of time is spend dealing with
+// tri_add - we should only need the uniqueness guarantee for triangles
+// associated with singularities, so it might be worth trying to make this
+// more nuanced and see if we can avoid the need for a general set container
+// for all triangles.
 bool
 cdt_mesh_t::tri_add(triangle_t &tri)
 {

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



_______________________________________________
BRL-CAD Source Commits mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/brlcad-commits

Reply via email to