<html>
  <head>
    <meta content="text/html; charset=UTF-8" http-equiv="Content-Type">
  </head>
  <body bgcolor="#FFFFFF" text="#000000">
    <div class="moz-cite-prefix">Hi Sangheon,<br>
      <br>
      On 23/09/14 15:31, Sangheon Kim wrote:<br>
    </div>
    <blockquote cite="mid:54217622.8010100@oracle.com" type="cite">Hi
      all,
      <br>
      <br>
        can I have reviews for the following small change that changes
      the
      <br>
      uses of the "int" type for thread ids into uints like everywhere
      else?
      <br>
      <br>
      Thomas agreed to sponsor the change.
      <br>
      <br>
      CR:
      <br>
      <a class="moz-txt-link-freetext" href="https://bugs.openjdk.java.net/browse/JDK-8036116">https://bugs.openjdk.java.net/browse/JDK-8036116</a>
      <br>
      <br>
      Webrev:
      <br>
      <a class="moz-txt-link-freetext" href="http://cr.openjdk.java.net/~tschatzl/8036116-kim/webrev.00/">http://cr.openjdk.java.net/~tschatzl/8036116-kim/webrev.00/</a>
      <a class="moz-txt-link-rfc2396E" href="http://cr.openjdk.java.net/%7Etschatzl/8036116-kim/webrev.00/"><http://cr.openjdk.java.net/%7Etschatzl/8036116-kim/webrev.00/></a>
      <br>
    </blockquote>
    <br>
    The patch looks good. However, it is based against
    <meta http-equiv="content-type" content="text/html; charset=UTF-8">
    <a class="moz-txt-link-freetext" href="http://hg.openjdk.java.net/jdk9/dev/hotspot">http://hg.openjdk.java.net/jdk9/dev/hotspot</a> instead of
    <meta http-equiv="content-type" content="text/html; charset=UTF-8">
    <a class="moz-txt-link-freetext" href="http://hg.openjdk.java.net/jdk9/hs-gc/hotspot">http://hg.openjdk.java.net/jdk9/hs-gc/hotspot</a>, where most of our
    changes should be pushed to. Parts of the code you change have
    already been removed in hs-gc.<br>
    <br>
    thanks,<br>
    StefanK<br>
    <br>
    <br>
    <blockquote cite="mid:54217622.8010100@oracle.com" type="cite">
      <br>
      Testing: jprt
      <br>
      <br>
      Thanks,
      <br>
      Sangheon
      <br>
    </blockquote>
    <br>
  </body>
</html>