On Fri, Mar 29, 2013 at 9:17 PM, C. Michael Pilato <cmpilato_at_collab.net>wrote:
> 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
> 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:
> 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
> 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.
*Join one of our free daily demo sessions on* *Scaling Subversion for the
Received on 2013-03-30 19:26:41 CET