Re: svn commit: r1456006 - in /subversion/trunk/subversion/bindings/javahl: native/ src/org/apache/subversion/javahl/ src/org/tigris/subversion/javahl/ tests/org/apache/subversion/javahl/

On Mar 16, 2013, at 6:41 AM, Branko Čibej <brane_at_wandisco.com> wrote:

> On 15.03.2013 17:57, Mark Phippard wrote:
>> Just starting to do some JavaHL review as requested.
>>
>> There is a problem with how this change was made. The rules for changing a
>> method are pretty much the same as they are for C. We cannot simply add
>> new options to the existing method we need to add those in a "move2" method.
>
> Instead of renaming, e.g., move -> move2, I've been adding additional
> overloads, to the same effect -- that the 1.7 interfaces remain unaffected.
>
> -- Brane
>

+1

I corrected myself to Philip but noticed I did not copy the list just as I hit Send.

Mark
Received on 2013-03-16 12:12:24 CET