August 2012 - core-libs-dev

hg: jdk8/tl/jdk: 7192202: Make sure keytool prints both unknown and unparseable extensions
by weijun.wang＠oracle.com 20 Aug '12

20 Aug '12

Changeset: 86c963b1dbf8 Author: weijun Date: 2012-08-20 07:59 +0800 URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/86c963b1dbf8 7192202: Make sure keytool prints both unknown and unparseable extensions Reviewed-by: mullan + test/sun/security/tools/keytool/UnknownAndUnparseable.java

1 0

hg: jdk8/tl/jdk: 7191467: (fs) WatchService periodically fails to queue ENTRY_DELETE event for short lived file [sol11]
by alan.bateman＠oracle.com 19 Aug '12

19 Aug '12

Changeset: 5e7cfe034df4 Author: alanb Date: 2012-08-19 13:29 +0100 URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/5e7cfe034df4 7191467: (fs) WatchService periodically fails to queue ENTRY_DELETE event for short lived file [sol11] Reviewed-by: chegar ! src/solaris/classes/sun/nio/fs/SolarisWatchService.java ! test/java/nio/file/WatchService/MayFlies.java

1 0

hg: jdk8/tl/jdk: 7192275: Minimize LogManager dependencies on java.beans
by alan.bateman＠oracle.com 19 Aug '12

19 Aug '12

Changeset: a2359f0f9533 Author: alanb Date: 2012-08-19 13:03 +0100 URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/a2359f0f9533 7192275: Minimize LogManager dependencies on java.beans Summary: Reduce dependency to PropertyChangeListener and PropertyChangeEvent. Also add basic test coverage. Reviewed-by: dcubed, dsamersoff, mchung ! src/share/classes/java/util/logging/LogManager.java + test/java/util/logging/Listeners.java + test/java/util/logging/ListenersWithSM.java + test/java/util/logging/java.policy

1 0

hg: jdk8/tl/jdk: 7190750: (pack200) the java unpacker produces non spec. compliant classfile with lambda classfiles
by kumar.x.srinivasan＠oracle.com 18 Aug '12

18 Aug '12

Changeset: 16f2865aac24 Author: ksrini Date: 2012-08-17 08:28 -0700 URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/16f2865aac24 7190750: (pack200) the java unpacker produces non spec. compliant classfile with lambda classfiles Reviewed-by: jrose ! src/share/classes/com/sun/java/util/jar/pack/ClassWriter.java

1 0

hg: jdk8/tl/langtools: 7192449: fix up tests to accommodate jtreg spec change
by jonathan.gibbons＠oracle.com 18 Aug '12

18 Aug '12

Changeset: 5ac2e9ee969e Author: jjg Date: 2012-08-17 17:30 -0700 URL: http://hg.openjdk.java.net/jdk8/tl/langtools/rev/5ac2e9ee969e 7192449: fix up tests to accommodate jtreg spec change Reviewed-by: darcy ! test/tools/javac/processing/6414633/T6414633.java ! test/tools/javac/processing/options/testPrintProcessorInfo/TestWithXstdout.java

1 0

hg: jdk8/tl/jdk: 7191322: add test for 7064927 to java.lang.instrument
by daniel.daugherty＠oracle.com 17 Aug '12

17 Aug '12

Changeset: 509421263cdd Author: dcubed Date: 2012-08-17 12:51 -0700 URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/509421263cdd 7191322: add test for 7064927 to java.lang.instrument Summary: Add support for canRetransform attribute to the test manager. Add test for 7064927. Reviewed-by: dsamersoff, sspitsyn, ohair ! test/java/lang/instrument/ATransformerManagementTestCase.java + test/java/lang/instrument/DummyClassWithLVT.java + test/java/lang/instrument/VerifyLocalVariableTableOnRetransformTest.java + test/java/lang/instrument/VerifyLocalVariableTableOnRetransformTest.sh + test/java/lang/instrument/retransformAgent.mf

1 0

hg: jdk8/tl/jdk: 6500133: REGRESSION: CertificateParsingException for CRL Distribution Point with blank
by sean.mullan＠oracle.com 17 Aug '12

17 Aug '12

Changeset: 6b2ebf3c4964 Author: mullan Date: 2012-08-17 14:32 -0400 URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/6b2ebf3c4964 6500133: REGRESSION: CertificateParsingException for CRL Distribution Point with blank Reviewed-by: mullan Contributed-by: jason.uh(a)oracle.com ! src/share/classes/sun/security/x509/URIName.java + test/sun/security/x509/URIName/Parse.java

1 0

hg: jdk8/tl/jaxp: 7191547: XMLEventFactory.newFactory(String factoryId, ClassLoader loader) does not work as expected
by huizhe.wang＠oracle.com 17 Aug '12

17 Aug '12

Changeset: 2946807a6d7f Author: joehw Date: 2012-08-17 09:49 -0700 URL: http://hg.openjdk.java.net/jdk8/tl/jaxp/rev/2946807a6d7f 7191547: XMLEventFactory.newFactory(String factoryId, ClassLoader loader) does not work as expected Summary: similar to the patch for 6756677 for XMLInputFactory and XMLOutputFactory, this patch fixes an error in XMLEventFactory where factoryId was taken as factory class. Reviewed-by: lancea ! src/javax/xml/stream/XMLEventFactory.java

1 0

RFR [7u8] (JAXP): 7191547 : XMLEventFactory.newFactory(String factoryId, ClassLoader loader) does not work as expected
by Joe Wang 17 Aug '12

17 Aug '12

In the patch for 6756677, we fixed errors in StAX's input and output factories where factoryId such as 'javax.xml.stream.XMLInputFactory" was taken as factory class e.g. com.sun.xml.internal.stream.XMLInputFactoryImpl. Unfortunately, we missed XMLEventFactory in that patch. So this is a patch for XMLEventFactory similar to that of XMLInputFactory and XMLOutputFactory. The change itself is to search an implementation class using the provided factory id instead of trying to create a instance directly. The webrev is here: http://cr.openjdk.java.net/~joehw/7u8/7191547/webrev/ All tests pass except two XMLEventFactory tests that assumed the factoryId to mean factory class. They will be updated once this patch is approved. Thanks, Joe

2 2

LinkedHashMap containsKeyBetween(K target, K low, K high) method?
by Paulo Levi 17 Aug '12

17 Aug '12

I tried to make a LinkedHashMap.subMap(K low, K high) with a map implementation i had implemented inspired by linked list where the map keeps linked Nodes as values and unwraps them on get() I failed of course, because i couldn't find a good way to deal with remove() on the original Map. The sublist implementation on the JDK depends on a modification counter to make sure no concurrent modifications from using the original list while the sublist is being used occur, which is fine for the linked list. However a submap needs to limit their get() on a different way from linkedlist since it can't just check for range - though about adding a index to the nodes but that's obviously wrong since any remove from the map would ruin efficient size() if based on those indexes (even removes from before submap construction). So i got less ambitious: public boolean containsKeyBetween(K target, K low, K high){ if(isEmpty()) return false; Node targetN = delegate.get(target); if(targetN == null) return false; Node highN, lowN; if(low == null) lowN = head.next; else lowN = delegate.get(low); if(high == null) highN = head.previous; else highN = delegate.get(high); return lowN != null && highN != null && lowN.index <= targetN.index && targetN.index <= highN.index; } Node needs a monotonically increasing int index created when put(). I don't actually think it needs the forward and backwards links (i was just reusing code), but LinkedHashMap/Set (where it would be useful), use them already. Is a method like this suitable for the JDK LinkedMap/Set?

1 1