Re: svn commit: r1069961 - in /subversion/trunk/subversion: include/svn_diff.h libsvn_diff/diff.c libsvn_diff/diff.h libsvn_diff/diff3.c libsvn_diff/diff4.c libsvn_diff/diff_file.c libsvn_diff/diff_memory.c libsvn_diff/lcs.c

On Sat, Feb 19, 2011 at 8:03 PM, Ivan Zhakov <ivan_at_visualsvn.com> wrote:
> On Sat, Feb 12, 2011 at 00:38,  <jcorvel_at_apache.org> wrote:
>> Author: jcorvel
>> Date: Fri Feb 11 21:38:55 2011
>> New Revision: 1069961
>>
>> URL: http://svn.apache.org/viewvc?rev=1069961&view=rev
>> Log:
>> Re-enable elimination of the identical suffix for diff (and diff3 (merge) and
>> diff4), which was disabled in r1068613 because of truncation of suffix lines
>> during merge.
>>
>> This time, count the number of suffix lines so we can create an entirely
>> correct LCS, also containing the common suffix, so diff3 (merge) can do its
>> work properly.
>>
>> * subversion/include/svn_diff.h
>>  (svn_diff_fns2_t): Add a new output parameter SUFFIX_LINES to the
>>   datasources_open prototype. The svn_diff_fns2_t struct is new in 1.7 (was
>>   already revved), so we can still do this.
>>
> Hi Johan,
>
> It seems libsvn_diff\deprecated.c:datasources_open() should be updated
> to new signature. I get the following warning when compiling:
> [[
> ..\..\..\subversion\libsvn_diff\deprecated.c(128) : warning C4113:
> 'svn_error_t *(__cdecl *)(void *,apr_off_t *,const
> svn_diff_datasource_e *,apr_size_t)' differs in parameter lists from
> 'svn_error_t *(__cdecl *)(void *,apr_off_t *,apr_off_t *,const
> svn_diff_datasource_e *,apr_size_t)'
> ]]

Ohhh, I knew I had forgotten something. Thanks for catching this.

Fixed in r1072447

-- 
Johan