March 2013 - core-libs-dev

hg: jdk8/tl: 8 new changesets
by lana.steuck＠oracle.com 06 Mar '13

06 Mar '13

Changeset: 91d35211e744 Author: katleman Date: 2013-02-21 11:12 -0800 URL: http://hg.openjdk.java.net/jdk8/tl/rev/91d35211e744 Added tag jdk8-b78 for changeset fd1a5574cf68 ! .hgtags Changeset: 85b5b4cc388c Author: katleman Date: 2013-02-28 10:41 -0800 URL: http://hg.openjdk.java.net/jdk8/tl/rev/85b5b4cc388c Added tag jdk8-b79 for changeset 91d35211e744 ! .hgtags Changeset: ab82853d3365 Author: erikj Date: 2013-02-21 14:16 +0100 URL: http://hg.openjdk.java.net/jdk8/tl/rev/ab82853d3365 8008451: Make mac builds on 10.8 work on 10.7 Reviewed-by: ohair, ddehaven ! common/autoconf/generated-configure.sh ! common/autoconf/spec.gmk.in ! common/autoconf/toolchain.m4 Changeset: d3e3d5b06f45 Author: ohair Date: 2013-02-23 10:47 -0800 URL: http://hg.openjdk.java.net/jdk8/tl/rev/d3e3d5b06f45 8004712: build-infra: Move user guide from web pages to repository Summary: Just the initial work, will need more changes. Reviewed-by: tbell ! README ! README-builds.html Changeset: 2778e6576e21 Author: katleman Date: 2013-02-26 13:23 -0800 URL: http://hg.openjdk.java.net/jdk8/tl/rev/2778e6576e21 Merge Changeset: 0adf9c626bb1 Author: katleman Date: 2013-02-28 20:29 -0800 URL: http://hg.openjdk.java.net/jdk8/tl/rev/0adf9c626bb1 Merge Changeset: c022bc48b7ed Author: lana Date: 2013-03-05 11:46 -0800 URL: http://hg.openjdk.java.net/jdk8/tl/rev/c022bc48b7ed Merge ! common/autoconf/generated-configure.sh ! common/autoconf/spec.gmk.in Changeset: c4901c0e0579 Author: lana Date: 2013-03-05 15:09 -0800 URL: http://hg.openjdk.java.net/jdk8/tl/rev/c4901c0e0579 Merge ! common/autoconf/generated-configure.sh ! common/autoconf/toolchain.m4

1 0

hg: jdk8/tl/jaxws: 2 new changesets
by lana.steuck＠oracle.com 06 Mar '13

06 Mar '13

Changeset: 70d8658d2a30 Author: katleman Date: 2013-02-21 11:13 -0800 URL: http://hg.openjdk.java.net/jdk8/tl/jaxws/rev/70d8658d2a30 Added tag jdk8-b78 for changeset 391de4c992d1 ! .hgtags Changeset: b0224010e2f0 Author: katleman Date: 2013-02-28 10:42 -0800 URL: http://hg.openjdk.java.net/jdk8/tl/jaxws/rev/b0224010e2f0 Added tag jdk8-b79 for changeset 70d8658d2a30 ! .hgtags

1 0

hg: jdk8/tl/corba: 3 new changesets
by lana.steuck＠oracle.com 06 Mar '13

06 Mar '13

Changeset: e41fb1aa0329 Author: katleman Date: 2013-02-21 11:12 -0800 URL: http://hg.openjdk.java.net/jdk8/tl/corba/rev/e41fb1aa0329 Added tag jdk8-b78 for changeset 27d6368ae8ba ! .hgtags Changeset: 5f3d4a6bdd02 Author: katleman Date: 2013-02-28 10:41 -0800 URL: http://hg.openjdk.java.net/jdk8/tl/corba/rev/5f3d4a6bdd02 Added tag jdk8-b79 for changeset e41fb1aa0329 ! .hgtags Changeset: 67ef27b4e16c Author: lana Date: 2013-03-05 11:46 -0800 URL: http://hg.openjdk.java.net/jdk8/tl/corba/rev/67ef27b4e16c Merge

1 0

hg: jdk8/tl: 8006988: build-infra: Configure fails if 'cl' is in path on linux
by martinrb＠google.com 05 Mar '13

05 Mar '13

Changeset: a9c8a32d09f9 Author: martin Date: 2013-03-05 13:16 -0800 URL: http://hg.openjdk.java.net/jdk8/tl/rev/a9c8a32d09f9 8006988: build-infra: Configure fails if 'cl' is in path on linux Summary: Respect user CC and CXX environment variables; use cl iff on windows Reviewed-by: erikj ! common/autoconf/generated-configure.sh ! common/autoconf/toolchain.m4

