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

Re: svn commit: r34149 - trunk/subversion/bindings/javahl/native

From: Mark Phippard <markphip_at_gmail.com>
Date: Wed, 12 Nov 2008 11:36:13 -0500

On Wed, Nov 12, 2008 at 11:28 AM, Hyrum K. Wright
<hyrum_wright_at_mail.utexas.edu> wrote:
> Mark Phippard wrote:
>> On Wed, Nov 12, 2008 at 11:13 AM, Hyrum K. Wright
>> <hyrum_wright_at_mail.utexas.edu> wrote:
>>> I'm concerned about this change. I understand that it removes a deprecated
>>> function warning, but the warning is there for a purpose. The Right Way to
>>> solve this problem would be to allow JavaHL to provide the right callbacks for
>>> the updated APIs. Can we revert this change as a reminder to do it right?
>> I am not aware of all the code involved, but how are we able to use
>> the new functions at all if it is not being done the "right way". If
>> you are concerned, then I am concerned too.
> The new functions take NULL arguments in place of the callbacks and basically
> behave as the old functions, so there shouldn't be any functional change with
> this patch.

I hope that is not indicative of a trend in our code in terms of
removing the usage of deprecated functions. If so, I am not sure
these changes we have made to identify deprecated functions is much of
a step forward.

Mark Phippard
To unsubscribe, e-mail: dev-unsubscribe_at_subversion.tigris.org
For additional commands, e-mail: dev-help_at_subversion.tigris.org
Received on 2008-11-12 17:36:31 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.