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

Re: [PATCH] wc-ng task: Remove svn_wc_entry_t's obtained only to find svn_node_kind_t.

From: Greg Stein <gstein_at_gmail.com>
Date: Tue, 18 Aug 2009 10:52:04 +0200

On Mon, Aug 17, 2009 at 18:44, Paul Burba<ptburba_at_gmail.com> wrote:
> wc-ng gurus:
>
> While trying (fruitlessly) to track down a switch test segfault last
> Friday I noticed that in several places we grab an svn_wc_entry_t
> simply so we can find a path's svn_node_kind_t.  This seems like an
> obvious place to get rid of some entry structures, so here's a stab at
> doing just that.
>
> The only seeming difficulty/question in this task is deciding how we
> map svn_wc__db_kind_t to svn_node_kind_t.  The attached patch takes
> this approach:
>
> svn_wc__db_kind_dir (not hidden*)  --> svn_node_dir
> svn_wc__db_kind_dir (hidden*)      --> svn_node_none
> svn_wc__db_kind_file (not hidden*) --> svn_node_file
> svn_wc__db_kind_file (hidden*)     --> svn_node_none
> svn_wc__db_kind_symlink            --> svn_node_unknown
> svn_wc__db_kind_unknown            --> svn_node_unknown
> svn_wc__db_kind_subdir             --> svn_node_unknown
>
> * As per svn_wc__db_node_hidden().  This is necessary because
> svn_wc__db_check_node() might report that a path is a
> svn_wc__db_kind_dir or svn_wc__db_kind_file, when in fact the path has
> been deleted.  This can happen if the path's parent is at an revision
> at which the child path was not deleted.
>
> Does that seem correct?

Some of the calls to svn_wc_entry() or get_entry_versioned() pass
show_hidden=TRUE, so the mapping to none shouldn't be done for those
cases.

Also, db_kind_symlink would map to wc-1's svn_node_file.

>...

Cheers,
-g

------------------------------------------------------
http://subversion.tigris.org/ds/viewMessage.do?dsForumId=462&dsMessageId=2384669
Received on 2009-08-18 10:52:20 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.