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

Re: svn commit: r1188256 - in /subversion/trunk/subversion: include/ libsvn_client/ libsvn_delta/ libsvn_ra_neon/ libsvn_ra_serf/ libsvn_ra_svn/ libsvn_repos/ libsvn_wc/ svnrdump/ svnsync/

From: Hyrum K Wright <hyrum.wright_at_wandisco.com>
Date: Mon, 24 Oct 2011 15:20:51 -0500

On Mon, Oct 24, 2011 at 3:13 PM, Daniel Shahaf <d.s_at_daniel.shahaf.name> wrote:
> Hyrum K Wright wrote on Mon, Oct 24, 2011 at 15:09:52 -0500:
>> On Mon, Oct 24, 2011 at 2:11 PM, Daniel Shahaf <d.s_at_daniel.shahaf.name> wrote:
>> > Hyrum K Wright wrote on Mon, Oct 24, 2011 at 14:01:20 -0500:
>> >> On Mon, Oct 24, 2011 at 1:13 PM, Daniel Shahaf <d.s_at_daniel.shahaf.name> wrote:
>> >> > hwright_at_apache.org wrote on Mon, Oct 24, 2011 at 17:48:07 -0000:
>> >> >> Author: hwright
>> >> >> Date: Mon Oct 24 17:48:06 2011
>> >> >> New Revision: 1188256
>> >> >>
>> >> >> URL: http://svn.apache.org/viewvc?rev=1188256&view=rev
>> >> >> Log:
>> >> >> Create a struct to hold the ever-growing collection of Ev2 shim callbacks.
>> >> >>
>> >> >> * subversion/include/svn_delta.h
>> >> >>   (svn_delta_shim_callbacks_t, svn_delta_shim_callbacks_default): New.
>> >> >>   (svn_editor__insert_shims): Use the struct in place of individual params.
>> >> >>
>> >> >
>> >> > svn_delta_shim_callbacks_dup()?
>> >>
>> >> I really hope this code never sees the light of a release.  We can add
>> >> that function if/when we need it.
>> >>
>> >
>> > Then put it in include/private/ ?
>>
>> gen-make.py complains loudly if a public header depends on a private one.
>>
>
> Then move svn_editor__insert_shims() to a private header too...?  Does
> it need to be public?

Unknown. This is all still very much work-in-progress. (To which you
are of course welcome to contribute!)

-Hyrum

>
>> >
>> >> -Hyrum
>> >>
>> >> >> * subversion/libsvn_ra_svn/editorp.c,
>> >> >>   subversion/libsvn_ra_neon/commit.c,
>> >> >>   subversion/svnsync/main.c,
>> >> >>   subversion/svnrdump/dump_editor.c,
>> >> >>   subversion/libsvn_wc/diff_editor.c,
>> >> >>   subversion/libsvn_wc/update_editor.c,
>> >> >>   subversion/libsvn_wc/status.c,
>> >> >>   subversion/libsvn_client/repos_diff.c,
>> >> >>   subversion/libsvn_client/mergeinfo.c,
>> >> >>   subversion/libsvn_client/export.c,
>> >> >>   subversion/libsvn_ra_serf/commit.c,
>> >> >>   subversion/libsvn_repos/commit.c,
>> >> >>   subversion/libsvn_repos/dump.c:
>> >> >>     Use the new struct and update callers to the shim API.
>> >> >>
>> >> >> * subversion/libsvn_delta/compat.c
>> >> >>   (svn_delta_shim_callbacks_default): New.
>> >> >>   (svn_editor__insert_shims): Use the new datatype.
>> >> >
>> >>
>> >>
>> >>
>> >> --
>> >>
>> >> uberSVN: Apache Subversion Made Easy
>> >> http://www.uberSVN.com/
>> >
>>
>>
>>
>> --
>>
>> uberSVN: Apache Subversion Made Easy
>> http://www.uberSVN.com/
>

-- 
uberSVN: Apache Subversion Made Easy
http://www.uberSVN.com/
Received on 2011-10-24 22:21:23 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.