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

Re: svn commit: rev 2943 - trunk/subversion/libsvn_wc

From: Philip Martin <philip_at_codematters.co.uk>
Date: 2002-08-12 00:57:35 CEST

rooneg@tigris.org writes:

> Modified: trunk/subversion/libsvn_wc/adm_files.c
> ==============================================================================
> --- trunk/subversion/libsvn_wc/adm_files.c (original)
> +++ trunk/subversion/libsvn_wc/adm_files.c Sun Aug 11 17:11:15 2002
> @@ -924,8 +924,8 @@
> svn_error_t *err = NULL;
> enum svn_node_kind kind;
> svn_boolean_t dir_exists = FALSE, wc_exists = FALSE;
> - apr_file_t *f = NULL;
> const char *tmp_path;
> + apr_finfo_t finfo;
>
> /** Step 1: check that the directory exists. **/
>
> @@ -963,7 +963,9 @@
> /** The directory exists, but is it a valid working copy yet?
> Try step 2: checking that SVN_WC__ADM_README exists. **/
>
> - err = svn_wc__open_adm_file (&f, path, SVN_WC__ADM_README, APR_READ, pool);
> + err = svn_io_stat (&finfo, svn_path_join (tmp_path, SVN_WC__ADM_README, pool),
> + APR_FINFO_MIN, pool);
> +

Would svn_io_check_path && kind == svn_node_file be better? That
boils down to a single stat call. Thinking of a directory called
README...

-- 
Philip Martin
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Mon Aug 12 00:59:08 2002

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.