hg: jdk8/tl/langtools: 8026758: Inefficient code in LambdaToMethod

2013-10-21 Thread kumar . x . srinivasan
Changeset: 87c950ea88be Author:ksrini Date: 2013-10-21 20:10 -0700 URL: http://hg.openjdk.java.net/jdk8/tl/langtools/rev/87c950ea88be 8026758: Inefficient code in LambdaToMethod Reviewed-by: jjg, jlahoda, rfield ! src/share/classes/com/sun/tools/javac/comp/LambdaToMethod.java

Re: [8] 8012636: OCSP validation fails even when public key is trusted

2013-10-21 Thread Xuelei Fan
OCSPRequest.java line 79-80, Note that Debug.getInstance("certpath") may return null. Looks like don't need dump variable any more. OCSPResponse.java line 134-135. - Debug.getInstance("certpath") may return null. May not need dump variable. line 299:

hg: jdk8/tl/jdk: 3 new changesets

2013-10-21 Thread jason . uh
Changeset: 975e3a89814e Author:darcy Date: 2013-10-21 13:36 -0700 URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/975e3a89814e 8024603: Turn on javac lint checking for auxiliaryclass, empty, and try in jdk build Reviewed-by: erikj, ihse, chegar ! makefiles/Setup.gmk Changeset: f4

hg: jdk8/tl/jdk: 8025238: nsk/jvmti/scenarios/bcinstr/BI04/bi04t002 crashed with SIGSEGV

2013-10-21 Thread coleen . phillimore
Changeset: f581b72e3715 Author:sla Date: 2013-10-21 23:32 -0400 URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/f581b72e3715 8025238: nsk/jvmti/scenarios/bcinstr/BI04/bi04t002 crashed with SIGSEGV Summary: Redefined class in stack trace may not be found by method_idnum so handle nu

hg: jdk8/tl/langtools: 8026855: AnnoConstruct.getAnnotationsByType includes inherited indirectly present annotations even when containee type is not inheritable

2013-10-21 Thread joel . franck
Changeset: ac839d6f4953 Author:jfranck Date: 2013-10-22 03:36 +0200 URL: http://hg.openjdk.java.net/jdk8/tl/langtools/rev/ac839d6f4953 8026855: AnnoConstruct.getAnnotationsByType includes inherited indirectly present annotations even when containee type is not inheritable Summary:

Re: [8] Review Request for 8021191: Add isAuthorized check to limited doPrivileged methods

