fixup

Author: Levi-Armstrong

tesseract_common/include/tesseract_common/property_tree.h

@@ -14,19 +14,18 @@ namespace tesseract_common
 namespace property_type
 {
 /**
- * @brief A utility for constructing the list type
+ * @brief A utility for constructing the std::vector<type>
  * @param type The type assoicated with the list
  * @return The string representation of the std::vector<type>, aka. type[]
  */
 std::string createList(std::string_view type);
 
 /**
- * @brief A utility for constructing the map type
- * @param key_type The type assoicated with the map key
- * @param value_type The type assoicated with the map value
- * @return The string representation of the std::make<key_type, value_type>, aka. {key_type : value_type}
+ * @brief A utility for constructing the std::map<std::string, type>
+ * @param type The value type assoicated with the map
+ * @return The string representation of the std::map<std::string, type>, aka. type{}
  */
-std::string createMap(std::string_view key_type, std::string_view value_type);
+std::string createMap(std::string_view type);
 
 // Integral Types
 constexpr std::string_view BOOL{ "bool" };

tesseract_common/src/property_tree.cpp

@@ -13,10 +13,7 @@ namespace tesseract_common
 namespace property_type
 {
 std::string createList(std::string_view type) { return std::string(type) + "[]"; };
-std::string createMap(std::string_view key_type, std::string_view value_type)
-{
-  return "{" + std::string(key_type) + " : " + std::string(value_type) + "}";
-};
+std::string createMap(std::string_view type) { return std::string(type) + "{}"; };
 }  // namespace property_type
 
 void PropertyTree::mergeConfig(const YAML::Node& config, bool allow_extra_properties)

tesseract_common/src/property_tree_demo.cpp

@@ -258,8 +258,7 @@ tesseract_common::PropertyTree getTaskComposerRasterOnlySchema()
     }
     {
       auto& prop = raster_config.get("remapping");
-      prop.setAttribute(property_attribute::TYPE,
-                        property_type::createMap(property_type::STRING, property_type::STRING));
+      prop.setAttribute(property_attribute::TYPE, property_type::createMap(property_type::STRING));
       prop.setAttribute(property_attribute::DOC, "The remapping of input and output keys");
     }
     {
@@ -291,8 +290,7 @@ tesseract_common::PropertyTree getTaskComposerRasterOnlySchema()
     }
     {
       auto& prop = transition_config.get("remapping");
-      prop.setAttribute(property_attribute::TYPE,
-                        property_type::createMap(property_type::STRING, property_type::STRING));
+      prop.setAttribute(property_attribute::TYPE, property_type::createMap(property_type::STRING));
       prop.setAttribute(property_attribute::DOC, "The remapping of input and output keys");
     }
     {