[PATCH 0/4] vcs-svn: remove repo_tree library

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@google.com [2] https://public-inbox.org/git/20170821000022.26729-3-sandals@crustytoothpaste.net