+1
Sent from my iPhone
> On May 8, 2018, at 5:45 PM, Claes Redestad <claes.redes...@oracle.com> wrote:
>
> Hi,
>
> please review the removal of this test that explicitly attempts to run the
> now removed jlink --class-for-name plugin:
>
> diff -r 7f05a1e6a02a test/jdk/tools/jlink/JLinkPluginsTest.java
> --- a/test/jdk/tools/jlink/JLinkPluginsTest.java Tue May 08 15:42:19 2018
> +0200
> +++ b/test/jdk/tools/jlink/JLinkPluginsTest.java Tue May 08 22:46:14 2018
> +0200
> @@ -77,14 +77,6 @@
> helper.checkImage(imageDir, moduleName, res, null);
> }
> {
> - // Optimize Class.forName
> - String[] userOptions = {"--class-for-name"};
> - String moduleName = "classforname";
> - helper.generateDefaultJModule(moduleName, "composite2");
> - Path imageDir = helper.generateDefaultImage(userOptions,
> moduleName).assertSuccess();
> - helper.checkImage(imageDir, moduleName, null, null);
> - }
> - {
> // disable generate jli classes - JDK-8160063
> String[] userOptions = {"--disable-plugin",
> "generate-jli-classes"};
> String moduleName = "jlidisabled";
>
> /Claes
