[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

Re: [PATCH] rev svn_wc_status2_t to svn_wc_status3_t

From: Daniel Näslund <daniel_at_longitudo.com>
Date: Fri, 9 Apr 2010 22:09:14 +0200

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.
>
> Thanks!
> -g
>
> 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 mailing list.