test

Author: BotellaA

src/geode/model/helpers/ray_tracing.cpp

@@ -24,7 +24,6 @@
 #include <geode/model/helpers/ray_tracing.hpp>
 
 #include <absl/base/call_once.h>
-#include <absl/synchronization/mutex.h>
 
 #include <geode/basic/pimpl_impl.hpp>
 
@@ -238,7 +237,15 @@ namespace geode
     class BRepRayTracing::Impl
     {
     public:
-        explicit Impl( const BRep& brep ) : brep_( brep ) {}
+        explicit Impl( const BRep& brep ) : brep_( brep )
+        {
+            aabb_trees_.reserve( brep.nb_surfaces() );
+            for( const auto& surface : brep.surfaces() )
+            {
+                aabb_trees_.emplace(
+                    surface.id(), std::make_unique< CachedTree >() );
+            }
+        }
 
         BoundarySurfaceIntersections find_intersections_with_boundaries(
             const InfiniteLine3D& infinite_line, const Block3D& block )
@@ -307,26 +314,16 @@ namespace geode
 
         const AABBTree3D& surface_aabb( const Surface3D& surface )
         {
-            auto& entry = cached_tree( surface.id() );
+            auto& entry = *aabb_trees_.at( surface.id() );
             absl::call_once( entry.built, [&surface, &entry] {
                 entry.tree = create_aabb_tree( surface.mesh() );
             } );
             return entry.tree;
         }
 
-        CachedTree& cached_tree( const uuid& surface_id )
-        {
-            absl::MutexLock lock{ mutex_ };
-            return *aabb_trees_
-                        .try_emplace(
-                            surface_id, std::make_unique< CachedTree >() )
-                        .first->second;
-        }
-
     private:
         const BRep& brep_;
         absl::flat_hash_map< uuid, std::unique_ptr< CachedTree > > aabb_trees_;
-        absl::Mutex mutex_;
     };
 
     BRepRayTracing::BRepRayTracing( const BRep& brep ) : impl_{ brep } {}