Approved. On Jan 26, 2012, at 12:50 AM, Jim Holmlund wrote:
> <<Hopefully, this can be approved before 6pm Wed PST. Lana's cut off for the > next build>> > > 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 > > Both the JDK 8 and the JDK 7u fixes were approved by Jon Gibbons > > Testing: > - langtools built and regression tests run on Windows > > Thanks > - jjh > >
