On Fri, Mar 29, 2013 at 9:17 PM, C. Michael Pilato <cmpilato_at_collab.net>wrote:
> Devs,
>
> I've just completed my review of the new-in-1.8 public APIs, minus the bits
> that Philip reviewed (thanks!) and the new merge-related stuff which, if I
> understand from recent threads correctly, is still subject to some churn.
>
> The results of my review revealed overwhelmingly positive results which, in
> my approximation, are non-contentious. I had (for some definition of
> "had")
> to touch up quite a few docstring in the process, but by and large those
> were stylistic nits with the occasionally overlooked item.
>
> The current state of the review work may be found here:
>
> http://wiki.apache.org/subversion/Svn18ApiReview
>
> Please note especially the section labeled "Reviewed But Need Further
> Attention". In it, I call out some of the stuff about which I couldn't
> come
> to clear and obvious conclusion/solution/etc. Please take a moment to
> review that section, and to follow up with an appropriate action on any API
> items which may have been the result of your own work or involve an area of
> the codebase with which you are suitably familiar.
>
r1462828 addresses the issues listed for svn_fs.h and svn_ra_svn.h.
Please review.
-- Stefan^2.
--
*Join one of our free daily demo sessions on* *Scaling Subversion for the
Enterprise <http://www.wandisco.com/training/webinars>*
*
*
Received on 2013-03-30 19:26:41 CET