Hi Tony --<br><br>For some reason, I cannot seem to be able to view either of the CR's you mention below, and thus<br>lack the requisite background context for this change. The localized change however looks fine,<br>and it is nice to distinguish the two states of the array in this way by means of the sign of the length field.<br>
<br>One question is whether, for the sake of debugging & representational uniformity, it would make sense to use the same<br>encoding in the other collectors that do this chunking (even though I understand they may not have<br>
the need to deal with 6484965, whatever that might be :-)<br><br>Reviewed!<br>-- ramki<br><br><div class="gmail_quote">On Wed, Dec 14, 2011 at 2:17 PM, Tony Printezis <span dir="ltr"><<a href="mailto:tony.printezis@oracle.com">tony.printezis@oracle.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi all,<br>
<br>
Can I have a couple of code review for this small change?<br>
<br>
<a href="http://cr.openjdk.java.net/%7Etonyp/7121623/webrev.0/" target="_blank">http://cr.openjdk.java.net/~<u></u>tonyp/7121623/webrev.0/</a><br>
<br>
The CR has a bit more explanation. The short version is that I'm now encoding the "start index of the next chunk" in the from-space length field of a chunked array (say *that* quickly!) as a negative value to always be able to distinguish it from the real length. This will simplify the code for the CR John is currently working on (6484965) but it's a small, self-contained change so it's good to get it code reviewed and pushed separately.<span class="HOEnZb"><font color="#888888"><br>

<br>
Tony<br>
<br>
</font></span></blockquote></div><br>