RE: svn commit: r997639 - /subversion/trunk/subversion/include/svn_client.h

> -----Original Message-----
> From: hwright_at_apache.org [mailto:hwright_at_apache.org]
> Sent: donderdag 16 september 2010 10:24
> To: commits_at_subversion.apache.org
> Subject: svn commit: r997639 -
> /subversion/trunk/subversion/include/svn_client.h
>
> Author: hwright
> Date: Thu Sep 16 08:23:54 2010
> New Revision: 997639
>

> + * than a directory; <br>
> + * #SVN_ERR_RA_ILLEGAL_URL if @a URL does not exist; <br>
> + * #SVN_ERR_CLIENT_BAD_REVISION if @a revision is not one of
> + * #svn_opt_revision_number, #svn_opt_revision_head, or
> + * #svn_opt_revision_date; <br>
> + * #SVN_NO_ERROR otherwise.

And after reading this as just an API user, how would you expect authorization, network, filesystem and/or other errors to be reported?

The old documentation documented a few specific errors and left the rest open. This new documentation would tell me that we never report other errors then this short (incomplete) list, while just checking out over an existing working copy from a different repository will give an error that is not in this list.

And this function returns an svn_error_t* which (when not NULL) can contain those error codes, or can be NULL/SVN_NO_ERROR. SVN_NO_ERROR and the specific error codes should not be in the same list without some separation.

        Bert
Received on 2010-09-16 12:06:11 CEST