Mladen, 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. Thanks, -chris
signature.asc
Description: OpenPGP digital signature