On Sun, Dec 8, 2013 at 1:31 PM, Bert Huijben <b...@qqmail.nl> wrote: > We revved almost the entire wc api for Subversion 1.7, with almost no > breakage for the older wc apis. (8 errata if I remember correctly). I don't > see why we can't do this for the fs or ra layer APIs before 2.0. The old > public APIs that miss arguments won't be as fast as the newer APIs that > have them but... I don't see this as an excuse to use a bad design for the > fs layer until 2.0. >
Well, the alternative would be to eliminate svn_fs_id_t entirely from the API, which implies revving svn_fs_dirent_t and svn_fs_path_change2_t plus related functions. Simply introducing svn_fs_id2_t in parallel to svn_fs_id_t would needlessly duplicate a lot of code exactly because the underlying representations are incompatible. I would actually like to remve svn_fs_id_t from the API since it violates our general path@rev / path@root pattern (deprecating the old API for backward compat). Any objections? -- Stefan^2.
