Remove duplication in tribits_advanced_set_cache_var_and_default() (T…

…riBITSPub#582) Now just calls tribits_set_cache_var_and_default() then mark_as_advanced(). Suggested by @KyleFromKitware
bartlettroscoe · Sep 21, 2023 · 4d848d8 · 4d848d8
1 parent e507636
commit 4d848d8
Showing 1 changed file with 2 additions and 6 deletions.
diff --git a/tribits/core/utils/TribitsSetCacheVarAndDefault.cmake b/tribits/core/utils/TribitsSetCacheVarAndDefault.cmake
@@ -57,12 +57,8 @@ include_guard()
 macro(tribits_advanced_set_cache_var_and_default  cacheVarName  cacheVarType
     defaultDefaultVal  docString
   )
-  if ("${${cacheVarName}_DEFAULT}" STREQUAL "")
-    set(${cacheVarName}_DEFAULT "${defaultDefaultVal}")
-  endif()
-  set(${cacheVarName} "${${cacheVarName}_DEFAULT}"
-    CACHE ${cacheVarType}
-    "${docString}" )
+  tribits_set_cache_var_and_default("${cacheVarName}" "${cacheVarType}"
+    "${defaultDefaultVal}" "${docString}")
   mark_as_advanced(${cacheVarName})
 endmacro()