API review for 1.10 -- svn_ra_list()

Previous Topic Next Topic
 
classic Classic list List threaded Threaded
3 messages Options
Reply | Threaded
Open this post in threaded view
|  
Report Content as Inappropriate

API review for 1.10 -- svn_ra_list()

Julian Foad-5
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
Reply | Threaded
Open this post in threaded view
|  
Report Content as Inappropriate

Re: API review for 1.10 -- svn_ra_list()

Stefan Fuhrmann
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.
Reply | Threaded
Open this post in threaded view
|  
Report Content as Inappropriate

Re: API review for 1.10 -- svn_ra_list()

Julian Foad-5
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

Loading...