[prev in list] [next in list] [prev in thread] [next in thread] 

List:       openjdk-2d-dev
Subject:    Re: [OpenJDK 2D-Dev] RFR: JDK8u20 backport of 8035569: JNI correctness fixes in WPrinterJob.cpp
From:       Sergey Bylokhov <Sergey.Bylokhov () oracle ! com>
Date:       2014-04-07 10:45:05
Message-ID: 534281B1.1040109 () oracle ! com
[Download RAW message or body]

Hi, Phil.
Looks fine.

On 07.04.2014 1:31, Phil Race wrote:
> On 4/5/14 8:14 AM, Sergey Bylokhov wrote:
>> Hi, Phil.
>> Looks like webrev/jbslink for JDK8u20 are for other fix.
>
> I think that's because I hand-edited the URL in thunderbird
> and it changed the visible text but not the actual link :-(
> So they are there .. you just needed to go where it looked like they'd be
> https://bugs.openjdk.java.net/browse/JDK-8035569 ..
> http://cr.openjdk.java.net/~prr/8035569.8u20/
>
> -phil.
>>
>> On 4/5/14 12:34 AM, Jennifer Godinez wrote:
>>> Looks good.
>>>
>>> Jennifer
>>> On 4/4/2014 12:01 PM, Phil Race wrote:
>>>> Jennifer & Sergey,
>>>>
>>>> Please check the identical JDK8u20 backport of the fix for the
>>>> JNI corrrectness fix in WPrinterJob.cpp that you previously 
>>>> approved for JDK 9
>>>> Bug link: https://bugs.openjdk.java.net/browse/JDK- 8035569
>>>> Synopsis: [parfait] JNI expection pending in 
>>>> jdk/src/windows/native/sun/windows/WPrinterJob.cpp
>>>> JDK9 changeset: 
>>>> http://hg.openjdk.java.net/jdk9/client/jdk/rev/06fd809d777f
>>>> JDK8u20 backport : http://cr.openjdk.java.net/~prr/8035569.8u20/
>>>>
>>>> -phil.
>>>>
>>>>
>>>>
>>>
>>
>>
>> -- 
>> Best regards, Sergey.
>


-- 
Best regards, Sergey.


[Attachment #3 (text/html)]

<html>
  <head>
    <meta content="text/html; charset=ISO-8859-1"
      http-equiv="Content-Type">
  </head>
  <body bgcolor="#FFFFFF" text="#000000">
    <div class="moz-cite-prefix">Hi, Phil.<br>
      Looks fine.<br>
      <br>
      On 07.04.2014 1:31, Phil Race wrote:<br>
    </div>
    <blockquote cite="mid:5341C7AB.2000700@oracle.com" type="cite">
      <meta content="text/html; charset=ISO-8859-1"
        http-equiv="Content-Type">
      <div class="moz-cite-prefix">On 4/5/14 8:14 AM, Sergey Bylokhov
        wrote:<br>
      </div>
      <blockquote cite="mid:53401DE7.9000208@oracle.com" type="cite">
        <meta content="text/html; charset=ISO-8859-1"
          http-equiv="Content-Type">
        <div class="moz-cite-prefix">Hi, Phil.<br>
          Looks like webrev/jbslink for JDK8u20 are for other fix.<br>
        </div>
      </blockquote>
      <br>
      I think that's because I hand-edited the URL in thunderbird<br>
      and it changed the visible text but not the actual link :-(<br>
      So they are there .. you just needed to go where it looked like
      they'd be<br>
      <a moz-do-not-send="true" class="moz-txt-link-freetext"
        href="https://bugs.openjdk.java.net/browse/JDK-8035569">https://bugs.openjdk.java.net/browse/JDK-8035569</a>
                
      ..<br>
      <a moz-do-not-send="true" class="moz-txt-link-freetext"
        href="http://cr.openjdk.java.net/%7Eprr/8035569.8u20/">http://cr.openjdk.java.net/~prr/8035569.8u20/</a><br>
  <br>
      -phil.<br>
      <blockquote cite="mid:53401DE7.9000208@oracle.com" type="cite">
        <div class="moz-cite-prefix"> <br>
          On 4/5/14 12:34 AM, Jennifer Godinez wrote:<br>
        </div>
        <blockquote cite="mid:533F1747.3020009@oracle.com" type="cite">
          <meta content="text/html; charset=ISO-8859-1"
            http-equiv="Content-Type">
          Looks good.<br>
          <br>
          Jennifer<br>
          <div class="moz-cite-prefix">On 4/4/2014 12:01 PM, Phil Race
            wrote:<br>
          </div>
          <blockquote cite="mid:533F0198.1060105@oracle.com" type="cite">
            <meta http-equiv="content-type" content="text/html;
              charset=ISO-8859-1">
            Jennifer &amp; Sergey,<br>
            <br>
            Please check the identical JDK8u20 backport of the fix for
            the<br>
            JNI corrrectness fix in WPrinterJob.cpp that you previously
            approved for JDK 9<br>
            Bug link: <a moz-do-not-send="true"
              class="moz-txt-link-freetext"
              href="https://bugs.openjdk.java.net/browse/JDK-8031095">https://bugs.openjdk.java.net/browse/JDK-





              <meta http-equiv="content-type" content="text/html;
                charset=ISO-8859-1">
              8035569</a><br>
            Synopsis:&nbsp;
            <meta http-equiv="content-type" content="text/html;
              charset=ISO-8859-1">
            [parfait] JNI expection pending in
            jdk/src/windows/native/sun/windows/WPrinterJob.cpp<br>
            JDK9 changeset: <a moz-do-not-send="true"
              class="moz-txt-link-freetext"
              href="http://hg.openjdk.java.net/jdk9/client/jdk/rev/06fd809d777f">http://hg.openjdk.java.net/jdk9/client/jdk/rev/06fd809d777f</a><br>
  JDK8u20 backport : <a moz-do-not-send="true"
              class="moz-txt-link-freetext"
              href="http://cr.openjdk.java.net/%7Eprr/8031095.8u20/">http://cr.openjdk.java.net/~prr/8035569.8u20/</a><br>
  <br>
            -phil.<br>
            <br>
            <br>
            <br>
          </blockquote>
          <br>
        </blockquote>
        <br>
        <br>
        <pre class="moz-signature" cols="72">-- 
Best regards, Sergey. </pre>
      </blockquote>
      <br>
    </blockquote>
    <br>
    <br>
    <pre class="moz-signature" cols="72">-- 
Best regards, Sergey. </pre>
  </body>
</html>



[prev in list] [next in list] [prev in thread] [next in thread] 

Configure | About | News | Add a list | Sponsored by KoreLogic