1 0

RE: RFR: 8006593: Performance and compatibility improvements to hash based Map implementations
by Salter, Thomas A 05 Mar '13

05 Mar '13

Will this change do anything to fix: http://bugs.sun.com/bugdatabase/view_bug.do?bug_id=8002283? -------------------- Date: Mon, 4 Mar 2013 12:21:08 -0800 From: Mike Duigou <mike.duigou(a)oracle.com> Subject: RFR: 8006593: Performance and compatibility improvements to hash based Map implementations To: "core-libs-dev(a)openjdk.java.net Libs" <core-libs-dev(a)openjdk.java.net> Message-ID: <79A5A703-12DA-4C66-94F7-34534761D217(a)oracle.com> Content-Type: text/plain; charset=us-ascii Hello all; The alternative hashing implementation introduced in 7u6 added an unfortunate bottleneck to the initialization of HashMap and Hashtable instances. This patch avoids the performance bottleneck of using a shared Random instance by using a ThreadLocalRandom instead. Along with this change are some additional performance improvements to further reduce the overhead of the alternative hashing feature and generally improve the performance of Hashtable or HashMap. http://cr.openjdk.java.net/~mduigou/JDK-8006593/3/webrev/ Once review is completed here this patch will be proposed to JDK7u-dev for integration into the next 7u performance/feature release. Mike

2 1

