[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

Re: 1.8 new public API review (mostly) complete.

From: Stefan Fuhrmann <stefan.fuhrmann_at_wandisco.com>
Date: Sat, 30 Mar 2013 19:26:06 +0100

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

This is an archived mail posted to the Subversion Dev mailing list.