Re: svn commit: r28102 - trunk/subversion/svnserve
Karl Fogel wrote:
> email@example.com writes:
>> Fix an annoying insconsistency inside svnserve, where its internal
>> 'fs_path' variable sometimes had a leading slash and sometimes didn't.
> (I already propchanged the "insconsistency" above, btw.)
>> * subversion/svnserve/server.h
>> (server_baton_t): Note the 'fs_path' always has a leading slash.
>> * subversion/svnserve/serve.c
>> (get_fs_path): Ensure that returned 'fs_path' always has a leading slash.
>> (find_repos): Ensure that baton's 'fs_path' always has a leading slash.
> As long as we're here:
> * subversion/svnserve/serve.c
> (get_fs_path, find_repos): Ensure that returned 'fs_path' always
> has a leading slash.
> ...saves the reader from having to read the same text twice, and
> emphasizes that it's the same change in two places.
But, but ... it's not! One is a returned path, another is a path stuffed
into a baton.
> Ack, this function's doc string was already deficient, but after this
> it became even more so :-). I've updated it in r28105.
C. Michael Pilato <firstname.lastname@example.org>
CollabNet <> www.collab.net <> Distributed Development On Demand
Received on Wed Nov 28 07:15:31 2007
This is an archived mail posted to the Subversion Dev