You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@subversion.apache.org by Greg Stein <gs...@gmail.com> on 2010/05/15 00:13:33 UTC
Re: svn commit: r944478 - /subversion/trunk/subversion/libsvn_wc/status.c
On Fri, May 14, 2010 at 17:22, <da...@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