On Mon, Feb 20, 2012 at 9:13 PM, Daniel Shahaf <d.s_at_daniel.shahaf.name> wrote:
> hwright_at_apache.org wrote on Tue, Feb 21, 2012 at 03:02:03 -0000:
>> Author: hwright
>> Date: Tue Feb 21 03:02:03 2012
>> New Revision: 1291568
>>
>> URL: http://svn.apache.org/viewvc?rev=1291568&view=rev
>> Log:
>> Ev2 shims: Add a no-op shim callback registration function for ra_local.
>>
>> * subversion/libsvn_ra_local/ra_plugin.c
>> Â (svn_ra_local__register_editor_shim_callbacks): New.
>> Â (ra_local_vtable): Add new callback to the vtable.
>>
>> Modified:
>> Â Â subversion/trunk/subversion/libsvn_ra_local/ra_plugin.c
>>
>> Modified: subversion/trunk/subversion/libsvn_ra_local/ra_plugin.c
>> URL: http://svn.apache.org/viewvc/subversion/trunk/subversion/libsvn_ra_local/ra_plugin.c?rev=1291568&r1=1291567&r2=1291568&view=diff
>> ==============================================================================
>> --- subversion/trunk/subversion/libsvn_ra_local/ra_plugin.c (original)
>> +++ subversion/trunk/subversion/libsvn_ra_local/ra_plugin.c Tue Feb 21 03:02:03 2012
>> @@ -1505,6 +1505,15 @@ svn_ra_local__get_deleted_rev(svn_ra_ses
>> Â Â return SVN_NO_ERROR;
>> Â }
>>
>> +static svn_error_t *
>> +svn_ra_local__register_editor_shim_callbacks(svn_ra_session_t *session,
>> + Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â svn_delta_shim_callbacks_t *callbacks)
>> +{
>> + Â /* This is currenly a no-op, since we don't provide our own editor, just
>> + Â Â use the one the libsvn_repos hands back to us. */
>
> Â Â *callbacks = NULL;
>
> ?
No. callbacks in an input parameters, and even though this function
doesn't make use if it, it certainly should go modifying it.
-Hyrum
>
>> + Â return SVN_NO_ERROR;
>> +}
>> +
>> Â /*----------------------------------------------------------------*/
>>
>> Â static const svn_version_t *
>> @@ -1551,7 +1560,8 @@ static const svn_ra__vtable_t ra_local_v
>> Â Â svn_ra_local__replay,
>> Â Â svn_ra_local__has_capability,
>> Â Â svn_ra_local__replay_range,
>> - Â svn_ra_local__get_deleted_rev
>> + Â svn_ra_local__get_deleted_rev,
>> + Â svn_ra_local__register_editor_shim_callbacks
>> Â };
>>
>>
>>
>>
--
uberSVN: Apache Subversion Made Easy
http://www.uberSVN.com/
Received on 2012-02-21 04:16:29 CET