On Thu, 2009-10-01 at 14:40 +0100, Julian Foad wrote:
> On Thu, 2009-10-01 at 05:44 -0700, Bert Huijben wrote:
> > Author: rhuijben
> > Date: Thu Oct 1 05:44:40 2009
> > New Revision: 39733
> >
> > Log:
> > Following up on r39729, fix the 'tree conflicts: tree missing, leaf del'
> > test by adding a temporary api for checking on working copy obstructions.
> >
> > Note that this class of obstructions will be gone once we move to a
> > central db.
>
> Why/how won't it be possible for a node to be obstructed?
I see you replaced this change with something different in r39737, but
the new version says in obstructed_or_missing(),
+ /* ### This check (and most of this function)
+ can be removed after we move to one DB */
Can you help me understand why?
> > Modified: trunk/subversion/libsvn_client/merge.c
> >
> > + /* svn_wc__maybe_get_entry ignores node kind errors, so check if we
> > + didn't get the parent stub */
> > + if (entry && entry->kind == svn_node_dir && *entry->name != '\0')
> > + return svn_wc_notify_state_missing; /* Only found parent entry */
> > +
> > + err = svn_wc__temp_node_obstructed(&obstructed, merge_b->ctx->wc_ctx,
> > + local_abspath, pool);
> > +
> > + if (err)
> > + svn_error_clear(err);
>
> Why is it OK to ignore an error?
And that code has gone away now.
- Julian
------------------------------------------------------
http://subversion.tigris.org/ds/viewMessage.do?dsForumId=462&dsMessageId=2402528
Received on 2009-10-01 15:48:28 CEST