8014318: tools/javac/profiles/ProfileOptionTest.java needs modifying now that javax.script is in compact1

Mandy Chung mandy.chung at oracle.com
Fri May 10 07:44:53 PDT 2013


Looks good to me.

Mandy

On 5/10/2013 3:00 AM, Alan Bateman wrote:
>
> A few days ago we pushed the changes to add javax.script to the 
> compact1 profile (it was previously in compact3).
>
> One thing that I missed is the javac test for the "javac -profile" 
> option. It uses sample types in each profile and just happens to use a 
> type in javax.script. I'd like to fix this with the attached patch. 
> It's a change trivial that selects another type that we know is in 
> compact3.
>
> Thanks,
>
> -Alan.
>
>
> diff --git a/test/tools/javac/profiles/ProfileOptionTest.java 
> b/test/tools/javac/profiles/ProfileOptionTest.java
> --- a/test/tools/javac/profiles/ProfileOptionTest.java
> +++ b/test/tools/javac/profiles/ProfileOptionTest.java
> @@ -179,7 +179,7 @@
>                  javax.xml.XMLConstants.class);
>
>          init(Profile.COMPACT3,
> -                javax.script.Bindings.class,
> +                javax.sql.rowset.Predicate.class,
>                  com.sun.security.auth.PolicyFile.class); // 
> specifically included in 3
>
>          init(Profile.DEFAULT,



More information about the compiler-dev mailing list