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

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/

From: Philip Martin <philip.martin_at_wandisco.com>
Date: Fri, 15 Mar 2013 17:04:34 +0000

Mark Phippard <markphip_at_gmail.com> writes:

> 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.

Hmm, I didn't realise we had strict guidelines for JavaHL. Looking at
the other functions I see overloading is used rather than a new name.

-- 
Certified & Supported Apache Subversion Downloads:
http://www.wandisco.com/subversion/download
Received on 2013-03-15 18:05:19 CET

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.