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

Re: svn commit: r38394 - in trunk/subversion: include libsvn_delta

From: Neels Janosch Hofmeyr <neels_at_elego.de>
Date: Sun, 12 Jul 2009 00:33:58 +0200

Julian Foad wrote:
> On Thu, 2009-07-09 at 15:53 -0700, Neels Janosch Hofmeyr wrote:
>> Author: neels
>> Date: Thu Jul 9 15:53:33 2009
>> New Revision: 38394
>>
>> Log:
>> Enable an atomic replace in editor v2.
>>
>> * subversion/include/svn_editor.h
>> (svn_editor_add_directory_t, svn_editor_add_directory,
>> svn_editor_add_file_t, svn_editor_add_file,
>> svn_editor_add_symlink_t, svn_editor_add_symlink,
>> svn_editor_add_absent_t, svn_editor_add_absent):
>> Add parameter REPLACES_REV.
>
> Where is this parameter documented?

That's a very reasonable question.

So far, svn_editor.h's *only* comment is:

[[[
/** An abstract object that edits a target tree.
 *
 * ### more docco
 */
]]]

It felt kind of silly to only document that parameter in the midst of void.
And I am too lazy right now to document all of them.
However, I added a rough description in notes/editor-v2.txt in r38396.

~Neels

>
> - Julian
>
>> * subversion/libsvn_delta/editor.c
>> (svn_editor_add_directory,
>> svn_editor_add_file,
>> svn_editor_add_symlink,
>> svn_editor_add_absent):
>> Add parameter REPLACES_REV and pass to callbacks.
>
> ------------------------------------------------------
> http://subversion.tigris.org/ds/viewMessage.do?dsForumId=462&dsMessageId=2369635

------------------------------------------------------
http://subversion.tigris.org/ds/viewMessage.do?dsForumId=462&dsMessageId=2370705

Received on 2009-07-12 00:35:10 CEST

This is an archived mail posted to the Subversion Dev mailing list.