[jdk7u-dev] Please review the fix for 7126832
Jonathan Gibbons
jonathan.gibbons at oracle.com
Wed Jan 25 17:22:21 PST 2012
Dalibor,
No, this is an internal method in javac we are talking about. No CCC is
required.
-- Jon
On 01/25/2012 05:03 PM, Dalibor Topic wrote:
> Would this be adding a new public API to JDK 7, i.e. need to go through CCC?
>
> cheers,
> dalibor topic
>
> On 1/25/12 10:29 AM, Jim Holmlund wrote:
>> Here is the bug:
>> http://bugs.sun.com/bugdatabase/view_bug.do?bug_id=7126832
>>
>> Here is the webrev
>> http://cr.openjdk.java.net/~jjh/7126832/
>>
>> The fix is the same as the fix in jdk8 with one exception:
>> - in JDK 8, the compile(...) methods in Main.java return Result whereas in JDK 7 they return int
>> - the patch adds a new compile method, so in 8 this new method is
>> public Result compile(String[] args, .....
>>
>> whereas in 7u-dev, it is
>> public int compile(String[] args, .....
>>
>> Here is the patch in jdk 8:
>> http://hg.openjdk.java.net/jdk8/tl/langtools/rev/ac36176b7de0
>>
>> Testing:
>> - langtools built and regression tests run on Windows
>>
>> Thanks
>> - jjh
>>
>>
>
More information about the compiler-dev
mailing list