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

RE: svn commit: r38186 - trunk/subversion/include

From: Bert Huijben <rhuijben_at_sharpsvn.net>
Date: Wed, 24 Jun 2009 16:56:15 +0200

> -----Original Message-----
> From: Stefan Sperling [mailto:stsp_at_elego.de]
> Sent: woensdag 24 juni 2009 16:45
> To: svn_at_subversion.tigris.org
> Subject: svn commit: r38186 - trunk/subversion/include
>
> Author: stsp
> Date: Wed Jun 24 07:45:20 2009
> New Revision: 38186
>
> Log:
> * subversion/include/svn_client.h
> (svn_client_commit_item3_t): New member 'adm_access', so every commit
> item can refer to its adm_access easily.
> (svn_client_commit_item3_dup): Change docstring to indicate
> that the adm_access member will be shared between copies.
>
> Patch-by: HuiHuang <yellow.flying_at_yahoo.com.cn>
> (log message tweaked by me)
>
> Modified:
> trunk/subversion/include/svn_client.h
>
> Modified: trunk/subversion/include/svn_client.h
> URL:
> http://svn.collab.net/viewvc/svn/trunk/subversion/include/svn_client.h?
> pathrev=38186&r1=38185&r2=38186
> =======================================================================
> =======
> --- trunk/subversion/include/svn_client.h Wed Jun 24 07:36:23 2009
> (r38185)
> +++ trunk/subversion/include/svn_client.h Wed Jun 24 07:45:20 2009
> (r38186)
> @@ -465,6 +465,9 @@ typedef struct svn_client_commit_item3_t
> * same lifetime as this data structure.
> */
> apr_array_header_t *outgoing_prop_changes;
> +
> + /** adm_access of this item */
> + svn_wc_adm_access_t *adm_access;
> } svn_client_commit_item3_t;

This addition needs a @since doxygen tag, and probably a '###' suggesting
that this field should be removed once we have one global svn_wc_context_t,
that can handle multiple working copies.

(And personally I would prefer not to hand off adm_access instances to third
party handlers of the logmessage callback, because you are not supposed to
do any working copy operations in this state).

        Bert

------------------------------------------------------
http://subversion.tigris.org/ds/viewMessage.do?dsForumId=462&dsMessageId=2364961
Received on 2009-06-24 16:56:43 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.