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?
Myrna,
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! :)
--
John
