December 2012 - core-libs-dev

hg: jdk8/tl/jdk: 8003881: Prevent lambda implementing inner classes from allowing the creation of new instances
by robert.field＠oracle.com 07 Dec '12

07 Dec '12

Changeset: 896d4af2ebfd Author: rfield Date: 2012-12-06 21:55 -0800 URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/896d4af2ebfd 8003881: Prevent lambda implementing inner classes from allowing the creation of new instances Summary: Lambda implementing inner classes now has private constructor (thanks Kumar) Reviewed-by: ksrini ! src/share/classes/java/lang/invoke/InnerClassLambdaMetafactory.java + test/java/lang/invoke/lambda/LambdaAccessControlDoPrivilegedTest.java + test/java/lang/invoke/lambda/LambdaAccessControlTest.java

1 0

hg: jdk8/tl: 8004685: add java.util.function to CORE_PKGS.gmk
by mike.duigou＠oracle.com 06 Dec '12

06 Dec '12

Changeset: fb1bf5e5bc9e Author: henryjen Date: 2012-12-06 15:38 -0800 URL: http://hg.openjdk.java.net/jdk8/tl/rev/fb1bf5e5bc9e 8004685: add java.util.function to CORE_PKGS.gmk Reviewed-by: mduigou ! common/makefiles/javadoc/CORE_PKGS.gmk

1 0

hg: jdk8/tl/jdk: 8004374: CachedRowSetSwriter.writeData reports wrong number of conflicts in SyncProviderException
by lance.andersen＠oracle.com 06 Dec '12

06 Dec '12

Changeset: 41a1b110f34d Author: lancea Date: 2012-12-06 15:51 -0500 URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/41a1b110f34d 8004374: CachedRowSetSwriter.writeData reports wrong number of conflicts in SyncProviderException Reviewed-by: naoto ! src/share/classes/com/sun/rowset/internal/CachedRowSetWriter.java

1 0

Re: Review request: 8003562: Provide a command-line tool to find static dependencies
by Mandy Chung 06 Dec '12

06 Dec '12

Thanks Erik for the help. I missed the files for images build. I'll incorporate your patch. Mandy On 12/6/2012 6:37 AM, Erik Joelsson wrote: > Hello, > > I've looked at the build changes and noted that the old and new build > did not produce the same results. Here is a webrev with adjustments (I > chose to do a full forest webrev because I needed to change a file in > the root too): > > http://cr.openjdk.java.net/~erikj/8003562/webrev.03/ > > The changes are: > > * Add the jdeps package to tools.jar in new build > * Exclude jdeps from jre in new build > * Add "-DEXPAND_CLASSPATH_WILDCARDS > -DNEVER_ACT_AS_SERVER_CLASS_MACHINE" to compile in old build. I wasn't > sure if these were intended for this launcher, but I'm guessing that > it makes sense to match the settings of javap and javah. The other > alternative is to remove these from both old and new build. > * Add ./bin/jdeps to some exception lists in compare script. (jdeps, > like most executables, contains the version string.) > > With these changes, I don't see any new regressions in the compare > script on my machine. > > /Erik > > On 2012-12-06 02:36, Mandy Chung wrote: >> This review request (add a new jdeps tool in the JDK) include changes in >> langtools and the jdk build. I would need reviewers from the langtools >> and the build-infra team. >> >> Webrev for review: >> http://cr.openjdk.java.net/~mchung/jdk8/webrevs/jdeps/langtools-webrev.02/ >> http://cr.openjdk.java.net/~mchung/jdk8/webrevs/jdeps/jdk-webrev.02/ >> >> The jdeps source is in the langtools repo. The change in the jdk >> repo is >> to add the new jdeps executable. I made change in the old and new build >> for the addition of this new jdeps tool. I discussed with Jon whether I >> should modify langtools/make/build.xml to add a new jdeps target. Since >> the old build will be replaced by the new build soon, I simply add >> com/sun/tools/jdeps as part of javap.includes. >> >> Alan, >> >> The -d option is kept as a hidden option and use as implementation. We >> can remove it when it's determined not useful in the future. I also >> rename -v:summary to -summary. >> >> Thanks >> Mandy >> >> ---------------------------- >> >> $ jdep -h >> Usage: jdeps <options> <files....> >> where possible options include: >> -version Version information >> -classpath <path> Specify where to find class files >> -summary Print dependency summary only >> -v:class Print class-level dependencies >> -v:package Print package-level dependencies >> -p <package name> Restrict analysis to classes in this package >> (may be given multiple times) >> -e <regex> Restrict analysis to packages matching >> pattern >> (-p and -e are exclusive) >> -P --profile Show profile or the file containing a package >> -R --recursive Traverse all dependencies recursively >> -all Process all classes specified in -classpath >> >> $ jdep Notepad.jar Ensemble.jar >> Notepad.jar -> D:\tools\devtools\jdk8\windows-i586\jre\lib\rt.jar >> <unnamed> (Notepad.jar) >> -> java.awt >> -> java.awt.event >> -> java.beans >> -> java.io >> -> java.lang >> -> java.net >> -> java.util >> -> java.util.logging >> -> javax.swing >> -> javax.swing.border >> -> javax.swing.event >> -> javax.swing.text >> -> javax.swing.tree >> -> javax.swing.undo >> >> Ensemble.jar -> D:\tools\devtools\jdk8\windows-i586\jre\lib\jfxrt.jar >> Ensemble.jar -> D:\tools\devtools\jdk8\windows-i586\jre\lib\rt.jar >> com.javafx.main (Ensemble.jar) >> -> java.applet >> -> java.awt >> -> java.awt.event >> -> java.io >> -> java.lang >> -> java.lang.reflect >> -> java.net >> -> java.security >> -> java.util >> -> java.util.jar >> -> javax.swing >> -> sun.misc JDK internal API >> (rt.jar)

