perf: faster ament_libraries_deduplicate implementation

Signed-off-by: Vincent Richard <[email protected]>
ament · May 11, 2023 · 1aee39a · 1aee39a
1 parent 154f198
commit 1aee39a
Show file tree

Hide file tree

Showing 4 changed files with 5 additions and 89 deletions.
diff --git a/ament_cmake_libraries/ament_cmake_libraries-extras.cmake b/ament_cmake_libraries/ament_cmake_libraries-extras.cmake
@@ -17,5 +17,3 @@
 set(AMENT_BUILD_CONFIGURATION_KEYWORD_SEPARATOR ":")
 
 include("${ament_cmake_libraries_DIR}/ament_libraries_deduplicate.cmake")
-include("${ament_cmake_libraries_DIR}/ament_libraries_pack_build_configuration.cmake")
-include("${ament_cmake_libraries_DIR}/ament_libraries_unpack_build_configuration.cmake")
diff --git a/ament_cmake_libraries/cmake/ament_libraries_deduplicate.cmake b/ament_cmake_libraries/cmake/ament_libraries_deduplicate.cmake
@@ -26,13 +26,9 @@
 # @public
 #
 macro(ament_libraries_deduplicate VAR)
-  ament_libraries_pack_build_configuration(_packed ${ARGN})
-  set(_unique "")
-  foreach(_lib ${_packed})
-    # remove existing value if it exists
-    list(REMOVE_ITEM _unique ${_lib})
-    # append value to the end
-    list(APPEND _unique ${_lib})
-  endforeach()
-  ament_libraries_unpack_build_configuration(${VAR} ${_unique})
+  string(REGEX REPLACE "(^|;)(debug|optimized|general);(.+)" "\\2${AMENT_BUILD_CONFIGURATION_KEYWORD_SEPARATOR}\\3" _packed "${ARGN}")
+  list(REVERSE _packed)
+  list(REMOVE_DUPLICATES _packed)
+  list(REVERSE _packed)
+  string(REGEX REPLACE "(^|;)(debug|optimized|general)${AMENT_BUILD_CONFIGURATION_KEYWORD_SEPARATOR}(.+)" "\\2;\\3" ${VAR} "${_packed}")
 endmacro()
diff --git a/ament_cmake_libraries/cmake/ament_libraries_pack_build_configuration.cmake b/ament_cmake_libraries/cmake/ament_libraries_pack_build_configuration.cmake
diff --git a/ament_cmake_libraries/cmake/ament_libraries_unpack_build_configuration.cmake b/ament_cmake_libraries/cmake/ament_libraries_unpack_build_configuration.cmake