> The new void *data pointer could use a comment on what it is and how it's > used.
Yes; as a matter of fact, it needs to be documented for mostly_copy_tree_r first, and probably just referred to for copy_if_shared_r and copy_if_shared. -- Eric Botcazou