<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.01 Transitional//EN">
<html>
  <head>
    <meta content="text/html; charset=ISO-8859-1"
      http-equiv="Content-Type">
  </head>
  <body bgcolor="#ffffff" text="#000000">
    Krystal,<br>
    <br>
    &nbsp; Thanks. Updated in same webrev.<br>
    &nbsp; I checked b10, the output looks OK<br>
    <br>
    /Yumin<br>
    <br>
    On 10/24/2012 12:24 PM, Krystal Mok wrote:
    <blockquote
cite="mid:CA+cQ+tTJA5crXrv=LygfNY7BOmzdY75hwMW1-YpMi1ae1j1kCw@mail.gmail.com"
      type="cite">Hi Yumin,
      <div><br>
      </div>
      <div>minor typos:</div>
      <div>s/<span style="white-space: pre-wrap;">instrurctions/</span><span
          style="white-space: pre-wrap;">instructions/</span></div>
      <div><span style="white-space: pre-wrap;"><br>
        </span></div>
      <div><span style="white-space: pre-wrap;">By the way, I tried to
          use a build of hsdis plugin after 6879063 in an earlier JDK
          build (e.g. JDK8b59) and it was working strangely. A rough
          look at the output looks like it's missing some newlines, I
          didn't go into the details. Could you please check if this fix
          makes the hsdis plugin work properly on earlier JDK builds as
          well?</span></div>
      <div><span style="white-space: pre-wrap;"><br>
        </span></div>
      <div><span style="white-space: pre-wrap;">Thanks,</span></div>
      <div><span style="white-space: pre-wrap;">Kris<br>
        </span><br>
        <div class="gmail_quote">On Thu, Oct 25, 2012 at 1:01 AM, <span
            dir="ltr">&lt;<a moz-do-not-send="true"
              href="mailto:yumin.qi@oracle.com" target="_blank">yumin.qi@oracle.com</a>&gt;</span>
          wrote:<br>
          <blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt
            0.8ex; border-left: 1px solid rgb(204, 204, 204);
            padding-left: 1ex;">&nbsp;Hi, all<br>
            <br>
            &nbsp; Can I have your codereview of 8000489: older builds of
            hsdis don't work anymore after 6879063<br>
            &nbsp; It caused old build broke the disassembler.<br>
            <br>
            &nbsp; Webrev: <a moz-do-not-send="true"
              href="http://cr.openjdk.java.net/%7Eminqi/8000489"
              target="_blank">http://cr.openjdk.java.net/~minqi/8000489</a><br>
            <br>
            Thanks<br>
            Yumin<br>
          </blockquote>
        </div>
        <br>
      </div>
    </blockquote>
  </body>
</html>