hg: jdk8/tl/langtools: 8010010: NPE generating serializedLambdaName for nested lambda

2013-03-14 Thread robert . field
Changeset: fbbf5376e7e4 Author:rfield Date: 2013-03-14 22:54 -0700 URL: http://hg.openjdk.java.net/jdk8/tl/langtools/rev/fbbf5376e7e4 8010010: NPE generating serializedLambdaName for nested lambda Reviewed-by: mcimadamore ! src/share/classes/com/sun/tools/javac/comp/LambdaToMethod.

RFR (M): 7198429: need checked categorization of caller-sensitive methods in the JDK

2013-03-14 Thread Christian Thalinger
[This is the HotSpot part of JEP 176] http://cr.openjdk.java.net/~twisti/7198429 7198429: need checked categorization of caller-sensitive methods in the JDK Reviewed-by: More information in JEP 176: http://openjdk.java.net/jeps/176 src/share/vm/ci/ciMethod.cpp src/share/vm/ci/ciMethod.hpp src/

Re: RFR 8005696: Add CompletableFuture - JEP 155

2013-03-14 Thread Martin Buchholz
Doing a mini-audit of these methods, here is a change to what seems to be the intent: Index: ./java/util/concurrent/CompletableFuture.java === RCS file: /export/home/jsr166/jsr166/jsr166/src/main/java/util/concurrent/CompletableFuture

hg: jdk8/tl/jdk: 8008576: Calendar mismatch using Host LocaleProviderAdapter

2013-03-14 Thread naoto . sato
Changeset: d79503c4c56f Author:naoto Date: 2013-03-14 11:29 -0700 URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/d79503c4c56f 8008576: Calendar mismatch using Host LocaleProviderAdapter Reviewed-by: okutsu ! make/java/java/FILES_java.gmk ! src/macosx/classes/sun/util/locale/provid

hg: jdk8/tl/langtools: 8005428: Update jdeps to read the same profile information as by javac

2013-03-14 Thread mandy . chung
Changeset: fd3fdaff0257 Author:mchung Date: 2013-03-14 10:33 -0700 URL: http://hg.openjdk.java.net/jdk8/tl/langtools/rev/fd3fdaff0257 8005428: Update jdeps to read the same profile information as by javac Reviewed-by: alanb ! make/netbeans/langtools/nbproject/project.xml ! src/shar

RFR 8005696: Add CompletableFuture - JEP 155

2013-03-14 Thread Chris Hegarty
As part of JEP 155 (Concurrency Updates) we are proposing to add CompletableFuture. A Future that may be explicitly completed (setting its value and status), and may include dependent functions and actions that trigger upon its completion. Specdiff: http://cr.openjdk.java.net/~chegar/8005696/

Re: @Supported design issues

2013-03-14 Thread Joe Darcy
FYI, I've submitted a JEP reviewed by Alan Bateman on "Capturing support and stability information about JDK classes in source and class files." -Joe

hg: jdk8/tl/jdk: 7183800: TEST_BUG: Update tests to run on Ubuntu 12.04 (localhost is 127.0.1.1)

2013-03-14 Thread alan . bateman
Changeset: ca9469a15792 Author:alanb Date: 2013-03-14 16:59 + URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/ca9469a15792 7183800: TEST_BUG: Update tests to run on Ubuntu 12.04 (localhost is 127.0.1.1) Reviewed-by: alanb, chegar Contributed-by: yiming.w...@oracle.com ! test/ja

Re: Review Request for 8005428: Update jdeps to read the same profile information as by javac

2013-03-14 Thread Alan Bateman
On 14/03/2013 16:56, Mandy Chung wrote: Thanks for the review. Updated webrev: http://cr.openjdk.java.net/~mchung/jdk8/webrevs/8005428/webrev.01/ This looks good to me. -Alan

hg: jdk8/tl/jdk: 8005716: Enhance JNI specification to allow support of static JNI libraries in Embedded JREs

2013-03-14 Thread alan . bateman
Changeset: f010eb1e696f Author:alanb Date: 2013-03-14 16:03 + URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/f010eb1e696f 8005716: Enhance JNI specification to allow support of static JNI libraries in Embedded JREs Reviewed-by: dlong, alanb, mduigou Contributed-by: bill.pitt..

Re: Review Request for 8005428: Update jdeps to read the same profile information as by javac

2013-03-14 Thread Mandy Chung
On 3/14/13 8:20 AM, Alan Bateman wrote: On 13/03/2013 20:23, Mandy Chung wrote: This fix removes the temporary workaround for jdeps to print the profile information. The profiles information is embedded in ct.sym generated during the jdk build. jdeps will read ct.sym and parse the annotation

Re: [JSR310 M7 Review request] 8007572: Replace existing jdk timezone data at /lib/zi with JSR310's tzdb.

2013-03-14 Thread Andrew Hughes
- Original Message - > On 14/03/2013 12:51 PM, Xueming Shen wrote: > > This should have been addessed via > > > > JDK-8008348 *The leftover jdk/make/tools/javazic causes build > > problems > > with hs25-b19 control job* > > > > Which is stil

Re: Please review fix for JDK-8010083: Fix ASM doc comments to avoid javadoc errors

2013-03-14 Thread Andrew Hughes
- Original Message - > On 14/03/2013 14:03, A. Sundararajan wrote: > > Please review http://cr.openjdk.java.net/~sundar/8010083/ > > > > Thanks > > -Sundar > This looks okay to me but do we want to be changing the ASM source? > It > has a few compiler warnings too. Maybe Rémi or someone clo

Re: Please review fix for JDK-8010083: Fix ASM doc comments to avoid javadoc errors

2013-03-14 Thread Alan Bateman
On 14/03/2013 14:03, A. Sundararajan wrote: Please review http://cr.openjdk.java.net/~sundar/8010083/ Thanks -Sundar This looks okay to me but do we want to be changing the ASM source? It has a few compiler warnings too. Maybe Rémi or someone closer to ASM could get the issues fixed upstream (

Re: Review Request for 8005428: Update jdeps to read the same profile information as by javac

2013-03-14 Thread Alan Bateman
On 13/03/2013 20:23, Mandy Chung wrote: This fix removes the temporary workaround for jdeps to print the profile information. The profiles information is embedded in ct.sym generated during the jdk build. jdeps will read ct.sym and parse the annotation to determine which profile a class belon

Re: Please review fix for JDK-8010083: Fix ASM doc comments to avoid javadoc errors

2013-03-14 Thread Hannes Wallnoefer
Looks good to me. (Note that I'm not a core-libs reviewer though.) Hannes Am 2013-03-14 15:03, schrieb A. Sundararajan: Please review http://cr.openjdk.java.net/~sundar/8010083/ Thanks -Sundar

Please review fix for JDK-8010083: Fix ASM doc comments to avoid javadoc errors

2013-03-14 Thread A. Sundararajan
Please review http://cr.openjdk.java.net/~sundar/8010083/ Thanks -Sundar

hg: jdk8/tl/langtools: 8008582: jtreg failures after conversion of shell tests to Java

2013-03-14 Thread vicente . romero
Changeset: 2e21ecd7a5ad Author:vromero Date: 2013-03-14 08:30 + URL: http://hg.openjdk.java.net/jdk8/tl/langtools/rev/2e21ecd7a5ad 8008582: jtreg failures after conversion of shell tests to Java Reviewed-by: jjg ! src/share/classes/com/sun/tools/javac/util/ArrayUtils.java ! tes