Review Request: BigInteger patch for efficient multiplication and division (#4837946)
by Tim Buktu 05 Mar '13

05 Mar '13

It's been over a year since my original patch, and it's been over 3 years since Alan posted his patch. I think I'll stop holding my breath. Tim >/ From: Brian Burkhalter <brian.burkhalter at oracle.com <http://mail.openjdk.java.net/mailman/listinfo/core-libs-dev>> />/ Subject: Re: core-libs-dev Digest, Vol 68, Issue 113 />/ Date: January 2, 2013 5:02:10 PM PST />/ To: core-libs-dev at openjdk.java.net <http://mail.openjdk.java.net/mailman/listinfo/core-libs-dev> />/ />/ Hello Tim, />/ />/ Thank you for the updates. We'll take a closer look at this as soon as practically feasible. I have only recently been transferred into the core-libs area and expect to be dealing with the learning curve for some time to come. />/ />/ Regards, />/ />/ Brian />/ />>/ Date: Sat, 29 Dec 2012 15:18:01 +0100 />>/ From: Tim Buktu <tbuktu at hotmail.com <http://mail.openjdk.java.net/mailman/listinfo/core-libs-dev>> />>/ Subject: Review Request: BigInteger patch for efficient multiplication />>/ and division (#4837946) />>/ To: core-libs-dev at openjdk.java.net <http://mail.openjdk.java.net/mailman/listinfo/core-libs-dev> />>/ Message-ID: <BLU0-SMTP2745F8AA2A40C750C8B0113C63E0 at phx.gbl <http://mail.openjdk.java.net/mailman/listinfo/core-libs-dev>> />>/ Content-Type: text/plain; charset="ISO-8859-15" />>/ />>/ Hello, />>/ />>/ I updated my patch that speeds up multiplication and division of large />>/ BigIntegers. The changes vs. the older patch were trivial. />>/ />>/ BigInteger.java: https://gist.github.com/1576025 />>/ Diff against current BigInteger: https://gist.github.com/1576016 />>/ />>/ The patch to the unit test still applies. />>/ BigIntegerTest.java: https://gist.github.com/1586990 />>/ Diff against current BigIntegerTest.java: https://gist.github.com/1586984 />>/ />>/ See the original posts at />>/ http://mail.openjdk.java.net/pipermail/core-libs-dev/2012-January/008860.ht… />>/ and />>/ http://mail.openjdk.java.net/pipermail/core-libs-dev/2012-January/008875.ht… />>/ />>/ Comments and critique are appreciated. />>/ />>/ Thanks, />>/ Tim /

4 8

Proposal: Put @Native and @GenerateNativeHeader in same package
by Paul Benedict 05 Mar '13

05 Mar '13

Compare the two javadocs: @java.lang.annotation.Native: "Indicates that a field defining a constant value may be referenced from native code. The annotation may be used as a hint by tools that generate native header files to determine whether a header file is required, and if so, what declarations it should contain." @javax.tools.annotation.GenerateNativeHeader: "An annotation used to indicate that a native header file should be generated for this class. Normally, the presence of native methods is a sufficient indication of the need for a native header file. However, in some cases, a class may contain constants of interest to native code, without containing any native methods." Both are hints for tools. Both discuss header files. So why divide them up? I think they should be in one package together. Personally, I think both are better suited to the javax.tools.annotation package. I just wanted to point out the division doesn't make all that sense. Paul

3 6

hg: jdk8/tl/langtools: 4 new changesets
by maurizio.cimadamore＠oracle.com 05 Mar '13

05 Mar '13

Changeset: 69cd2bfd4a31 Author: mcimadamore Date: 2013-03-05 14:04 +0000 URL: http://hg.openjdk.java.net/jdk8/tl/langtools/rev/69cd2bfd4a31 8004962: Code generation crash with lambda and local classes Summary: Translation info should be propagated from LambdaToMethod to Lower Reviewed-by: jjg, rfield ! src/share/classes/com/sun/tools/javac/comp/LambdaToMethod.java ! src/share/classes/com/sun/tools/javac/comp/Lower.java + test/tools/javac/lambda/LambdaCapture07.java Changeset: d2a98dde7ecc Author: mcimadamore Date: 2013-03-05 14:12 +0000 URL: http://hg.openjdk.java.net/jdk8/tl/langtools/rev/d2a98dde7ecc 8009227: Certain diagnostics should not be deferred Summary: Add new diagnostic flag to mark non deferrable diagnostics Reviewed-by: jjg ! src/share/classes/com/sun/tools/javac/comp/Check.java ! src/share/classes/com/sun/tools/javac/comp/DeferredAttr.java ! src/share/classes/com/sun/tools/javac/util/JCDiagnostic.java ! src/share/classes/com/sun/tools/javac/util/Log.java + test/tools/javac/lambda/abort/CompletionFailure.java Changeset: a708c5f1da06 Author: mcimadamore Date: 2013-03-05 14:16 +0000 URL: http://hg.openjdk.java.net/jdk8/tl/langtools/rev/a708c5f1da06 8009154: Missing cast in method reference bridge leads to NoSuchMethodError Summary: Missing cast in generated method reference bridge Reviewed-by: rfield, jjg ! src/share/classes/com/sun/tools/javac/comp/LambdaToMethod.java + test/tools/javac/lambda/MethodReference65.java Changeset: 12202e6ab78a Author: mcimadamore Date: 2013-03-05 14:19 +0000 URL: http://hg.openjdk.java.net/jdk8/tl/langtools/rev/12202e6ab78a 8009129: Illegal access error when calling method reference Summary: Javac generates method handle referencing non public type Reviewed-by: jjg, rfield ! src/share/classes/com/sun/tools/javac/comp/Attr.java ! src/share/classes/com/sun/tools/javac/comp/LambdaToMethod.java ! src/share/classes/com/sun/tools/javac/resources/compiler.properties ! src/share/classes/com/sun/tools/javac/tree/JCTree.java + test/tools/javac/diags/examples/NotDefPublicCantAccessFragment/NotDefPublicCantAccessFragment.java + test/tools/javac/diags/examples/NotDefPublicCantAccessFragment/p/C.java + test/tools/javac/lambda/inaccessibleMref01/InaccessibleMref01.java + test/tools/javac/lambda/inaccessibleMref01/InaccessibleMref01.out + test/tools/javac/lambda/inaccessibleMref01/p1/C.java + test/tools/javac/lambda/inaccessibleMref02/InaccessibleMref02.java + test/tools/javac/lambda/inaccessibleMref02/p1/C.java

1 0

hg: jdk8/tl/jdk: 4880778: URL final class has protected methods
by chris.hegarty＠oracle.com 05 Mar '13

05 Mar '13

Changeset: be79440b8026 Author: jzavgren Date: 2013-03-05 09:50 -0500 URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/be79440b8026 4880778: URL final class has protected methods Summary: The two set() methods have been defined to be package private. Reviewed-by: alanb, chegar, khazra ! src/share/classes/java/net/URL.java ! src/share/classes/java/net/URLStreamHandler.java

1 0

hg: jdk8/tl/jdk: 8008804: file descriptor leak in src/windows/native/java/net/DualStackPlainSocketImpl.c
by chris.hegarty＠oracle.com 05 Mar '13

05 Mar '13

Changeset: b5bef1f71de6 Author: jzavgren Date: 2013-03-05 14:30 +0000 URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/b5bef1f71de6 8008804: file descriptor leak in src/windows/native/java/net/DualStackPlainSocketImpl.c Reviewed-by: alanb, chegar, dsamersoff ! src/windows/native/java/net/DualStackPlainDatagramSocketImpl.c

1 0