Re: svn commit: r16045 - trunk/subversion/libsvn_repos

On 9/20/05, Daniel Rall <dlr@finemaltcoding.com> wrote:
> On Mon, 19 Sep 2005, kfogel@collab.net wrote:
>
> > danderson@tigris.org writes:
> ...
> > > --- trunk/subversion/libsvn_repos/repos.c (original)
> > > +++ trunk/subversion/libsvn_repos/repos.c Sun Sep 4 18:02:12 2005
> > > @@ -1404,7 +1404,8 @@
> > > APR_EOL_STR
> > > "### Uncomment the line below to use the default authorization file."
> > > APR_EOL_STR
> > > - "# authz-db = authz"
> > > + "# authz-db = "
> > > + SVN_REPOS__CONF_AUTHZ
> > > APR_EOL_STR
> > > "### This option specifies the authentication realm of the repository."
> > > APR_EOL_STR
> >
> > Looks great, but why not put it on the same line, so the output is
> > more easily deducible from the input? E.g.:
> >
> > "# authz-db = " SVN_REPOS__CONF_AUTHZ
>
> +1
Committed in r16209.

Cheers,

David

--
David James -- http://www.cs.toronto.edu/~james
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org