Another (at this point perhaps needless) optimization attempt

1 files changed, 12 insertions, 12 deletions

diff --git a/server/src/api.cpp b/server/src/api.cpp
index 9953675..5dd3fc5 100644
--- a/server/src/api.cpp
+++ b/server/src/api.cpp
@@ -92,7 +92,7 @@ auto tag_invoke(json::value_from_tag, json::value& jv, sysinfo const& info)
   jv = json::object{
     {"using_publication_of",
      std::format("{:%FT%TZ}", info.using_publication_of)},
-    {"lse_index_size", info.lse_index_size},
+    {"blse_index_size", info.blse_index_size},
     {"p_index_size", info.p_index_size},
   };
 }
@@ -135,13 +135,13 @@ handler::handler(log::logger const& l, datex2::situation_publication pub)
     {
       for (auto const& ls : rc->relevant_line_strings)
       {
-        auto box = geo::box{};
-        bgeo::envelope(*ls, box);
         auto buffered_ls = geo::multi_polygon{};
         bgeo::buffer(
             *ls, buffered_ls, distance_strategy, side_strategy, join_strategy,
             end_strategy, circle_strategy);
-        lse_index_.insert(std::make_tuple(box, buffered_ls, rc));
+        auto blse = geo::box{};
+        bgeo::envelope(buffered_ls, blse);
+        blse_index_.insert(std::make_tuple(blse, buffered_ls, rc));
       }
       for (auto p : rc->relevant_points)
       {
@@ -161,7 +161,7 @@ handler::handler(log::logger const& l, datex2::situation_publication pub)
   auto const after_build = chrono::steady_clock::now();
   auto const dur_build_s = chrono::duration<double>{after_build - before_build};
   l_.info("Indices built in {}", dur_build_s);
-  l_.info("LSE index size: {}", lse_index_.size());
+  l_.info("BLSE index size: {}", blse_index_.size());
   l_.info("Point index size: {}", p_index_.size());
 }
 
@@ -200,19 +200,19 @@ auto handler::process_gpx(gpx::file&& gpx_file)
     auto part_box = geo::box{};
     bgeo::envelope(part, part_box);
 
-    for (auto it = lse_index_.qbegin(bgeo::index::intersects(part_box));
-         it != lse_index_.qend(); it++)
+    for (auto it = blse_index_.qbegin(bgeo::index::intersects(part_box));
+         it != blse_index_.qend(); it++)
     {
       // Cannot use structured bindings here, as boost::geometry::get
       // interferes with ADL. It is a candidate as the namespace
       // boost::geometry is part of the associated namespace set, which
       // happens because geo::linestring ≡
       // boost::geometry::model::linestring<geo::point> is part of the
-      // whole tuple type (lse_index_value) that is the value_type of the
+      // whole tuple type (blse_index_value) that is the value_type of the
       // iterator.
 
-      // geo::box const& lse = std::get<0>(*it);
-      geo::multi_polygon const& buffered_ls = std::get<1>(*it);
+      geo::box const& blse = std::get<0>(*it);
+      geo::multi_polygon const& bls = std::get<1>(*it);
       std::shared_ptr<datex2::road_closure> const& rc = std::get<2>(*it);
       if (rc->validity
           && rc->validity->intersect(check_periods).periods().empty())
@@ -221,7 +221,7 @@ auto handler::process_gpx(gpx::file&& gpx_file)
       // auto small_parts = bgeo::model::multi_linestring<geo::linestring>{};
       // bgeo::intersection(part, lse, small_parts);
 
-      if (bgeo::intersects(buffered_ls, part))
+      if (bgeo::intersects(blse, part) && bgeo::intersects(bls, part))
         relevant_road_closures.emplace(rc);
 
       // if (!bgeo::is_empty(small_parts))
@@ -318,7 +318,7 @@ auto handler::sysinfo() -> struct sysinfo
 {
   return {
     .using_publication_of = pub_.publication_time,
-    .lse_index_size = lse_index_.size(),
+    .blse_index_size = blse_index_.size(),
     .p_index_size = p_index_.size(),
   };
 }