Re: [PATCH] rev svn_wc_status2_t to svn_wc_status3_t
Committed in r932571.
On Fri, Apr 09, 2010 at 03:35:17PM -0400, Greg Stein wrote:
> Because this is "just" a type rename, it is very easy to review.
> I'm +1 to commit with the mods I mentioned on IRC. I'll do a followup
> review post-commit.
> On Fri, Apr 9, 2010 at 11:06, Daniel Näslund <daniel_at_longitudo.com> wrote:
> > Hi!
> > This is a _big_ patch but it does very little. I've just replaced every
> > occurence of svn_wc_status2_t with the new svn_wc_status3_t. It's
> > needed, in the cause of removing the entry field which I intend to do in
> > a follow-up. There's a WIP on the list, that handles most of the chores of the
> > status->entry.
> > Hopefully someone will not be put of by this gigantic blob. If we hadn't
> > already revved svn_wc_status3() for 1.7 I could have done the compat
> > wrappers and declarations in one patch and update the callers in a
> > follow-up. But since it was already revved, here you go! :)
> > make check passed.
Received on 2010-04-09 22:10:11 CEST
This is an archived mail posted to the Subversion Dev