Hi, Stefan noticed that repo_init from vcs-svn/repo_tree.h conflicts with repository.h[1]. Earlier brian m. carlson noticed the same thing[2]. Originally repo_tree.h was used to manage an in-memory representation of the state of the svn tree being imported. When that in-memory representation was retired, we were lazy and left some utility functions there. Here is a patch series to finish cleaning up and remove vcs-svn/repo_tree.h completely. This is an alternative to bc/vcs-svn-cleanup from 'next'. Those patches weren't cc-ed to me and I missed them --- sorry about that. I can rebase on top of them if that is more convenient. Thoughts of all kinds welcome, as always. Thanks, Jonathan Jonathan Nieder (4): vcs-svn: remove prototypes for missing functions vcs-svn: remove custom mode constants vcs-svn: remove repo_delete wrapper function vcs-svn: move remaining repo_tree functions to fast_export.h Makefile | 1 - vcs-svn/fast_export.c | 41 +++++++++++++++++++++++++++++++++++++---- vcs-svn/fast_export.h | 3 +++ vcs-svn/repo_tree.c | 48 ------------------------------------------------ vcs-svn/repo_tree.h | 23 ----------------------- vcs-svn/svndump.c | 33 ++++++++++++++++----------------- 6 files changed, 56 insertions(+), 93 deletions(-) delete mode 100644 vcs-svn/repo_tree.c delete mode 100644 vcs-svn/repo_tree.h [1] https://public-inbox.org/git/20170822213501.5928-1-sbeller@xxxxxxxxxx [2] https://public-inbox.org/git/20170821000022.26729-3-sandals@xxxxxxxxxxxxxxxxxxxx
