<div dir="ltr"><div style>Hi Thomas,</div><div style><br></div><div style>Thanks for your comments. I'll be catching up with them.</div><div class="gmail_extra"><br><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
<div class=""><div class="h5"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><br>
    - in cmsOopClosures.hpp, MarkRefsIntoClosure and the new<br>
Par_MarkRefsIntoClosure could be refactored slightly as they have<br>
exactly the same member variables. Not sure how this situation is<br>
handled in other code though, and what others (Jon) think.<br>
</blockquote>
Thomas,<br>
<br>
If you don't mind I'd like to keep this changeset to a minimum so<br>
not do any additional refactoring.  That's a good suggestion but<br>
since this is the first sizable contribution I'm sponsoring, simpler<br>
is better for me.<br>
</blockquote>
Okay. It would be a tiny additional change though, which has only been<br>
enabled by the addition of the Par_MarkRefsIntoClosure, and of course<br>
depends on whether the old serial initial marking code is kept.<br>
</blockquote>
<br></div></div>
Thanks.<div><div class="h5"><br></div></div></blockquote><div><br></div><div>Regarding whether to refactor <span style="color:rgb(80,0,80)">MarkRefsIntoClosure and </span><span style="color:rgb(80,0,80)">Par_MarkRefsIntoClosure, </span>it's a valid point. I assume you are referring to factoring out the common parts into a common super class.</div>
<div><br></div><div style><span style="color:rgb(80,0,80)">I don't have a strong opinion. L</span>ooking at the exchanges, <span style="color:rgb(80,0,80)">I'm interpreting it as "let's not do it right now." Let me know if it's not the case.</span></div>
<div><br></div></div></div></div>