You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@subversion.apache.org by Julian Foad <ju...@apache.org> on 2017/03/16 15:32:22 UTC

API review for 1.10 -- svn_ra_list()

Stefan2 et al.:

In svn_ra.h around line 1132, two new-in-1.10 additions, with several 
issues with their doc strings.

svn_ra_dirent_receiver_t:

   bad @a/@c references: svn_repos_list, path/rel_path, path_info_only

svn_ra_list():

   bad @a/@c references: root, authz_read_func, cancel_func, cancel_baton
   no reference for the 'dirent_fields'


Thanks.

- Julian

Re: API review for 1.10 -- svn_ra_list()

Posted by Julian Foad <ju...@apache.org>.

Stefan Fuhrmann wrote:
> On 16.03.2017 16:32, Julian Foad wrote:
>> In svn_ra.h around line 1132, two new-in-1.10 additions, with several
>> issues with their doc strings.
>>
>> svn_ra_dirent_receiver_t:
>>
>>   bad @a/@c references: svn_repos_list, path/rel_path, path_info_only
>>
>> svn_ra_list():
>>
>>   bad @a/@c references: root, authz_read_func, cancel_func, cancel_baton
>>   no reference for the 'dirent_fields'
>
> Thank for the review.
> r1791771 should fix these.

Thanks. That looks better.

- Julian

Re: API review for 1.10 -- svn_ra_list()

Posted by Stefan Fuhrmann <st...@apache.org>.

On 16.03.2017 16:32, Julian Foad wrote:
> Stefan2 et al.:
>
> In svn_ra.h around line 1132, two new-in-1.10 additions, with several 
> issues with their doc strings.
>
> svn_ra_dirent_receiver_t:
>
>   bad @a/@c references: svn_repos_list, path/rel_path, path_info_only
>
> svn_ra_list():
>
>   bad @a/@c references: root, authz_read_func, cancel_func, cancel_baton
>   no reference for the 'dirent_fields'

Thank for the review.
r1791771 should fix these.

-- Stefan^2.