db-derby-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From John Embretsen <John.Embret...@Sun.COM>
Subject Re: Test Harness useprocess=false question
Date Fri, 23 Dec 2005 08:51:32 GMT
John Embretsen wrote:

> Myrna van Lunteren wrote:
>> John,
>> I found that indeed, using mainMethod.invoke(null,arguments) was 
>> required and worked well. Thx.
> I'm glad to hear that!
> I noticed that you included this code change in your (rather large) 
> patch for DERBY-413. One consequence of this code change is that the 
> first parameter to RunClass.java ("Class theClass") is not used or 
> needed anymore, hence it should be removed (I don't think you included 
> this in your patch). This means that the classes instantiating a 
> RunClass object (RunTest.java at least, any others?) also need to be 
> modified somewhat. I guess it looks cleaner if we do this in a separate 
> patch - maybe even creating a new Jira issue - or what do you think?


For some reason, I did not get any e-mails from Jira with your comments 
  to DERBY-413 (could not find it in the derby-dev archives either), but 
I got the "[jira] Resolved" e-mail. I see you have implemented my 
suggestions in the patch that was committed by Andrew. Thanks for doing 
the work! :)


View raw message