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
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Wed Sep 21 01:24:59 2005