On Fri, Mar 29, 2013 at 9:17 PM, C. Michael Pilato <cmpil...@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>* * *