1 0

RFR: 8004317 TestLibrary.getUnusedRandomPort() fails intermittently, but exception not reported
by Jim Gish 06 Dec '12

06 Dec '12

Please review http://cr.openjdk.java.net/~jgish/Bug8004317-TestLibrary-getUnusedRandomPor… <http://cr.openjdk.java.net/%7Ejgish/Bug8004317-TestLibrary-getUnusedRandomP…> We're getting intermittent failures on a number of RMI tests attempting to get an unused ephemeral port. This is a temporary change to print a stack trace until we can figure what's going on. For some reason the test framework isn't surfacing the nested exception. (There probably should be a separate bug for that, but my first priority here is to fix the intermittent test failures.) Thanks, Jim -- Jim Gish | Consulting Member of Technical Staff | +1.781.442.0304 Oracle Java Platform Group | Core Libraries Team 35 Network Drive Burlington, MA 01803 jim.gish(a)oracle.com

4 13

hg: jdk8/tl/jdk: 8004317: TestLibrary.getUnusedRandomPort() fails intermittently, but exception not reported
by stuart.marks＠oracle.com 06 Dec '12

06 Dec '12

Changeset: a971516029ab Author: jgish Date: 2012-12-05 21:08 -0800 URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/a971516029ab 8004317: TestLibrary.getUnusedRandomPort() fails intermittently, but exception not reported Reviewed-by: alanb, dmocek, smarks ! test/java/rmi/testlibrary/TestLibrary.java

1 0

hg: jdk8/tl/jdk: 8004491: Build breakage on Linux due to 8004188
by alan.bateman＠oracle.com 05 Dec '12

05 Dec '12

Changeset: b54a5b7d2e65 Author: alanb Date: 2012-12-05 12:20 +0000 URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/b54a5b7d2e65 8004491: Build breakage on Linux due to 8004188 Reviewed-by: chegar, erikj ! makefiles/CopyFiles.gmk

1 0

hg: jdk8/tl/jdk: 8004188: Rename src/share/lib/security/java.security to java.security-linux
by sean.mullan＠oracle.com 04 Dec '12

04 Dec '12

Changeset: 44ae777564eb Author: mullan Date: 2012-12-04 17:40 -0500 URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/44ae777564eb 8004188: Rename src/share/lib/security/java.security to java.security-linux Reviewed-by: mullan, mchung Contributed-by: jason.uh(a)oracle.com ! make/java/security/Makefile - src/share/lib/security/java.security + src/share/lib/security/java.security-linux

1 0

hg: jdk8/tl/jdk: 8003596: TEST_BUG: java/util/logging/CheckLockLocationTest.java failing [win]
by alan.bateman＠oracle.com 04 Dec '12

04 Dec '12

Changeset: 7004848974a2 Author: jgish Date: 2012-12-04 20:21 +0000 URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/7004848974a2 8003596: TEST_BUG: java/util/logging/CheckLockLocationTest.java failing [win] Reviewed-by: alanb ! test/ProblemList.txt ! test/java/util/logging/CheckLockLocationTest.java

1 0

hg: jdk8/tl/langtools: 8004360: regression test DefaultMethodRegressionTests fails in langtools
by maurizio.cimadamore＠oracle.com 04 Dec '12

04 Dec '12

Changeset: 0e70eb71fec0 Author: mcimadamore Date: 2012-12-04 17:19 +0000 URL: http://hg.openjdk.java.net/jdk8/tl/langtools/rev/0e70eb71fec0 8004360: regression test DefaultMethodRegressionTests fails in langtools Summary: ignore broken failing test Reviewed-by: jjg - test/tools/javac/defaultMethodExecution/DefaultMethodRegressionTests.java + test/tools/javac/defaultMethods/defaultMethodExecution/DefaultMethodRegressionTests.java

1 0