hg: jdk8/tl: 7132204: Default testset in JPRT should not run all tests

2012-01-24 Thread alan . bateman
Changeset: 0f653ee93477
Author:alanb
Date:  2012-01-24 09:08 +
URL:   http://hg.openjdk.java.net/jdk8/tl/rev/0f653ee93477

7132204: Default testset in JPRT should not run all tests
Reviewed-by: ohair

! make/jprt.properties



hg: jdk8/tl/jdk: 7132204: Default testset in JPRT should not run all tests

2012-01-24 Thread alan . bateman
Changeset: 237319a01a9a
Author:alanb
Date:  2012-01-24 09:09 +
URL:   http://hg.openjdk.java.net/jdk8/tl/jdk/rev/237319a01a9a

7132204: Default testset in JPRT should not run all tests
Reviewed-by: ohair

! make/jprt.properties
! test/ProblemList.txt



hg: jdk8/tl/jdk: 7132386: makefile support for tracing/Java Flight Recorder framework phase I

2012-01-24 Thread rickard . backman
Changeset: 718bca4e685f
Author:rbackman
Date:  2012-01-17 16:20 +0100
URL:   http://hg.openjdk.java.net/jdk8/tl/jdk/rev/718bca4e685f

7132386: makefile support for tracing/Java Flight Recorder framework phase I
Reviewed-by: ohair, dholmes, rottenha
Contributed-by: Markus Gronlund markus.gronl...@oracle.com, Erik Gahlin 
erik.gah...@oracle.com, Nils Loodin nils.loo...@oracle.com, Rickard Backman 
rickard.back...@oracle.com, Staffan Larsen staffan.lar...@oracle.com

! make/com/oracle/Makefile
+ make/com/oracle/jfr/Makefile
! make/common/Defs.gmk
! make/common/Release.gmk



hg: jdk8/tl/langtools: 7129801: Merge the two method applicability routines

2012-01-24 Thread maurizio . cimadamore
Changeset: 51fb17abfc32
Author:mcimadamore
Date:  2012-01-24 17:52 +
URL:   http://hg.openjdk.java.net/jdk8/tl/langtools/rev/51fb17abfc32

7129801: Merge the two method applicability routines
Summary: Resolve.java and Infer.java should reuse the same method applicability 
check routine
Reviewed-by: dlsmith, jjg

! src/share/classes/com/sun/tools/javac/comp/Infer.java
! src/share/classes/com/sun/tools/javac/comp/Resolve.java
! src/share/classes/com/sun/tools/javac/resources/compiler.properties
+ test/tools/javac/diags/examples/InferVarargsArgumentMismatch.java



hg: jdk8/tl/jdk: 7132270: tools/launcher/DefaultLocaleTestRun.java failing (win)

2012-01-24 Thread kumar . x . srinivasan
Changeset: f64ea40293db
Author:ksrini
Date:  2012-01-24 09:58 -0800
URL:   http://hg.openjdk.java.net/jdk8/tl/jdk/rev/f64ea40293db

7132270: tools/launcher/DefaultLocaleTestRun.java failing (win)
Reviewed-by: alanb, chegar

! test/tools/launcher/DefaultLocaleTestRun.java
! test/tools/launcher/TestHelper.java



hg: jdk8/tl/jdk: 7132879: address Findbugs issue in WebRowSetXmlWriter

2012-01-24 Thread lance . andersen
Changeset: 303b67074666
Author:lancea
Date:  2012-01-24 15:13 -0500
URL:   http://hg.openjdk.java.net/jdk8/tl/jdk/rev/303b67074666

7132879: address Findbugs issue in WebRowSetXmlWriter
Reviewed-by: forax

! src/share/classes/com/sun/rowset/internal/WebRowSetXmlWriter.java



Re: Code review request [JDK 8]: 7132248, sun/security/ssl/sun/net/www/protocol/https/HttpsURLConnection/CookieHttpsClientTest.java failing

2012-01-24 Thread Brad Wetmore

Looks good also...

Brad



On 1/23/2012 4:27 AM, Xuelei Fan wrote:

Remove the serviceabilty-dev.

Thanks for the quick code review.

Xuelei

On 1/23/2012 8:25 PM, Alan Bateman wrote:

On 23/01/2012 12:21, Xuelei Fan wrote:

Webrev: http://cr.openjdk.java.net/~xuelei/7132248/webrev.00/

In JDK 8, the regression tests of JSSE (HTTP/TLS) run in agentvm
mode. In agentvm mode, multiple threads may share the thread pool.
SunJSSE implementation initialize the SSL/TLS context at the first
time the context get loaded, and will not dynamically change the
context anymore after the initialization. If a test case has
initialized the context, another test case share the same thread will
use the same context. New settings in the later will have no impact
on the context.

The cause of the bug is that some other test case updated the
context, and CookieHttpsClientTest cannot setup the context as
expected. Need to make the test case run in othervm mode.

Thanks,
Xuelei

This looks fine to me, thanks for jumping on this annoying test failure.

-Alan.

PS: cc'ing security-dev as I'm guessing you cc'ed serviceabilty-dev in
error.




hg: jdk8/tl/langtools: 7126832: com.sun.tools.javac.api.ClientCodeWrapper$WrappedJavaFileManager cannot be cast

2012-01-24 Thread jim . holmlund
Changeset: ac36176b7de0
Author:jjh
Date:  2012-01-24 15:51 -0800
URL:   http://hg.openjdk.java.net/jdk8/tl/langtools/rev/ac36176b7de0

7126832: com.sun.tools.javac.api.ClientCodeWrapper$WrappedJavaFileManager 
cannot be cast
Reviewed-by: jjg

! src/share/classes/com/sun/tools/javac/api/JavacTaskImpl.java
! src/share/classes/com/sun/tools/javac/main/Main.java
+ test/tools/javah/T7126832/T7126832.java
+ test/tools/javah/T7126832/java.java



hg: jdk8/tl/langtools: 7129225: javac fails to run annotation processors when star import of package of gensrc

2012-01-24 Thread jim . holmlund
Changeset: d16b464e742c
Author:jjh
Date:  2012-01-24 16:31 -0800
URL:   http://hg.openjdk.java.net/jdk8/tl/langtools/rev/d16b464e742c

7129225: javac fails to run annotation processors when star import of package 
of gensrc
Reviewed-by: jjg

! src/share/classes/com/sun/tools/javac/comp/MemberEnter.java
+ test/tools/javac/7129225/Anno.java
+ test/tools/javac/7129225/AnnoProcessor.java
+ test/tools/javac/7129225/NegTest.ref
+ test/tools/javac/7129225/TestImportStar.java
+ test/tools/javac/7129225/TestImportStar.ref