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/08/03 17:34:49 UTC

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

On Tue, Aug 3, 2010 at 10:13,  <rh...@apache.org> wrote:
> Author: rhuijben
> Date: Tue Aug  3 14:13:10 2010
> New Revision: 981893
>
> URL: http://svn.apache.org/viewvc?rev=981893&view=rev
> Log:
> * subversion/libsvn_wc/status.c
>  (get_dir_status): In single-db don't use svn_wc__db_node_hidden, if we
>    already have status.

This seems like a dangerous disparity. It is exposing too much of
SINGLE_DB outside of wc_db.h.

I would very much encourage finding a way to collapse this under the
wc_db covers before continuing this pattern. Ideally, the only
SINGLE_DB conditions should be *inside* wc_db. If not, then something
is probably wrong in that API.

Cheers,
-g