2013-10-21 Thread Weijun Wang
In AccessController.java, existing doc always has a "specified" between "the" and "{@code AccessControlContext}". BTW, maybe a little off-topic, I am not sure about the exact meaning of "with no permissions". jre/lib/security/java.policy has granted some permissions (e.g. reading "java.version

Re: [8] Review Request for 8021191: Add isAuthorized check to limited doPrivileged methods

2013-10-21 Thread Xuelei Fan
It's a behavior update. Looks fine to me. Xuelei On 10/19/2013 5:24 AM, Sean Mullan wrote: > Please review this change to the limited doPrivileged methods to check > that the passed-in access control context is authorized before using. > > http://cr.openjdk.java.net/~mullan/webrevs/8021191/

hg: jdk8/tl/jdk: 8008386: (cs) Unmappable leading should be decoded to replacement.

2013-10-21 Thread xueming . shen
Changeset: 3b00bf85a6f5 Author:sherman Date: 2013-10-21 18:22 -0700 URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/3b00bf85a6f5 8008386: (cs) Unmappable leading should be decoded to replacement. Summary: updated the unmappable/malformed detecting handling for db charsets Reviewed-b

[8] Request for Review: 8025287: NPE in api/java_security/cert/PKIXRevocationChecker/GeneralTests_GeneralTests

2013-10-21 Thread Jason Uh
Hi Sean, Could I please get a review of this changeset? The change checks for null params when RevocationChecker.init is called. Bug: https://bugs.openjdk.java.net/browse/JDK-8025287 Webrev: http://cr.openjdk.java.net/~juh/8025287/webrev.00/ Thanks, Jason

hg: jdk8/tl/langtools: 8026984: Clarity intended use of jdk.Exported

2013-10-21 Thread joe . darcy
Changeset: b82982ac3ca2 Author:darcy Date: 2013-10-21 15:37 -0700 URL: http://hg.openjdk.java.net/jdk8/tl/langtools/rev/b82982ac3ca2 8026984: Clarity intended use of jdk.Exported Reviewed-by: psandoz, mr, alanb ! src/share/classes/jdk/Exported.java

Re: Code Review Request: 8025763

2013-10-21 Thread Sean Mullan
On 10/18/2013 10:52 PM, Anthony Scarpino wrote: I've updated the webrev http://cr.openjdk.java.net/~ascarpino/8025763/webrev.01/ Update looks good. --Sean

hg: jdk8/tl/jdk: 7140929: NotSerializableNotifTest.java fails intermittently

2013-10-21 Thread jaroslav . bachorik
Changeset: 698baf22e081 Author:jbachorik Date: 2013-10-21 13:57 +0200 URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/698baf22e081 7140929: NotSerializableNotifTest.java fails intermittently Reviewed-by: sjiang, alanb ! test/javax/management/remote/mandatory/notif/NotSerializableNo

hg: jdk8/tl/nashorn: 8026871: NASHORN TEST: Enable possibility to test Nashorn use of JavaFX canvas.

2013-10-21 Thread konstantin . shefov
Changeset: f22742d5daa3 Author:kshefov Date: 2013-10-21 13:31 +0400 URL: http://hg.openjdk.java.net/jdk8/tl/nashorn/rev/f22742d5daa3 8026871: NASHORN TEST: Enable possibility to test Nashorn use of JavaFX canvas. Reviewed-by: jlaskey, sundar ! make/build.xml ! make/project.properti

hg: jdk8/tl/jdk: 7197919: java/lang/management/ThreadMXBean/ThreadBlockedCount.java has concurency issues

2013-10-21 Thread jaroslav . bachorik
Changeset: 30c46debdf0f Author:jbachorik Date: 2013-10-21 10:40 +0200 URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/30c46debdf0f 7197919: java/lang/management/ThreadMXBean/ThreadBlockedCount.java has concurency issues Reviewed-by: sla, mchung ! test/java/lang/management/ThreadMX

hg: jdk8/tl/jdk: 8024613: javax/management/remote/mandatory/connection/RMIConnector_NPETest.java failing intermittently

2013-10-21 Thread jaroslav . bachorik
Changeset: d8694ad1ed2d Author:jbachorik Date: 2013-10-21 10:54 +0200 URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/d8694ad1ed2d 8024613: javax/management/remote/mandatory/connection/RMIConnector_NPETest.java failing intermittently Summary: RMID needs a varying amount of time to

hg: jdk8/tl/hotspot: 8026893: Push 8026365 to TL early and add test

2013-10-21 Thread karen . kinnear
Changeset: e39b138b2518 Author:acorn Date: 2013-10-19 18:32 -0400 URL: http://hg.openjdk.java.net/jdk8/tl/hotspot/rev/e39b138b2518 8026893: Push 8026365 to TL early and add test Reviewed-by: dcubed, kamg ! src/share/vm/classfile/verifier.cpp ! test/TEST.groups + test/runtime/802636

[8] 8012636: OCSP validation fails even when public key is trusted

2013-10-21 Thread Vincent Ryan
Please review this fix to support key-rollover certs (same name, different keys): Bug: https://bugs.openjdk.java.net/browse/JDK-8012636 Webrev: http://cr.openjdk.java.net/~vinnie/8012636/webrev.00/ This issue arises when an OCSP responder replaces its public key but retains its subject name. The

hg: jdk8/tl/jdk: 8020037: String.toLowerCase incorrectly increases length, if string contains \u0130 char

2013-10-21 Thread yuka . kamiya
Changeset: e8683d5b2b0a Author:peytoia Date: 2013-10-22 06:13 +0900 URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/e8683d5b2b0a 8020037: String.toLowerCase incorrectly increases length, if string contains \u0130 char Reviewed-by: naoto ! src/share/classes/java/lang/ConditionalSpe

hg: jdk8/tl/jdk: 8022658: Revisit FunctionalInterface on some core libs types

2013-10-21 Thread joe . darcy
Changeset: c1700125d041 Author:darcy Date: 2013-10-21 12:52 -0700 URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/c1700125d041 8022658: Revisit FunctionalInterface on some core libs types Reviewed-by: briangoetz, mduigou, mr ! src/share/classes/java/io/Closeable.java ! src/share/cl

hg: jdk8/tl/jdk: 8026842: Remove Time-Zone IDs HST/EST/MST

2013-10-21 Thread xueming . shen
Changeset: f0c18a5e3ae5 Author:sherman Date: 2013-10-21 11:16 -0700 URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/f0c18a5e3ae5 8026842: Remove Time-Zone IDs HST/EST/MST Summary: removed these ids from ZoneId's zid list, supported via short_ids list Reviewed-by: okutsu ! make/tool

Re: InvocationTargetException when running applet locally

2013-10-21 Thread Sean Mullan
I think this is a known issue. There is a workaround documented in the JDK 7u45 release notes, can you try it out? See "JNLP applet fails to load if using JNLP versioning.": http://www.oracle.com/technetwork/java/javase/7u45-relnotes-2016950.html --Sean On 10/18/2013 04:44 PM, Mickey Segal wr

hg: jdk8/tl/langtools: 8026956: test tools/javac/lambda/TargetType58.java is failing after a libs change

2013-10-21 Thread vicente . romero
Changeset: 9f876bd43f55 Author:vromero Date: 2013-10-21 15:55 +0100 URL: http://hg.openjdk.java.net/jdk8/tl/langtools/rev/9f876bd43f55 8026956: test tools/javac/lambda/TargetType58.java is failing after a libs change Reviewed-by: jfranck ! test/tools/javac/lambda/TargetType58.java

hg: jdk8/tl/jdk: 8026499: Root Logger level can be reset unexpectedly

2013-10-21 Thread daniel . fuchs
Changeset: c81125493ca6 Author:dfuchs Date: 2013-10-21 12:00 +0200 URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/c81125493ca6 8026499: Root Logger level can be reset unexpectedly Summary: This fix prevents the logger's level to be re-initialized if it has already been initialized

hg: jdk8/tl/jdk: 8016344: (props) Properties.storeToXML behaviour has changed from JDK 6 to 7

2013-10-21 Thread daniel . fuchs
Changeset: 567d47fd3fe2 Author:dfuchs Date: 2013-10-21 11:15 +0200 URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/567d47fd3fe2 8016344: (props) Properties.storeToXML behaviour has changed from JDK 6 to 7 Summary: When storing Properties to XML only locally defined properties must b

hg: jdk8/tl/nashorn: 2 new changesets

2013-10-21 Thread sundararajan . athijegannathan
Changeset: d8aa87d292eb Author:hannesw Date: 2013-10-18 22:42 +0200 URL: http://hg.openjdk.java.net/jdk8/tl/nashorn/rev/d8aa87d292eb 8026858: Array length does not handle defined properties correctly Reviewed-by: jlaskey ! src/jdk/nashorn/internal/codegen/Lower.java ! src/jdk/nasho