On Mon, 30 Sep 2024 08:02:16 GMT, Jaikiran Pai <j...@openjdk.org> wrote:

> Can I please get a review of this change, which simplifies the interaction 
> between the `java` launcher's native code with the 
> `sun.launcher.LauncherHelper`? 
> 
> As noted in https://bugs.openjdk.org/browse/JDK-8341184, this proposed change 
> reduces the back and forth between the launcher's native code and the 
> `LauncherHelper` class. This also removes the additional reflective lookups 
> from the native code after the main class and main method have been 
> determined by the `LauncherHelper`. 
> 
> Although this is a clean up of the code, the changes in the `LauncherHelper` 
> to return a `MainEntry` have been done in a way to facilitate additional 
> upcoming changes in this area, which propose to get rid of the JAR manifest 
> parsing from the launcher's native code.
> 
> No new tests have been added. Existing tests in tier1, tier2 and tier3 
> continue to pass.

This pull request has been closed without being integrated.

-------------

PR: https://git.openjdk.org/jdk/pull/21256

Reply via email to