Split on : to deduce function group name

score-p · Nov 20, 2023 · f72c8db · f72c8db
1 parent a24dda1
commit f72c8db
Showing 1 changed file with 12 additions and 6 deletions.
diff --git a/src/scorepy/events.cpp b/src/scorepy/events.cpp
@@ -35,12 +35,15 @@ void region_begin(std::string_view function_name, std::string module, std::strin
 
     if (region == uninitialised_region_handle)
     {
-        const auto region_name = make_region_name(std::move(module), function_name);
+        auto region_name = make_region_name(std::move(module), function_name);
         SCOREP_User_RegionInit(&region.value, NULL, NULL, region_name.c_str(),
                                SCOREP_USER_REGION_TYPE_FUNCTION, file_name.c_str(), line_number);
 
-        SCOREP_User_RegionSetGroup(region.value,
-                                   std::string(region_name, 0, region_name.find('.')).c_str());
+        if (const auto pos = region_name.find(':'); pos != std::string::npos)
+        {
+            region_name.resize(pos);
+            SCOREP_User_RegionSetGroup(region.value, region_name.c_str());
+        }
     }
     SCOREP_User_RegionEnter(region.value);
 }
@@ -50,16 +53,19 @@ void region_begin(std::string_view function_name, std::string module, std::strin
 void region_begin(std::string_view function_name, std::string module, std::string file_name,
                   const std::uint64_t line_number)
 {
-    const auto region_name = make_region_name(std::move(module), function_name);
+    auto region_name = make_region_name(std::move(module), function_name);
     region_handle& region = user_regions[region_name];
 
     if (region == uninitialised_region_handle)
     {
         SCOREP_User_RegionInit(&region.value, NULL, NULL, region_name.c_str(),
                                SCOREP_USER_REGION_TYPE_FUNCTION, file_name.c_str(), line_number);
 
-        SCOREP_User_RegionSetGroup(region.value,
-                                   std::string(region_name, 0, region_name.find('.')).c_str());
+        if (const auto pos = region_name.find(':'); pos != std::string::npos)
+        {
+            region_name.resize(pos);
+            SCOREP_User_RegionSetGroup(region.value, region_name.c_str());
+        }
     }
     SCOREP_User_RegionEnter(region.value);
 }