Adding TODOs for function reuse

Author: ruffsl

rcl/src/rcl/node.c

@@ -136,8 +136,11 @@ const char * rcl_get_secure_root(
     char * node_fqn = NULL;
     char * node_root_path = NULL;
     // Combine node namespace with node name
+    // TODO: This make assumption on the length of the root namespace as well as its value.
+    // We should likely come from another (rcl/rmw?) function for reuse.
     node_fqn = rcutils_format_string(*allocator, "%s%s%s", node_namespace, "/", node_name);
     // Get native path, ignore the leading forward slash.
+    // TODO: The offset should be a define we can access, instead of hardcoding + 1
     node_root_path = rcutils_to_native_path(node_fqn + 1, *allocator);
     node_secure_root = rcutils_join_path(ros_secure_root_env, node_root_path, *allocator);
     allocator->deallocate(node_fqn, allocator->state);