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/06/03 20:32:41 UTC

Re: svn commit: r948581 - in /subversion/trunk/subversion: include/svn_wc.h libsvn_wc/status.c

On Wed, May 26, 2010 at 16:46,  <rh...@apache.org> wrote:
>...
> +++ subversion/trunk/subversion/libsvn_wc/status.c Wed May 26 20:46:48 2010
>...
> @@ -2042,7 +2045,7 @@ close_directory(void *dir_baton,
>           eb->anchor_status->repos_text_status = repos_text_status;
>
>           /* If the root dir is out of date set the ood info directly too. */
> -          if (db->ood_last_cmt_rev != eb->anchor_status->entry->revision)
> +          if (db->ood_last_cmt_rev != eb->anchor_status->revision)
>             {
>               eb->anchor_status->ood_last_cmt_rev = db->ood_last_cmt_rev;
>               eb->anchor_status->ood_last_cmt_date = db->ood_last_cmt_date;

entry->revision and status->revision have different semantics. I don't
know if you can make this change (without more thought upon the
context). Please review the implications here.

>...

Cheers,
-g