RFR: 8189733: Cleanup Full GC setup and tear down

Thomas Schatzl thomas.schatzl at oracle.com
Wed Nov 22 15:41:18 UTC 2017


Hi,

On Wed, 2017-11-22 at 16:39 +0100, Stefan Johansson wrote:
> Thanks Thomas for the review,
> 
> I got some comments from Erik and have updated the review
> accordingly. 
> Here are the new webrevs:
> Full: http://cr.openjdk.java.net/~sjohanss/8189733/01/
> Inc: http://cr.openjdk.java.net/~sjohanss/8189733/00-01/
> 
> This change removes the heap() function as it's only used by member 
> functions which can use _heap. Also changed the initialization list
> to use the parameters rather than the members initialized.
> 

  still good.

Thomas



More information about the hotspot-gc-dev mailing list