The following functions seem very redundant. Because each is slightly different, I always have to compare/contrast them to isolate their differences. It would be much better if we could pick JUST ONE, and run with that:
svn_*_is_child() svn_*_is_ancestor() svn_*_skip_ancestor() I do realize that we published the svn_dirent_* functions in 1.6, but we don't have to carry their analogues to the new functions. (and possibly even deprecate the 1.6 funcs) It seems that the is_child variant is the only function needed. The is_ancestor() is merely is_child() != NULL, and the skip_ancestor is simply pool=NULL. Am I missing something, and/or can/should we remove this redundancy? Cheers, -g
