On Fri, Sep 19, 2008 at 11:09 PM, Blair Zajac <blair_at_orcaware.com> wrote:
> Greg Stein wrote:
>> On Fri, Sep 19, 2008 at 10:33 PM, <blair_at_tigris.org> wrote:
>>> ...
>>> +++ branches/file-externals/subversion/libsvn_client/externals.c
>>> Fri Sep 19 22:33:04 2008 (r33207)
>>> @@ -347,9 +347,8 @@ switch_file_external(const char *path,
>>> conflict on the directory. To prevent resolving a conflict
>>> due to another change on the directory, do not allow a file
>>> external to be added when one exists. */
>>> - SVN_ERR(svn_wc_entry(&anchor_dir_entry, anchor, adm_access, FALSE,
>>> - subpool));
>>> - SVN_ERR_ASSERT(anchor_dir_entry);
>>> + SVN_ERR(svn_wc__entry_versioned(&anchor_dir_entry, anchor,
>>> adm_access,
>>> + FALSE, subpool));
>>> SVN_ERR(svn_wc_conflicted_p2(&text_conflicted, &prop_conflicted,
>>> &has_tree_conflicted_children,
>>> anchor, anchor_dir_entry, subpool));
>>
>> Wait a sec here... the client library can't use internal WC functions.
>
> It's declared in subversion/include/private/svn_wc_private.h and it's used
> throughout libsvn_client:
>
> $ grep svn_wc__entry_versioned subversion/libsvn_client/*c | wc -l
> 38
>
> Presumably being in include/private means it can be used across lib's.
This isn't right... Our libraries should not have privileged access to
certain APIs that other users cannot reach. IOW, all 38 lines must
change :-P
Cheers,
-g
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe_at_subversion.tigris.org
For additional commands, e-mail: dev-help_at_subversion.tigris.org
Received on 2008-09-20 09:18:51 CEST