[29/36] libcamera: value_node: Support adding nested children in one operation
diff mbox series

Message ID 20260113000808.15395-30-laurent.pinchart@ideasonboard.com
State New
Headers show
Series
  • libcamera: Global configuration file improvements
Related show

Commit Message

Laurent Pinchart Jan. 13, 2026, 12:08 a.m. UTC
The GlobalConfiguration class will need to add nested children to a
ValueNode. Add a new overload to the add() function for this purpose.

Signed-off-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
---
 include/libcamera/internal/value_node.h |  3 ++
 src/libcamera/value_node.cpp            | 56 +++++++++++++++++++++++++
 2 files changed, 59 insertions(+)

Patch
diff mbox series

diff --git a/include/libcamera/internal/value_node.h b/include/libcamera/internal/value_node.h
index c285be4957ed..abb0991fd87e 100644
--- a/include/libcamera/internal/value_node.h
+++ b/include/libcamera/internal/value_node.h
@@ -8,6 +8,7 @@ 
 
 #pragma once
 
+#include <initializer_list>
 #include <iterator>
 #include <map>
 #include <memory>
@@ -237,6 +238,8 @@  public:
 
 	ValueNode *add(std::unique_ptr<ValueNode> child);
 	ValueNode *add(std::string key, std::unique_ptr<ValueNode> child);
+	ValueNode *add(std::initializer_list<std::string_view> path,
+		       std::unique_ptr<ValueNode> child);
 
 private:
 	LIBCAMERA_DISABLE_COPY_AND_MOVE(ValueNode)
diff --git a/src/libcamera/value_node.cpp b/src/libcamera/value_node.cpp
index 50b23284f930..1256adf8361d 100644
--- a/src/libcamera/value_node.cpp
+++ b/src/libcamera/value_node.cpp
@@ -13,6 +13,8 @@ 
 #include <string>
 #include <vector>
 
+#include <libcamera/base/log.h>
+#include <libcamera/base/span.h>
 #include <libcamera/base/utils.h>
 
 /**
@@ -22,6 +24,8 @@ 
 
 namespace libcamera {
 
+LOG_DEFINE_CATEGORY(ValueNode)
+
 namespace {
 
 /* Empty static ValueNode as a safe result for invalid operations */
@@ -539,4 +543,56 @@  ValueNode *ValueNode::add(std::string key, std::unique_ptr<ValueNode> child)
 	return elem.value.get();
 }
 
+/**
+ * \brief Add a child node at the given path
+ * \param[in] path The path
+ * \param[in] child The child node
+ *
+ * Add the \a child node at the given \a path starting at this node. Missing
+ * nodes are created along the path. Nodes along the path must be empty, in
+ * which case they are converted to the Type::Dictionary type, be a dictionary,
+ * or be missing. Otherwise, the \a child is discarded and the function returns
+ * a nullptr.
+ *
+ * Path elements are unique in the context of a parent node. If a child with the
+ * same \a key already exist at the end of the path, the \a child is discarded
+ * and the function returns a nullptr.
+ *
+ * \return A pointer to the \a child node if successfully added, nullptr
+ * otherwise
+ */
+ValueNode *ValueNode::add(std::initializer_list<std::string_view> path,
+			  std::unique_ptr<ValueNode> child)
+{
+	if (!path.size())
+		return NULL;
+
+	ValueNode *node = this;
+
+	for (const auto [i, name] : utils::enumerate(path)) {
+		auto iter = node->dictionary_.find(name);
+		if (iter == node->dictionary_.end()) {
+			std::unique_ptr<ValueNode> obj;
+
+			if (i < path.size() - 1)
+				obj = std::make_unique<ValueNode>();
+			else
+				obj = std::move(child);
+
+			node = node->add(std::string{ name }, std::move(obj));
+			if (!node) {
+				Span<const std::string_view> pathName{ std::data(path), i + 1 };
+				LOG(ValueNode, Error)
+					<< "Failed to populate '"
+					<< utils::join(pathName, "/") << "'";
+				return nullptr;
+			}
+		} else {
+			node = iter->second;
+		}
+	}
+
+	return node;
+}
+
 } /* namespace libcamera */