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

Re: svn commit: r1669769 - /subversion/branches/1.9.x/STATUS

From: Stefan Fuhrmann <stefan.fuhrmann_at_wandisco.com>
Date: Sun, 29 Mar 2015 20:20:00 +0200

On Sun, Mar 29, 2015 at 9:49 AM, Daniel Shahaf <d.s_at_daniel.shahaf.name>
wrote:

> stefan2_at_apache.org wrote on Sat, Mar 28, 2015 at 13:16:22 -0000:
> > Author: stefan2
> > Date: Sat Mar 28 13:16:21 2015
> > New Revision: 1669769
> >
> > URL: http://svn.apache.org/r1669769
> > Log:
> > * STATUS: Add 1.9 API doccu & declaration fixes r1668598 through
> r1668608.
> >
> > Modified:
> > subversion/branches/1.9.x/STATUS
> >
> > Modified: subversion/branches/1.9.x/STATUS
> > URL:
> http://svn.apache.org/viewvc/subversion/branches/1.9.x/STATUS?rev=1669769&r1=1669768&r2=1669769&view=diff
> >
> ==============================================================================
> > --- subversion/branches/1.9.x/STATUS (original)
> > +++ subversion/branches/1.9.x/STATUS Sat Mar 28 13:16:21 2015
> > @@ -231,6 +231,54 @@ Other candidate changes:
> > Votes:
> > +1: stefan2
> >
> > + * r1668598
> > + Fixes to the public svn_config API declarations and documentation.
> > + Justification:
> > + Have a complete, correct and easily accessible API description.
> > + None of these changes should affect generated code nor the ABI.
> > + Votes:
> > + +1: stefan2
>
> Voting is not required for documentation fixes, according to
> <
> https://subversion.apache.org/docs/community-guide/releasing.html#release-stabilization
> >.
>
> I think you can just move the doxygen fixes to the approved section.
> They're practically obvious fixes.
>

Good point. That reduces noise for reviewers.
I moved the docstring-only patches to "approved" now.

-- Stefan^2.
Received on 2015-03-29 20:20:29 CEST

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

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.