On Fri, May 14, 2010 at 17:22, <dan...@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
