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

Re: svn commit: r38671 - trunk/subversion/libsvn_wc

From: Greg Stein <gstein_at_gmail.com>
Date: Tue, 11 Aug 2009 16:41:17 +0200

On Tue, Aug 11, 2009 at 16:24, Hyrum K. Wright<hyrum_at_hyrumwright.org> wrote:
>...
> +++ trunk/subversion/libsvn_wc/status.c Tue Aug 11 07:24:26 2009        (r38671)
>...
> @@ -1712,9 +1715,8 @@ delete_entry(const char *path,
>   /* Read the parent's entries file.  If the deleted thing is not
>      versioned in this working copy, it was probably deleted via this
>      working copy.  No need to report such a thing. */
> -  /* ### use svn_wc_entry() instead? */
> -  SVN_ERR(svn_wc__entry_versioned(&entry, full_path, eb->adm_access,
> -                                  FALSE, pool));
> +  SVN_ERR(svn_wc__get_entry(&entry, eb->db, local_abspath, TRUE,
> +                            svn_node_unknown, FALSE, pool, pool));
>   if (entry->kind == svn_node_dir)
>     {

allow_unversioned should be FALSE. entry_versioned always requires
that the requested node be present. Also note that returning
entry==NULL would be bad here, given the immediate dereference.

Cheers,
-g

------------------------------------------------------
http://subversion.tigris.org/ds/viewMessage.do?dsForumId=462&dsMessageId=2382502
Received on 2009-08-11 16:41:39 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.