<div dir="ltr">Yes, sure you can use my current gmail<div><br></div><div>Thanks.</div></div><div class="gmail_extra"><br><br><div class="gmail_quote">2013/8/12 Artem Ananiev <span dir="ltr">&lt;<a 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>
the new version looks fine.<br>
<br>
As Sergey wrote in another email, he will help you with pushing this fix to the workspace. No further actions are required from your side. Thank you for help in making Java better :)<br>
<br>
We usually specify contributor&#39;s email in the &quot;Contributed-by&quot; field in mercurial comments. Can we use your current @<a href="http://gmail.com" target="_blank">gmail.com</a> address?<br>
<br>
Thanks,<br>
<br>
Artem<br>
<br>
On 8/12/2013 2:52 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>
Any updates on review of my fix?<br>
<br>
Konstantin.<br>
<br>
<br>
2013/8/9 Konstantin Perikov &lt;<a href="mailto:konstantin.perikov@gmail.com" target="_blank">konstantin.perikov@gmail.com</a><br>
&lt;mailto:<a href="mailto:konstantin.perikov@gmail.com" target="_blank">konstantin.perikov@<u></u>gmail.com</a>&gt;&gt;<div class="im"><br>
<br>
    Hi, AWT team<br>
<br>
    I update fix. Short summary:<br>
