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

Re: svn commit: r12738 - in trunk/subversion: include libsvn_subr mod_dav_svn

From: Peter N. Lundblad <peter_at_famlundblad.se>
Date: 2005-01-16 12:58:27 CET

On Sun, 15 Jan 2005, C. Michael Pilato wrote:

> Wow, take a day trip, and see what you miss out on...
>
> For the record, yes, in a way I was fixing symptoms. It turns out
> that most bugs have symptoms, and most symptoms that bugs have are the
> kind you want to fix. But I didn't make a single change to
> Subversion's code that wasn't in line with what I feel the Complete
> Right solution should be. I just didn't have time to go all the way.

First, let's just drop this "fixing symptoms" part of the discussion. I
started that, and I hereby wirthdraw it.

You exposed an internal function as an API that doesn't do what the
docstring says it does. If you don't have time to fix the implementation,
then please add a note to the docstring saying that it is incomplete.

Regards,
//Peter

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Sun Jan 16 12:59:43 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.