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

Jonathan Gibbons jonathan.gibbons at oracle.com
Fri May 10 08:09:41 PDT 2013


Ditto by me

-- jon

On 05/10/2013 07:44 AM, Mandy Chung wrote:
> 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