<html>
  <head>
    <meta content="text/html; charset=ISO-8859-1"
      http-equiv="Content-Type">
  </head>
  <body text="#000000" bgcolor="#FFFFFF">
    <div class="moz-cite-prefix">On 08.08.2013 16:54, Konstantin Perikov
      wrote:<br>
    </div>
    <blockquote
cite="mid:CAD=kh=+rQcDggG+RUCm8LYu5mj_xMU8ub0PfbSSPUuBxW++=cg@mail.gmail.com"
      type="cite">
      <div dir="ltr">Artem, could you answer on my last question?
        <div><br>
        </div>
        <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"><span
            style="font-family:arial,sans-serif;font-size:13px">P.S.
            Since, I'm newcomer, I don't have rights to push, so I need
            a sponsor. Who could help me?</span></blockquote>
      </div>
    </blockquote>
    I will help to push these changes after review<br>
    <blockquote
cite="mid:CAD=kh=+rQcDggG+RUCm8LYu5mj_xMU8ub0PfbSSPUuBxW++=cg@mail.gmail.com"
      type="cite">
      <div dir="ltr">
        <div><br>
        </div>
        <div>Thanks, Konstantin.&nbsp;</div>
      </div>
      <div class="gmail_extra"><br>
        <br>
        <div class="gmail_quote">2013/8/8 Artem Ananiev <span dir="ltr">&lt;<a
              moz-do-not-send="true"
              href="mailto:artem.ananiev@oracle.com" target="_blank">artem.ananiev@oracle.com</a>&gt;</span><br>
          <blockquote class="gmail_quote" style="margin:0 0 0
            .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi,
            Konstantin,<br>
            <br>
            I looked through the changes, which mostly replacements
            &lt;code&gt;&lt;/code&gt; with {@code}. This part of the fix
            looks fine.<br>
            <br>
            Changing /* to /** for the "isInShow" field doesn't make
            sense, as this field is transient and is not serialized
            anyway. However, it doesn't hurt as well.<br>
            <br>
            Please, keep comment about synchronization for the "type"
            field. It's a hint for developers, that this field should
            only be accessed or modified under the object lock. I agree,
            JavaDoc is not the best place for such hints, but I don't
            see any better solutions.<br>
            <br>
            Non-technical comments:<br>
            <br>
            1. Please, provide a direct link to webrev, so people can
            click and see the changes. Downloading archives is not as
            convenient (and in this particular case just impossible, as
            Yandex.Disk is in Russian, people just won't be able to read
            the "Download" button).<br>
            <br>
            2. Please, wait for at least one more person to have
            reviewed this fix.<br>
            <br>
            Thanks,<br>
            <br>
            Artem<br>
            <br>
            On 8/8/2013 2:20 PM, Konstantin Perikov wrote:<br>
            <blockquote class="gmail_quote" style="margin:0 0 0
              .8ex;border-left:1px #ccc solid;padding-left:1ex">
              Hi, AWT team,<br>
              <br>
              Could you please review the fix for the following bug:<br>
              <br>
              *7027045: &nbsp;: (doc) java/awt/Window.java has several typos
              in javadoc*<br>
              <br>
              Fix for version OpenJDK8. Also, I fix some
              &lt;code&gt;&lt;/code&gt; stuff and change it for {@code }<br>
              <br>
              The webrev is available here:<br>
              <br>
              <a moz-do-not-send="true"
                href="http://yadi.sk/d/KHB0hBlA7gkke" target="_blank">http://yadi.sk/d/KHB0hBlA7gkke</a><br>
              <br>
              <br>
              P.S. Since, I'm newcomer, I don't have rights to push, so
              I need a sponsor. Who could help me?<br>
              <br>
              Thanks,<br>
              <br>
              Konstantin<br>
              <br>
            </blockquote>
          </blockquote>
        </div>
        <br>
      </div>
    </blockquote>
    <br>
    <br>
    <pre class="moz-signature" cols="72">-- 
Best regards, Sergey. </pre>
  </body>
</html>