hg: jdk7/tl/jdk: 7015827: Fix HTML validation issues in java.math package

2011-02-01 Thread joe . darcy
Changeset: f110edeb4428 Author:darcy Date: 2011-02-01 00:29 -0800 URL: http://hg.openjdk.java.net/jdk7/tl/jdk/rev/f110edeb4428 7015827: Fix HTML validation issues in java.math package Reviewed-by: mduigou ! src/share/classes/java/math/BigDecimal.java ! src/share/classes/java/math/R

hg: jdk7/tl/jdk: 7011497: new CertPathValidatorException.BasicReason enum constant for constrained algorithm

2011-02-01 Thread xuelei . fan
Changeset: d4bc38aa7594 Author:xuelei Date: 2011-02-01 04:45 -0800 URL: http://hg.openjdk.java.net/jdk7/tl/jdk/rev/d4bc38aa7594 7011497: new CertPathValidatorException.BasicReason enum constant for constrained algorithm Summary: add new BasicReason and improve trust anchor searchin

Re: Request for review, 6766644: Redefinition of compiled method fails with assertion "Can not load classes with the Compiler thread"

2011-02-01 Thread Keith McGuigan
On Jan 31, 2011, at 9:20 PM, David Holmes wrote: Where we have: 961 void JvmtiDeferredEventQueue::enqueue(const JvmtiDeferredEvent& event) { ... 967 QueueNode* node = new QueueNode(event); and 1007 void JvmtiDeferredEventQueue::add_pending_event( 1008 const JvmtiDeferredEvent& eve

hg: jdk7/tl/jdk: 7015908: 3 javax.script tests fail with openjdk build

2011-02-01 Thread sundararajan . a
Changeset: 21d7cd823247 Author:sundar Date: 2011-02-01 21:00 +0530 URL: http://hg.openjdk.java.net/jdk7/tl/jdk/rev/21d7cd823247 7015908: 3 javax.script tests fail with openjdk build Reviewed-by: alanb ! test/javax/script/CauseExceptionTest.java ! test/javax/script/StringWriterPrint

hg: jdk7/tl/langtools: 6961571: Update visitors to support ARM's ElementKind.RESOURCE_VARIABLE

2011-02-01 Thread joe . darcy
Changeset: cad51b6eb7a6 Author:darcy Date: 2011-02-01 10:11 -0800 URL: http://hg.openjdk.java.net/jdk7/tl/langtools/rev/cad51b6eb7a6 6961571: Update visitors to support ARM's ElementKind.RESOURCE_VARIABLE Reviewed-by: jjg + src/share/classes/javax/lang/model/type/DisjunctiveType.ja

hg: jdk7/hotspot-rt/hotspot: 6588413: Use -fvisibility=hidden for gcc compiles

2011-02-01 Thread coleen . phillimore
Changeset: d70fe6ab4436 Author:coleenp Date: 2011-02-01 11:23 -0500 URL: http://hg.openjdk.java.net/jdk7/hotspot-rt/hotspot/rev/d70fe6ab4436 6588413: Use -fvisibility=hidden for gcc compiles Summary: Add option for gcc 4 and above, define JNIEXPORT and JNIIMPORT to visibility=defau

hg: jdk7/tl/jdk: 7015391: (zipfs) Update zip provider for 1/2011 changes; ...

2011-02-01 Thread xueming . shen
Changeset: 312dc0abb960 Author:sherman Date: 2011-02-01 14:20 -0800 URL: http://hg.openjdk.java.net/jdk7/tl/jdk/rev/312dc0abb960 7015391: (zipfs) Update zip provider for 1/2011 changes 7014948: (zipfs) ZipFileSystem.newFileSystem(Path...) should not throw FileSystemAlreadyExistsExc

Re: Request for review, 6766644: Redefinition of compiled method fails with assertion "Can not load classes with the Compiler thread"

2011-02-01 Thread David Holmes
Keith McGuigan said the following on 02/01/11 23:11: On Jan 31, 2011, at 9:20 PM, David Holmes wrote: As per your other email NULL check is pointless as "new" will never return null but will instead call vm_exit_out_of_memory to abort the VM - which is exactly my point. I don't believe that a f

Re: Request for review, 6766644: Redefinition of compiled method fails with assertion "Can not load classes with the Compiler thread"

2011-02-01 Thread Keith McGuigan
On Feb 1, 2011, at 6:38 PM, David Holmes wrote: The point is to avoid the need to dynamically allocate the QueueNodes. If the elements to be queued are self-linking then you don't need the extra wrapper objects. Take a look at VM_Operations as an example of self-linking objects. It also m

Re: Request for review, 6766644: Redefinition of compiled method fails with assertion "Can not load classes with the Compiler thread"

2011-02-01 Thread David Holmes
Keith McGuigan said the following on 02/02/11 10:26: On Feb 1, 2011, at 6:38 PM, David Holmes wrote: The point is to avoid the need to dynamically allocate the QueueNodes. If the elements to be queued are self-linking then you don't need the extra wrapper objects. Take a look at VM_Operations a

Re: Request for review, 6766644: Redefinition of compiled method fails with assertion "Can not load classes with the Compiler thread"

2011-02-01 Thread David Holmes
My apologies to Keith, I was completely missing the fact that the QueueNode class has to hold a copy of the event object due to the limited lifetime of the event objects themselves. Sorry. David David Holmes said the following on 02/02/11 10:30: Keith McGuigan said the following on 02/02/11