[master] noteed RBTree::nodeFission should preserve the current node's name.

This will help at the phase of revisiting the design/implementation.

diff --git a/src/lib/datasrc/rbtree.h b/src/lib/datasrc/rbtree.h
index 1200f58126a176e7de55d5e60118e8a4f6c4cccb..dbf05914c0738e8eafdd782696f99a19a096bd74 100644 (file)
--- a/src/lib/datasrc/rbtree.h
+++ b/src/lib/datasrc/rbtree.h
@@ -1162,6 +1162,10 @@ RBTree<T>::insert(const isc::dns::Name& target_name, RBNode<T>** new_node) {
 }
 
 
+// Note: when we redesign this (still keeping the basic concept), we should
+// change this part so the newly created node will be used for the inserted
+// name (and therefore the name for the existing node doesn't change).
+// Otherwise, things like shortcut links between nodes won't work.
 template <typename T>
 void
 RBTree<T>::nodeFission(RBNode<T>& node, const isc::dns::Name& base_name) {