RFR: 8202810: Remove class-for-name test

James Laskey james.laskey at oracle.com
Tue May 8 20:56:25 UTC 2018


+1

Sent from my iPhone

> On May 8, 2018, at 5:45 PM, Claes Redestad <claes.redestad at 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



More information about the jigsaw-dev mailing list