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

Re: [PATCH] Doc strings for WC log file processing functions

From: Julian Foad <julianfoad_at_btopenworld.com>
Date: Tue, 05 Aug 2008 22:43:23 +0100

On Tue, 2008-08-05 at 13:02 -0400, Karl Fogel wrote:
> Julian Foad <julianfoad_at_btopenworld.com> writes:
> > On Tue, 2008-07-29 at 17:14 +0100, Julian Foad wrote:
> >> While investigating Stefan Kung's crash, I found an opportunity to
> >> improve documentation in libsvn_wc. Every little helps. Could anyone
> >> read through and let me have any comments, especially on the three
> >> places I marked "###", before I commit?
> >>
> >> Log message and patch attached.
> >
> > Committed revision 32366, with only one "###" comment, the one noting
> > that file_xfer_under_path()'s "special_only" parameter is ignored.
> I looked into this a bit: r17495 is the "culprit".
> It seems r17495 just meant to undo some of r17362. And r17362
> apparently converted some inline code in file_xfer_under_copy() to call
> svn_wc_translated_file2() instead, and used 'special_only' to decide
> whether or not to pass SVN_WC_TRANSLATE_SPECIAL_ONLY flag to that
> function. When SVN_WC_TRANSLATE_SPECIAL_ONLY got removed in r17495,
> though, we didn't just revert to the old code in file_xfer_under_copy().

OK, thanks for looking.

> (Historical background: the special_only arg was added in r16645, and
> the dependent code introduced with it seems to have remained
> more-or-less in place until r17362.)
> r17495 says:
> Review by: julianfoad
> malcolm

Ha, so I was here before? I don't remember a thing about it :-)
> Note: Doc string updates still under discussion with Julian;
> to be committed later.
> Did this ever happen?

Yes, it did: <http://svn.haxx.se/dev/archive-2005-11/1330.shtml> and

- Julian

To unsubscribe, e-mail: dev-unsubscribe_at_subversion.tigris.org
For additional commands, e-mail: dev-help_at_subversion.tigris.org
Received on 2008-08-05 23:43:49 CEST

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.