tomcat-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Christopher Schultz <>
Subject Re: Using reflection to call tcnative methods
Date Sat, 09 Jul 2011 14:24:42 GMT

On 7/8/2011 11:44 PM, Mladen Turk wrote:
> On 07/09/2011 04:14 AM, Christopher Schultz wrote:
>> All,
>> I've been working in AprLifecycleListener to add FIPS support. I need to
>> make an additional native method call in initializeSSL and I'm wondering
>> why all of the native methods are called using reflection instead of
>> direct method calls.
> Think that originates from the initial code when the java part of
> native was not part of tomcat code.
> We needed a way to make (at that time a separate .jar) tomcat uses
> it without run time dependency.

That makes sense.

>> If there's no good reason, then all the reflective-call clutter should
>> be removed.
> +1.
> There is absolutely no reason to use the reflection any more.

Okay. I have a patch I'd like to make that should be as small as
possible in general, so I think I shouldn't change the existing
reflective calls, at least not in this patch.

Should I therefore code my patch to match the existing code (using
reflection) and then write another patch to remove the reflection, or
should my initial patch just avoid the reflection and the code will have
two different calling strategies until both patches are applied?

(I'm more concerned with backporting to TC 6 than my initial patches
which are of course targeted towards TC 7).

I'll wait for another opinion or two before I make any commits.


View raw message