<html>
<head>
<meta http-equiv="content-type" content="text/html; charset=ISO-8859-1">
</head>
<body bgcolor="#FFFFFF" text="#000000">
<br>
Hi everyone,<br>
<br>
Could I have a couple of reviews of this change?<br>
<br>
<a class="moz-txt-link-freetext" href="http://cr.openjdk.java.net/~brutisso/8022800/webrev.00/">http://cr.openjdk.java.net/~brutisso/8022800/webrev.00/</a><br>
<br>
The interesting change is in genCollectedHeap.hpp, where I change
the methods prev_gen(), next_gen() and get_gen() to never return
NULL. The rest of the changes are ripple effects of this change.<br>
<br>
From the CR:<br>
<br>
<meta http-equiv="content-type" content="text/html;
charset=ISO-8859-1">
The generations abstraction in the GC code was intended to be very
general. In practice we now always have exactly two generations -
the young and the old. This means that the GC code can be simplified
quite a bit.
<br>
<br>
A first step is to change from iteration over generations to
explicitly specify which generation is needed. This makes the code
easier to follow and it also helps tools like Parfait to understand
the code better.
<br>
<br>
This bug targets some simple clean-ups. A natural follow up of this
is probably to change the name of GenCollectedHeap to
TwoGenerationCollectedHeap (similar to the existing
TwoGenerationCollectorPolicy). But that will be handled separately.
<br>
<br>
Thanks,<br>
Bengt<br>
</body>
</html>