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

List:       openjdk-2d-dev
Subject:    Re: [OpenJDK 2D-Dev] RFR 3 windows related JNI correctness backports to jdk8u20
From:       Jennifer Godinez <jennifer.godinez () oracle ! com>
Date:       2014-04-03 17:01:25
Message-ID: 533D93E5.10003 () oracle ! com
[Download RAW message or body]

Looks good.

- Jennifer
On 04/03/2014 09:13 AM, Sergey Bylokhov wrote:
> Hi, Phil.
> looks good.
>
> On 4/2/14 1:34 AM, Phil Race wrote:
>> Sergey & Jennifer,
>>
>> Please review 3 mostly windows related JNI correctness backports to
>> jdk8u20 that you previously approved for JDK 9
>>
>> Bug link: https://bugs.openjdk.java.net/browse/JDK-8035623
>> Synopsis: [parfait] JNI exception pending in 
>> jdk/src/windows/native/sun/windows/awt_Font.cpp
>> 9 changeset: http://hg.openjdk.java.net/jdk9/client/jdk/rev/ebb3e01cace5
>> 8 backport : http://cr.openjdk.java.net/~prr/8035623.8u20/
>>
>> Bug link: https://bugs.openjdk.java.net/browse/JDK-8037910
>> Synopsis: JNI warnings in 
>> jdk/src/windows/native/sun/java2d/d3d/D3DSurfaceData.cpp
>> 9 changeset: http://hg.openjdk.java.net/jdk9/client/jdk/rev/218ba3f17a70
>> 8 backport : http://cr.openjdk.java.net/~prr/8037910.8u20/
>>
>> Bug link: https://bugs.openjdk.java.net/browse/JDK-8035630
>> Synopsis: [parfait] JNI exception pending in 
>> jdk/src/windows/native/sun/font/fontpath.c
>> 9 changesets: http://hg.openjdk.java.net/jdk9/client/jdk/rev/453c97485189
>> 9 changesets: http://hg.openjdk.java.net/jdk9/client/jdk/rev/ae1056458b47
>> 8 backport : http://cr.openjdk.java.net/~prr/8035630.8u20/
>>
>> Note that in the case of the latter of these, the 2nd changeset was a 
>> jcheck
>> related editing error that occurred during the commit. The 8 backport 
>> won't
>> have that error !
>>
>> -phil.
>
>
> -- 
> 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">
    Looks good.<br>
    <br>
    - Jennifer<br>
    <div class="moz-cite-prefix">On 04/03/2014 09:13 AM, Sergey Bylokhov
      wrote:<br>
    </div>
    <blockquote cite="mid:533D88AD.3040901@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 good.<br>
        <br>
        On 4/2/14 1:34 AM, Phil Race wrote:<br>
      </div>
      <blockquote cite="mid:533B30FF.3050504@oracle.com" type="cite">
        <meta http-equiv="content-type" content="text/html;
          charset=ISO-8859-1">
        Sergey &amp; Jennifer,<br>
        <br>
        Please review 3 mostly windows related JNI correctness backports
        to <br>
        jdk8u20 that you previously approved for JDK 9<br>
        <br>
        Bug link: <a moz-do-not-send="true"
          class="moz-txt-link-freetext"
          href="https://bugs.openjdk.java.net/browse/JDK-8034030">https://bugs.openjdk.java.net/browse/JDK-8035623</a><br>
  Synopsis:
        <meta http-equiv="content-type" content="text/html;
          charset=ISO-8859-1">
        [parfait] JNI exception pending in
        jdk/src/windows/native/sun/windows/awt_Font.cpp <br>
        9 changeset: <a moz-do-not-send="true"
          class="moz-txt-link-freetext"
          href="http://hg.openjdk.java.net/jdk9/client/jdk/rev/ebb3e01cace5">http://hg.openjdk.java.net/jdk9/client/jdk/rev/ebb3e01cace5</a><br>
  8 backport : <a moz-do-not-send="true"
          class="moz-txt-link-freetext"
          href="http://cr.openjdk.java.net/%7Eprr/8035623.8u20/">http://cr.openjdk.java.net/~prr/8035623.8u20/</a><br>
  <br>
        Bug link: <a moz-do-not-send="true"
          class="moz-txt-link-freetext"
          href="https://bugs.openjdk.java.net/browse/JDK-8034030">https://bugs.openjdk.java.net/browse/JDK-8037910</a><br>
  Synopsis: JNI warnings in
        jdk/src/windows/native/sun/java2d/d3d/D3DSurfaceData.cpp <span
          class="overlay-icon icon icon-edit-sml"></span><br>
        9 changeset: <a moz-do-not-send="true"
          class="moz-txt-link-freetext"
          href="http://hg.openjdk.java.net/jdk9/client/jdk/rev/218ba3f17a70">http://hg.openjdk.java.net/jdk9/client/jdk/rev/218ba3f17a70</a><br>
  8 backport : <a moz-do-not-send="true"
          class="moz-txt-link-freetext"
          href="http://cr.openjdk.java.net/%7Eprr/8037910.8u20/">http://cr.openjdk.java.net/~prr/8037910.8u20/</a><br>
  <br>
        Bug link: <a moz-do-not-send="true"
          class="moz-txt-link-freetext"
          href="https://bugs.openjdk.java.net/browse/JDK-8034030">https://bugs.openjdk.java.net/browse/JDK-8035630</a><br>
  <meta http-equiv="content-type" content="text/html;
          charset=ISO-8859-1">
        Synopsis: [parfait] JNI exception pending in
        jdk/src/windows/native/sun/font/fontpath.c <span
          class="overlay-icon icon icon-edit-sml"></span><br>
        9 changesets: <a moz-do-not-send="true"
          class="moz-txt-link-freetext"
          href="http://hg.openjdk.java.net/jdk9/client/jdk/rev/453c97485189">http://hg.openjdk.java.net/jdk9/client/jdk/rev/453c97485189</a><br>
  9 changesets: <a moz-do-not-send="true"
          class="moz-txt-link-freetext"
          href="http://hg.openjdk.java.net/jdk9/client/jdk/rev/ae1056458b47">http://hg.openjdk.java.net/jdk9/client/jdk/rev/ae1056458b47</a><br>
  8 backport : <a moz-do-not-send="true"
          class="moz-txt-link-freetext"
          href="http://cr.openjdk.java.net/%7Eprr/8035630.8u20/">http://cr.openjdk.java.net/~prr/8035630.8u20/</a><br>
  <br>
        Note that in the case of the latter of these, the 2nd changeset
        was a jcheck<br>
        related editing error that occurred during the commit. The 8
        backport won't<br>
        have that error !<br>
        <br>
        -phil.<br>
      </blockquote>
      <br>
      <br>
      <pre class="moz-signature" cols="72">-- 
Best regards, Sergey. </pre>
    </blockquote>
    <br>
  </body>
</html>



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

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