>> - apr_hash_set(*result, mergedfrom, APR_HASH_KEY_STRING,
>> + apr_hash_set(*result, lastmergedfrom,
>> APR_HASH_KEY_STRING,
>
> Really? I think *this* change would cause wrong paths mapped to wrong
> rev-ranges. Do you have any tests to prove the need for this change?
I don't have a python testsuite to prove.
But you can use the small dump at
http://puggy.symonds.net/~kameshj/my_test_area_multi_merge.dmp
(mergetracking svn) cp url/to/trunk url/to/branches/somethingelse -m
'blah blah'
Observe the property of 'svn:mergeinfo' on /trunk and
/branches/somethingelse to see the defect.
Apply the patch and observe the same you will get what I mean.
>
> Apart from that, I do really see a problem where the first merged from
> path will never get set into the *result hash. To counter this, we
> should have the if condition as below:
>
>> if (lastmergedfrom == NULL || strcmp(mergedfrom,
>> lastmergedfrom) != 0)
if (lastmergedfrom && strcmp(mergedfrom, lastmergedfrom) !=
0) (I terribly fail to see the difference!).
Observe the code carefully you will see nothing is lost!.
With regards
Kamesh Jayachandran
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Thu Nov 2 07:23:35 2006