<html><head><style>body{font-family:Helvetica,Arial;font-size:13px}</style></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;"><div id="bloop_customfont" style="font-family:Helvetica,Arial;font-size:13px; color: rgba(0,0,0,1.0); margin: 0px; line-height: auto;">Thomas,</div><div id="bloop_customfont" style="font-family:Helvetica,Arial;font-size:13px; color: rgba(0,0,0,1.0); margin: 0px; line-height: auto;"><br></div><div id="bloop_customfont" style="font-family:Helvetica,Arial;font-size:13px; color: rgba(0,0,0,1.0); margin: 0px; line-height: auto;">Thanks for looking at it. I have to patch a couple more places to change ParGCCardsPerStrideChunk to size_t (but I agree that this is the right type for it). Do you mind if I leave it as intx for now (I’ll change everything else to size_t) and do that as a separate change?</div><div id="bloop_customfont" style="font-family:Helvetica,Arial;font-size:13px; color: rgba(0,0,0,1.0); margin: 0px; line-height: auto;"><br></div><div id="bloop_customfont" style="font-family:Helvetica,Arial;font-size:13px; color: rgba(0,0,0,1.0); margin: 0px; line-height: auto;">Tony</div> <br><p class="airmail_on" style="color:#000;">On June 25, 2015 at 9:55:13 AM, Thomas Schatzl (<a href="mailto:thomas.schatzl@oracle.com">thomas.schatzl@oracle.com</a>) wrote:</p> <blockquote type="cite" class="clean_bq"><span><div><div></div><div>Hi Tony,
<br>
<br>  just a small comment:
<br>
<br>On Thu, 2015-06-25 at 09:29 -0400, Tony Printezis wrote:
<br>> PS I’ll post another version of the code shortly...
<br>>  
<br>> On June 25, 2015 at 9:29:17 AM, Tony Printezis
<br>> (tprintezis@twitter.com) wrote:
<br>>  
<br>[...]
<br>> > What do I do with the types of the args? Let’s go for consistency:
<br>> > if you guys want me to leave ParGCCardsPerStrideChunk as intx, I’ll
<br>> > leave everything as intx. My preference would be to change
<br>> > everything to uintx.
<br>
<br>You could use size_t too for some time now if it fits better.
<br>
<br>Thanks,
<br>  Thomas
<br>
<br>>  
<br>
<br>
<br></div></div></span></blockquote> <div id="bloop_sign_1435247043043130880" class="bloop_sign"><div style="font-family:helvetica,arial;font-size:13px"><div>-----</div><div><br></div><div>Tony Printezis | JVM/GC Engineer / VM Team | Twitter</div><div><br></div><div>@TonyPrintezis</div><div><a href="mailto:tprintezis@twitter.com">tprintezis@twitter.com</a></div><div><br></div></div></div></body></html>