Hi, On Fri, 2013-05-24 at 09:09 +0100, Alan Bateman wrote: > On 24/05/2013 08:20, Thomas Schatzl wrote: > > Hi all, > > > > On Fri, 2013-05-24 at 14:30 +1000, David Holmes wrote: > >> Thomas, > >> > >> Are we still waiting for a second core-libs reviewer on this? > > Yes. Would some additional core-libs reviewer be so kind and look at > > this issue? > > > > http://cr.openjdk.java.net/~tschatzl/7038914/webrev.3/ > > > > Thomas > > > Only one reviewer is required but if you want a second then the proposed > change looks fine to me (and I'm happy the discussion got to the issue, > it wasn't initially clear). For the test then I guess I'd probably > rename it to ReferenceHandlerOOME but that's a minor comment. > > -Alan
thanks Alan, David and Peter and all others who contributed to this patch. I will move it further through the process then. Thanks a lot, Thomas
