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

Re: [PATCH] Fix javahl crash in mergeinfo.c

From: Paul Burba <ptburba_at_gmail.com>
Date: Fri, 21 Aug 2009 23:25:53 -0400

Dave,

Fixed this in r38911. The correct baton is being passed to the
callback again, the JavaHL tests no longer segfault, and r38832 still
does what it should.

Paul

On Fri, Aug 21, 2009 at 8:48 PM, Dave Brown<dave.brown_at_wandisco.com> wrote:
> Please disregard this patch, it will break some mergeinfo changes from
> pburba.  There's some non-standard baton passing in mergeinfo.c, but
> pburba is aware and is on it.
>
> -dave
>
> Dave Brown wrote:
>> [[[
>> Fix crash in javahl tests. A callback to LogMessageCallback.cpp was
>> being passed a bad baton.  This seems to be just a small typo from
>> r38832 to this file.
>>
>> * subversion/libsvn_client/mergeinfo.c
>>    (filter_log_entry_with_rangelist): Fix baton passing through
>>    this function, use correct field from filter_log_entry_baton_t
>> ]]]
>>
>> ------------------------------------------------------
>> http://subversion.tigris.org/ds/viewMessage.do?dsForumId=462&dsMessageId=2386168
>>
>
> ------------------------------------------------------
> http://subversion.tigris.org/ds/viewMessage.do?dsForumId=462&dsMessageId=2386179
>

------------------------------------------------------
http://subversion.tigris.org/ds/viewMessage.do?dsForumId=462&dsMessageId=2386202
Received on 2009-08-22 05:26:44 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.