cmPropertyDefinitionMap: simplify and shorten

1 files changed, 30 insertions, 9 deletions

diff --git a/Source/cmPropertyDefinition.cxx b/Source/cmPropertyDefinition.cxx
index c8efaf65a9..1796bb8d0e 100644
--- a/Source/cmPropertyDefinition.cxx
+++ b/Source/cmPropertyDefinition.cxx
@@ -2,17 +2,38 @@
    file Copyright.txt or https://cmake.org/licensing for details.  */
 #include "cmPropertyDefinition.h"
 
-#include <utility>
+#include <tuple>
 
-cmPropertyDefinition::cmPropertyDefinition(std::string name,
-                                           cmProperty::ScopeType scope,
-                                           std::string shortDescription,
+cmPropertyDefinition::cmPropertyDefinition(std::string shortDescription,
                                            std::string fullDescription,
-                                           bool chain)
-  : Name(std::move(name))
-  , ShortDescription(std::move(shortDescription))
+                                           bool chained)
+  : ShortDescription(std::move(shortDescription))
   , FullDescription(std::move(fullDescription))
-  , Scope(scope)
-  , Chained(chain)
+  , Chained(chained)
 {
 }
+
+void cmPropertyDefinitionMap::DefineProperty(
+  const std::string& name, cmProperty::ScopeType scope,
+  const std::string& ShortDescription, const std::string& FullDescription,
+  bool chain)
+{
+  auto it = this->Map_.find(key_type(name, scope));
+  if (it == this->Map_.end()) {
+    // try_emplace() since C++17
+    this->Map_.emplace(
+      std::piecewise_construct, std::forward_as_tuple(name, scope),
+      std::forward_as_tuple(ShortDescription, FullDescription, chain));
+  }
+}
+
+cmPropertyDefinition const* cmPropertyDefinitionMap::GetPropertyDefinition(
+  const std::string& name, cmProperty::ScopeType scope) const
+{
+  auto it = this->Map_.find(key_type(name, scope));
+  if (it != this->Map_.end()) {
+    return &it->second;
+  }
+
+  return nullptr;
+}