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

Re: svn commit: r944478 - /subversion/trunk/subversion/libsvn_wc/status.c

From: Greg Stein <gstein_at_gmail.com>
Date: Fri, 14 May 2010 20:13:33 -0400

On Fri, May 14, 2010 at 17:22, <dannas_at_apache.org> wrote:
>...
> +++ subversion/trunk/subversion/libsvn_wc/status.c Fri May 14 21:22:52 2010
>...
> @@ -1463,11 +1463,11 @@ find_dir_url(const struct dir_baton *db,
>       const svn_wc_status3_t *status = apr_hash_get(pb->statii,
>                                                     db->local_abspath,
>                                                     APR_HASH_KEY_STRING);
> -      /* Note that status->entry->url is NULL in the case of a missing
> +      /* Note that status->url is NULL in the case of a missing
>        * directory, which means we need to recurse up another level to
>        * get a useful URL. */
> -      if (status && status->entry && status->entry->url)
> -        return status->entry->url;
> +      if (status)
> +        return status->url;

I don't think the comment is correct. status->url is always set, right?

Cheers,
-g
Received on 2010-05-15 02:14:02 CEST

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