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

Re: svn commit: r940651 - in /subversion/trunk/subversion: include/svn_wc.h libsvn_wc/status.c libsvn_wc/util.c

From: Daniel Näslund <daniel_at_longitudo.com>
Date: Tue, 4 May 2010 20:50:09 +0200

On Mon, May 03, 2010 at 08:41:14PM -0400, Greg Stein wrote:
> On Mon, May 3, 2010 at 18:01, <dannas_at_apache.org> wrote:
> >...
> > +++ subversion/trunk/subversion/libsvn_wc/util.c Mon May  3 22:01:53 2010
> >...
> > @@ -565,6 +567,12 @@ svn_wc__status2_from_3(svn_wc_status2_t
> >   SVN_ERR(svn_wc__get_entry(&entry, wc_ctx->db, local_abspath, TRUE,
> >                             svn_node_unknown, FALSE, result_pool,
> >                             scratch_pool));
> > +
> > +  SVN_ERR(svn_wc__db_op_read_tree_conflict(&tree_conflict, wc_ctx->db,
> > +                                           local_abspath, result_pool,
> > +                                           scratch_pool));
>
> You don't need 'tree_conflict' to persist, so both pools can be scratch_pool.
>
> > +  old_tree_conflict = svn_wc__cd2_to_cd(tree_conflict, scratch_pool);
>
> But you're returning this one, so that should be result_pool.

Committed in r940982.

Thanks,
Daniel
Received on 2010-05-04 20:51:30 CEST

This is an archived mail posted to the Subversion Dev mailing list.

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.