Code review request for 7058133 : Javah should use the freshly built classes instead of those from the BOOTDIR jdk

2012-01-16 Thread Seán Coffey
Valerie, I'm porting this fix to the JDK 7u repo. It's been fixed in JDK 6 JDK 8 repos already. full builds have been made with no issues seen. Bug report : http://bugs.sun.com/bugdatabase/view_bug.do?bug_id=7058133 webrev : http://cr.openjdk.java.net/~coffeys/webrev.7058133.jdk7u/ The fix

Re: 7130398: ProblemList.txt updates (1/2012)

2012-01-16 Thread Chris Hegarty
test/java/lang/annotation/loaderLeak/Main.java - java/lang/annotation/loaderLeak/Main.java ,otherwise fine. Thanks for adding these, I've been running into them also. -Chris. On 16/01/2012 16:12, Alan Bateman wrote: It's time to sync up test/ProblemList.txt again as it's currently not

hg: jdk8/tl/jdk: 7130398: ProblemList.txt updates (1/2012)

2012-01-16 Thread alan . bateman
Changeset: d1b0bda3a3c7 Author:alanb Date: 2012-01-16 16:30 + URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/d1b0bda3a3c7 7130398: ProblemList.txt updates (1/2012) Reviewed-by: chegar ! test/ProblemList.txt

Re: 7130398: ProblemList.txt updates (1/2012)

2012-01-16 Thread Alan Bateman
On 16/01/2012 16:32, Chris Hegarty wrote: test/java/lang/annotation/loaderLeak/Main.java - java/lang/annotation/loaderLeak/Main.java ,otherwise fine. Thanks for adding these, I've been running into them also. -Chris. I noticed the additional test/ after sending the mail. Thanks, pushing

hg: jdk8/tl/jdk: 7129083: CookieManager does not store cookies if url is read before setting cookie manager

2012-01-16 Thread chris . hegarty
Changeset: e8a143213c65 Author:chegar Date: 2012-01-16 18:05 + URL: http://hg.openjdk.java.net/jdk8/tl/jdk/rev/e8a143213c65 7129083: CookieManager does not store cookies if url is read before setting cookie manager Reviewed-by: michaelm !