hg: jdk7/tl/jdk: 6510892: com/sun/net/httpserver/bugs/B6361557.java fails

2010-07-30 Thread michael . x . mcmahon
Changeset: 4d72d0ec83f5 Author:michaelm Date: 2010-07-30 18:16 +0100 URL: http://hg.openjdk.java.net/jdk7/tl/jdk/rev/4d72d0ec83f5 6510892: com/sun/net/httpserver/bugs/B6361557.java fails Reviewed-by: chegar ! test/com/sun/net/httpserver/bugs/B6361557.java

Re: [Fwd: Re: Review request: 6581734 CMS Old Gen's collection usage is zero after GC which is incorrect]

2010-07-30 Thread Mandy Chung
Thumbs up. Thanks Mandy Kevin Walls wrote: Of course... Uploaded in the same place. 8-) Mandy Chung wrote: Kevin Walls wrote: Great, thanks - respun as http://cr.openjdk.java.net/~kevinw/6581734/webrev.02/ This looks better. Test6581734.java: All methods/fields in this class should

Re: [Fwd: Re: Review request: 6581734 CMS Old Gen's collection usage is zero after GC which is incorrect]

2010-07-30 Thread Kevin Walls
Of course... Uploaded in the same place. 8-) Mandy Chung wrote: Kevin Walls wrote: Great, thanks - respun as http://cr.openjdk.java.net/~kevinw/6581734/webrev.02/ This looks better. Test6581734.java: All methods/fields in this class should have indented (currently not indented). Als

Re: [Fwd: Re: Review request: 6581734 CMS Old Gen's collection usage is zero after GC which is incorrect]

2010-07-30 Thread Mandy Chung
Kevin Walls wrote: Great, thanks - respun as http://cr.openjdk.java.net/~kevinw/6581734/webrev.02/ This looks better. Test6581734.java: All methods/fields in this class should have indented (currently not indented). Also line 118 has 3 extra spaces. Thanks Mandy Mandy Chung wrote:

Re: [Fwd: Re: Review request: 6581734 CMS Old Gen's collection usage is zero after GC which is incorrect]

2010-07-30 Thread Kevin Walls
Great, thanks - respun as http://cr.openjdk.java.net/~kevinw/6581734/webrev.02/ Mandy Chung wrote: Hi Kevin, Thanks for fixing this. Looks good with some minor comments: memoryManager.hpp line 181-182: get_last_gc_stat() returns gc_index but the comment says it returns the size of the

Re: [Fwd: Re: Review request: 6581734 CMS Old Gen's collection usage is zero after GC which is incorrect]

2010-07-30 Thread Mandy Chung
Hi Kevin, Thanks for fixing this. Looks good with some minor comments: memoryManager.hpp line 181-182: get_last_gc_stat() returns gc_index but the comment says it returns the size of the array pool. management.cpp line 1908: if (!mgr->get_last_gc_stat(stat) == 0) { You forgot to take ou

hg: jdk7/tl/langtools: 8 new changesets

2010-07-30 Thread lana . steuck
Changeset: 4cca8d7ce6c1 Author:lana Date: 2010-06-21 22:09 -0700 URL: http://hg.openjdk.java.net/jdk7/tl/langtools/rev/4cca8d7ce6c1 Merge Changeset: d1d7595fa824 Author:lana Date: 2010-06-29 22:43 -0700 URL: http://hg.openjdk.java.net/jdk7/tl/langtools/rev/d1d7595fa

hg: jdk7/tl/jaxws: 7 new changesets

2010-07-30 Thread lana . steuck
Changeset: db63f482182d Author:lana Date: 2010-06-21 22:07 -0700 URL: http://hg.openjdk.java.net/jdk7/tl/jaxws/rev/db63f482182d Merge Changeset: bd26d0ce0c3c Author:lana Date: 2010-06-29 22:31 -0700 URL: http://hg.openjdk.java.net/jdk7/tl/jaxws/rev/bd26d0ce0c3c Mer

hg: jdk7/tl/jdk: 58 new changesets

2010-07-30 Thread lana . steuck
Changeset: 4d55419ce99e Author:andrew Date: 2010-06-08 17:52 +0100 URL: http://hg.openjdk.java.net/jdk7/tl/jdk/rev/4d55419ce99e 6959123: Remove use of obsolete png_check_sig function in splashscreen_png.c Summary: Avoid use of deprecated libpng macro (removed in some 1.4.x releases)

hg: jdk7/tl/jaxp: 7 new changesets

2010-07-30 Thread lana . steuck
Changeset: 478835e100cd Author:lana Date: 2010-06-21 22:07 -0700 URL: http://hg.openjdk.java.net/jdk7/tl/jaxp/rev/478835e100cd Merge Changeset: d524be5ef62e Author:lana Date: 2010-06-29 22:31 -0700 URL: http://hg.openjdk.java.net/jdk7/tl/jaxp/rev/d524be5ef62e Merge