<br></div>
      * Javadoc fixed from /* to /** for two serializable fields<br>
      * Hint about synchronization still exists<br>
      * Also, I change  &lt;code&gt;&lt;/code&gt; to {@code}, but left some of them,<div class="im"><br>
        where &lt;em&gt;&lt;/em&gt; is using inside &lt;code&gt; tags<br></div>
      * Remove &quot;and and&quot; at line 2887<div class="im"><br>
<br>
<br>
    Now webrev placed in Dropbox here-<br>
    <a href="https://www.dropbox.com/sh/fz7qws34btuvq8f/KvIB0ZdL79" target="_blank">https://www.dropbox.com/sh/<u></u>fz7qws34btuvq8f/KvIB0ZdL79</a><br>
    You could donwload webrev in one click or open jdk.patch and looks<br>
    inside, using your favourite browser.<br>
<br>
    Please review it.<br>
<br>
    Thanks,<br>
<br>
    Konstantin.<br>
<br>
<br>
<br>
    2013/8/8 Konstantin Perikov &lt;<a href="mailto:konstantin.perikov@gmail.com" target="_blank">konstantin.perikov@gmail.com</a><br></div>
    &lt;mailto:<a href="mailto:konstantin.perikov@gmail.com" target="_blank">konstantin.perikov@<u></u>gmail.com</a>&gt;&gt;<div class="im"><br>
<br>
        So, what is the best place for webrev, if I don&#39;t have access to<br>
        ftp server?<br>
        I don&#39;t have possibility to set up Apache server for that.<br>
        Also, maybe I&#39;m wrong, but to view index.html on ftp, you need<br>
        to download it as well.<br>
<br>
        Okay, I will remove &quot;and and&quot; at line 2887.<br>
<br>
        About &lt;em&gt; tag, let me look at Javadoc documentation, I have no<br>
        idea, how it will behave in {@code} tag<br>
<br>
        Konstantin.<br>
<br>
<br>
<br>
<br>
        2013/8/8 Sergey Bylokhov &lt;<a href="mailto:Sergey.Bylokhov@oracle.com" target="_blank">Sergey.Bylokhov@oracle.com</a><br></div>
        &lt;mailto:<a href="mailto:Sergey.Bylokhov@oracle.com" target="_blank">Sergey.Bylokhov@<u></u>oracle.com</a>&gt;&gt;<div class="im"><br>
<br>
            Hi, Konstantin.<br>
            A few comments about the fix.<br>
            &lt;code&gt;&lt;em&gt;Foo&lt;/em&gt;Listener&lt;/<u></u>code&gt; in a few places was<br>
            changed to the {@code &lt;em&gt;Foo&lt;/em&gt;Listener} but it is not<br>
            equivalent.<br>
            Also I suggest to remove one of &quot;and&quot; from the &quot;and and&quot; in<br>
            the line 2887, because you change this line anyway.<br>
<br>
<br>
            On 08.08.2013 16:54, Konstantin Perikov wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
            Okay. I keep comment about synchronization for the &quot;type&quot;<br>
            field and return comment style for transient field.<br>
<br>
            Is Google Drive better place for it? (that&#39;s why I ask<br>
            yesterday about &quot;good&quot; place for webrev)<br>
            <a href="https://drive.google.com/folderview?id=0B4QwwAaNe6wZUWJvTE9HRG40dnM&amp;usp=sharing" target="_blank">https://drive.google.com/<u></u>folderview?id=<u></u>0B4QwwAaNe6wZUWJvTE9HRG40dnM&amp;<u></u>usp=sharing</a><br>

</blockquote>
            No, it is not better, atleast i don&#39;t understand how to view<br>
            index.html in this case or download all files at once.<br>
<br>
</div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
<br>
<br>
            2013/8/8 Artem Ananiev &lt;<a href="mailto:artem.ananiev@oracle.com" target="_blank">artem.ananiev@oracle.com</a><br>
            &lt;mailto:<a href="mailto:artem.ananiev@oracle.com" target="_blank">artem.ananiev@oracle.<u></u>com</a>&gt;&gt;<div><div class="h5"><br>
<br>
                Hi, Konstantin,<br>
<br>
                I looked through the changes, which mostly<br>
                replacements &lt;code&gt;&lt;/code&gt; with {@code}. This part of<br>
                the fix looks fine.<br>
<br>
                Changing /* to /** for the &quot;isInShow&quot; field doesn&#39;t<br>
                make sense, as this field is transient and is not<br>
                serialized anyway. However, it doesn&#39;t hurt as well.<br>
<br>
                Please, keep comment about synchronization for the<br>
                &quot;type&quot; field. It&#39;s a hint for developers, that this<br>
                field should only be accessed or modified under the<br>
                object lock. I agree, JavaDoc is not the best place<br>
                for such hints, but I don&#39;t see any better solutions.<br>
<br>
                Non-technical comments:<br>
<br>
                1. Please, provide a direct link to webrev, so people<br>
                can click and see the changes. Downloading archives is<br>
                not as convenient (and in this particular case just<br>
                impossible, as Yandex.Disk is in Russian, people just<br>
                won&#39;t be able to read the &quot;Download&quot; button).<br>
<br>
                2. Please, wait for at least one more person to have<br>
                reviewed this fix.<br>
<br>
                Thanks,<br>
<br>
                Artem<br>
<br>
                On 8/8/2013 2:20 PM, Konstantin Perikov wrote:<br>
<br>
                    Hi, AWT team,<br>
<br>
                    Could you please review the fix for the following bug:<br>
<br>
                    *7027045:  : (doc) java/awt/Window.java has<br>
                    several typos in javadoc*<br>
<br>
                    Fix for version OpenJDK8. Also, I fix some<br>
                    &lt;code&gt;&lt;/code&gt; stuff and change it for {@code }<br>
<br>
                    The webrev is available here:<br>
<br>
                    <a href="http://yadi.sk/d/KHB0hBlA7gkke" target="_blank">http://yadi.sk/d/KHB0hBlA7gkke</a><br>
<br>
<br>
                    P.S. Since, I&#39;m newcomer, I don&#39;t have rights to<br>
                    push, so I need a sponsor. Who could help me?<br>
<br>
                    Thanks,<br>
<br>
                    Konstantin<br>
<br>
<br>
</div></div></blockquote>
<br>
<br>
            --<br>
            Best regards, Sergey.<br>
<br>
<br>
<br>
<br>
</blockquote>
</blockquote></div><br></div>