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

Re: [PATCH] dav_svn_get_repos_path

From: Julian Foad <julianfoad_at_btopenworld.com>
Date: 2005-04-01 18:12:40 CEST

C. Michael Pilato wrote:
> Marcus Rueckert <darix@web.de> writes:
>>Add a function to mod_dav_svn to get the path of the repository on diski.
>>This can be usefull for other apache modules.
>
> s/diski/disk/
> s/usefull/useful/
> s/apache/Apache/
[...]
> Noisy. You just need:
>
> Add a function to mod_dav_svn to get the path of the repository on disk.
> This can be useful for other Apache modules.
>
> * subversion/include/mod_dav_svn.h
> (dav_svn_get_repos_path): New.
>
> * subversion/mod_dav_svn/mod_dav_svn.c
> Include mod_dav_svn.h.
> (dav_svn_get_repos_path): New.
[...]
> Docstring is fine, I guess. Were it my patch, I might do the
> following instead:
[...]

Right. Given some or all of C. Michael Pilato's textual clean-ups, I'm happy
with this patch. However, it's not my area of expertise at all, and I can't
successfully run "make check" at the moment, so I'll have to leave this for
some other committer to commit.

- Julian

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Fri Apr 1 18:16:17 2005

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.