<html>
  <head>
    <meta content="text/html; charset=windows-1252"
      http-equiv="Content-Type">
  </head>
  <body bgcolor="#FFFFFF" text="#000000">
    +1<br>
    <br>
    -phil.<br>
    <br>
    <div class="moz-cite-prefix">On 09/21/2016 02:29 AM, Rajeev Chamyal
      wrote:<br>
    </div>
    <blockquote cite="mid:3dcccb28-fedd-4e24-8f4d-4f66d4c15f80@default"
      type="cite">
      <meta http-equiv="Content-Type" content="text/html;
        charset=windows-1252">
      <meta name="Generator" content="Microsoft Word 15 (filtered
        medium)">
      <style><!--
/* Font Definitions */
@font-face
        {font-family:"Cambria Math";
        panose-1:2 4 5 3 5 4 6 3 2 4;}
@font-face
        {font-family:Calibri;
        panose-1:2 15 5 2 2 2 4 3 2 4;}
@font-face
        {font-family:Consolas;
        panose-1:2 11 6 9 2 2 4 3 2 4;}
/* Style Definitions */
p.MsoNormal, li.MsoNormal, div.MsoNormal
        {margin:0cm;
        margin-bottom:.0001pt;
        font-size:12.0pt;
        font-family:"Times New Roman",serif;
        color:black;}
a:link, span.MsoHyperlink
        {mso-style-priority:99;
        color:blue;
        text-decoration:underline;}
a:visited, span.MsoHyperlinkFollowed
        {mso-style-priority:99;
        color:purple;
        text-decoration:underline;}
pre
        {mso-style-priority:99;
        mso-style-link:"HTML Preformatted Char";
        margin:0cm;
        margin-bottom:.0001pt;
        font-size:10.0pt;
        font-family:"Courier New";
        color:black;}
span.HTMLPreformattedChar
        {mso-style-name:"HTML Preformatted Char";
        mso-style-priority:99;
        mso-style-link:"HTML Preformatted";
        font-family:Consolas;
        color:black;}
span.new
        {mso-style-name:new;}
span.EmailStyle20
        {mso-style-type:personal-reply;
        font-family:"Calibri",sans-serif;
        color:#1F497D;}
.MsoChpDefault
        {mso-style-type:export-only;
        font-size:10.0pt;}
@page WordSection1
        {size:612.0pt 792.0pt;
        margin:72.0pt 72.0pt 72.0pt 72.0pt;}
div.WordSection1
        {page:WordSection1;}
--></style><!--[if gte mso 9]><xml>
<o:shapedefaults v:ext="edit" spidmax="1026" />
</xml><![endif]--><!--[if gte mso 9]><xml>
<o:shapelayout v:ext="edit">
<o:idmap v:ext="edit" data="1" />
</o:shapelayout></xml><![endif]-->
      <div class="WordSection1">
        <p class="MsoNormal"><span
style="font-size:11.0pt;font-family:"Calibri",sans-serif;color:#1F497D;mso-fareast-language:EN-US">Hello
            Phil,<o:p></o:p></span></p>
        <p class="MsoNormal"><span
style="font-size:11.0pt;font-family:"Calibri",sans-serif;color:#1F497D;mso-fareast-language:EN-US"><o:p> </o:p></span></p>
        <p class="MsoNormal"><span
style="font-size:11.0pt;font-family:"Calibri",sans-serif;color:#1F497D;mso-fareast-language:EN-US">Please
            review the updated webrev.<o:p></o:p></span></p>
        <p class="MsoNormal"><span
style="font-size:11.0pt;font-family:"Calibri",sans-serif;color:#1F497D;mso-fareast-language:EN-US"><a
              moz-do-not-send="true"
              href="http://cr.openjdk.java.net/%7Erchamyal/8151787/webrev.10/">http://cr.openjdk.java.net/~rchamyal/8151787/webrev.10/</a>
            <o:p></o:p></span></p>
        <p class="MsoNormal"><span
style="font-size:11.0pt;font-family:"Calibri",sans-serif;color:#1F497D;mso-fareast-language:EN-US"><o:p> </o:p></span></p>
        <p class="MsoNormal"><span
style="font-size:11.0pt;font-family:"Calibri",sans-serif;color:#1F497D;mso-fareast-language:EN-US">Updates:
            Updated launcher.properties as suggested.<o:p></o:p></span></p>
        <p class="MsoNormal"><span
style="font-size:11.0pt;font-family:"Calibri",sans-serif;color:#1F497D;mso-fareast-language:EN-US"><o:p> </o:p></span></p>
        <p class="MsoNormal"><span
style="font-size:11.0pt;font-family:"Calibri",sans-serif;color:#1F497D;mso-fareast-language:EN-US">Regards,<o:p></o:p></span></p>
        <p class="MsoNormal"><span
style="font-size:11.0pt;font-family:"Calibri",sans-serif;color:#1F497D;mso-fareast-language:EN-US">Rajeev
            Chamyal<o:p></o:p></span></p>
        <p class="MsoNormal"><span
style="font-size:11.0pt;font-family:"Calibri",sans-serif;color:#1F497D;mso-fareast-language:EN-US"><o:p> </o:p></span></p>
        <div>
          <div style="border:none;border-top:solid #E1E1E1
            1.0pt;padding:3.0pt 0cm 0cm 0cm">
            <p class="MsoNormal"><b><span
style="font-size:11.0pt;font-family:"Calibri",sans-serif;color:windowtext"
                  lang="EN-US">From:</span></b><span
style="font-size:11.0pt;font-family:"Calibri",sans-serif;color:windowtext"
                lang="EN-US"> Philip Race <br>
                <b>Sent:</b> 21 September 2016 02:41<br>
                <b>To:</b> Rajeev Chamyal<br>
                <b>Cc:</b> Alexander Scherbatiy;
                <a class="moz-txt-link-abbreviated" href="mailto:awt-dev@openjdk.java.net">awt-dev@openjdk.java.net</a>; Sergey Bylokhov; Kumar
                Srinivasan<br>
                <b>Subject:</b> Re: <AWT Dev> <Swing Dev>[9]
                Review Request JDK-8151787 Unify the HiDPI splash screen
                image naming convention<o:p></o:p></span></p>
          </div>
        </div>
        <p class="MsoNormal"><o:p> </o:p></p>
        <pre><span class="new">\                  HiDPI scaled image is also supported\n\</span><o:p></o:p></pre>
        <pre><span class="new">  94 \                  Unscaled image name i.e. image.ext should be passed\n\</span><o:p></o:p></pre>
        <pre><span class="new">  95 \                  to -splash option for all image types irrespective of\n\</span><o:p></o:p></pre>
        <pre><span class="new">  96 \                  HiDPI and Non-HiDPI. Scaled filename convention should be\n\</span><o:p></o:p></pre>
        <pre><span class="new">  97 \                  used for HiDPI images\n\<o:p></o:p></span></pre>
        <pre><span class="new"><o:p> </o:p></span></pre>
        <pre><span class="new"><o:p> </o:p></span></pre>
        <pre><span class="new">try this :-<o:p></o:p></span></pre>
        <pre><span class="new">HiDPI scaled images are automatically supported and used if available.<o:p></o:p></span></pre>
        <pre><span class="new">The unscaled image filename, e.g. image.ext, should always be passed as<o:p></o:p></span></pre>
        <pre><span class="new">the argument to the -splash option. The most appropriate scaled image<o:p></o:p></span></pre>
        <pre><span class="new">provided will be picked up automatically.<o:p></o:p></span></pre>
        <pre><span class="new">See the SplashScreen API documentation for more information.<o:p></o:p></span></pre>
        <p class="MsoNormal">-phil.<br>
          <br>
          On 9/13/16, 12:03 AM, Rajeev Chamyal wrote: <o:p></o:p></p>
        <blockquote style="margin-top:5.0pt;margin-bottom:5.0pt">
          <pre>Hello Phil,<o:p></o:p></pre>
          <pre><o:p> </o:p></pre>
          <pre>Please review the updated webrev.<o:p></o:p></pre>
          <pre><a moz-do-not-send="true" href="http://cr.openjdk.java.net/%7Erchamyal/8151787/webrev.09/">http://cr.openjdk.java.net/~rchamyal/8151787/webrev.09/</a><o:p></o:p></pre>
          <pre><o:p> </o:p></pre>
          <pre>Updates:<o:p></o:p></pre>
          <pre>Updated documentation in src/java.desktop/share/classes/java/awt/SplashScreen.java <o:p></o:p></pre>
          <pre>Review comments from Kumar are also updated in launcher.properties.<o:p></o:p></pre>
          <pre><o:p> </o:p></pre>
          <pre>Regards,<o:p></o:p></pre>
          <pre>Rajeev Chamyal<o:p></o:p></pre>
          <pre><o:p> </o:p></pre>
          <pre>-----Original Message-----<o:p></o:p></pre>
          <pre>From: Kumar Srinivasan <o:p></o:p></pre>
          <pre>Sent: 12 September 2016 20:46<o:p></o:p></pre>
          <pre>To: Rajeev Chamyal<o:p></o:p></pre>
          <pre>Cc: Philip Race; Alexander Scherbatiy; <a moz-do-not-send="true" href="mailto:awt-dev@openjdk.java.net">awt-dev@openjdk.java.net</a>; Sergey Bylokhov<o:p></o:p></pre>
          <pre>Subject: Re: <AWT Dev> <Swing Dev>[9] Review Request JDK-8151787 Unify the HiDPI splash screen image naming convention<o:p></o:p></pre>
          <pre><o:p> </o:p></pre>
          <pre>-\                  HiDPI and Non-HiDPI.Scaled filename convention should be\n\<o:p></o:p></pre>
          <pre>+\                  HiDPI and Non-HiDPI. Scaled filename convention should be\n\<o:p></o:p></pre>
          <pre>+\                  used for HiDPI images\n\<o:p></o:p></pre>
          <pre><o:p> </o:p></pre>
          <pre>Space after .<o:p></o:p></pre>
          <pre><o:p> </o:p></pre>
          <pre>Approved, contingent on the above fix, I don't need to see another iteration.<o:p></o:p></pre>
          <pre><o:p> </o:p></pre>
          <pre>Thanks<o:p></o:p></pre>
          <pre>Kumar<o:p></o:p></pre>
          <pre><o:p> </o:p></pre>
          <pre><o:p> </o:p></pre>
          <pre>On 9/12/2016 5:25 AM, Rajeev Chamyal wrote:<o:p></o:p></pre>
          <blockquote style="margin-top:5.0pt;margin-bottom:5.0pt">
            <pre>Hello Kumar,<o:p></o:p></pre>
            <pre><o:p> </o:p></pre>
            <pre>Thanks for the review.<o:p></o:p></pre>
            <pre>Please review the updated webrev.<o:p></o:p></pre>
            <pre><o:p> </o:p></pre>
            <pre><a moz-do-not-send="true" href="http://cr.openjdk.java.net/%7Erchamyal/8151787/webrev.09/">http://cr.openjdk.java.net/~rchamyal/8151787/webrev.09/</a><o:p></o:p></pre>
            <pre>Updates:<o:p></o:p></pre>
            <pre>1) Updated launcher properties.<o:p></o:p></pre>
            <pre>2) Corrected indentation in splashscreen_impl.c<o:p></o:p></pre>
            <pre><o:p> </o:p></pre>
            <pre>Regards,<o:p></o:p></pre>
            <pre>Rajeev Chamyal<o:p></o:p></pre>
            <pre><o:p> </o:p></pre>
            <pre>-----Original Message-----<o:p></o:p></pre>
            <pre>From: Kumar Srinivasan<o:p></o:p></pre>
            <pre>Sent: 09 September 2016 21:32<o:p></o:p></pre>
            <pre>To: Rajeev Chamyal<o:p></o:p></pre>
            <pre>Cc: Philip Race; Alexander Scherbatiy; <a moz-do-not-send="true" href="mailto:awt-dev@openjdk.java.net">awt-dev@openjdk.java.net</a>; <o:p></o:p></pre>
            <pre>Sergey Bylokhov<o:p></o:p></pre>
            <pre>Subject: Re: <AWT Dev> <Swing Dev>[9] Review Request JDK-8151787 Unify <o:p></o:p></pre>
            <pre>the HiDPI splash screen image naming convention<o:p></o:p></pre>
            <pre><o:p> </o:p></pre>
            <pre><o:p> </o:p></pre>
            <pre>Hi Rajeev,<o:p></o:p></pre>
            <pre>In launcher.properties do you need an additional line to say, one must use the filename conventions to specify resolution ?<o:p></o:p></pre>
            <pre><o:p> </o:p></pre>
            <pre>btw: I think you have formatting/indent issues in this file:<o:p></o:p></pre>
            <pre>libsplashscreen/splashscreen_impl.c<o:p></o:p></pre>
            <pre><o:p> </o:p></pre>
            <pre><o:p> </o:p></pre>
            <pre>Thanks<o:p></o:p></pre>
            <pre>Kumar<o:p></o:p></pre>
            <pre><o:p> </o:p></pre>
            <pre><o:p> </o:p></pre>
            <blockquote style="margin-top:5.0pt;margin-bottom:5.0pt">
              <pre>Hello Kumar,Phil,<o:p></o:p></pre>
              <pre><o:p> </o:p></pre>
              <pre>Please review the update updated webrev.<o:p></o:p></pre>
              <pre><o:p> </o:p></pre>
              <pre><a moz-do-not-send="true" href="http://cr.openjdk.java.net/%7Erchamyal/8151787/webrev.08/">http://cr.openjdk.java.net/~rchamyal/8151787/webrev.08/</a><o:p></o:p></pre>
              <pre>Updates in files :<o:p></o:p></pre>
              <pre>src/java.base/share/classes/sun/launcher/resources/launcher.propertie<o:p></o:p></pre>
              <pre>s src/java.desktop/share/classes/java/awt/SplashScreen.java<o:p></o:p></pre>
              <pre><o:p> </o:p></pre>
              <pre>Regards,<o:p></o:p></pre>
              <pre>Rajeev Chamyal<o:p></o:p></pre>
              <pre><o:p> </o:p></pre>
              <pre>-----Original Message-----<o:p></o:p></pre>
              <pre>From: Kumar Srinivasan<o:p></o:p></pre>
              <pre>Sent: 02 September 2016 19:10<o:p></o:p></pre>
              <pre>To: Rajeev Chamyal<o:p></o:p></pre>
              <pre>Cc: Alexander Scherbatiy; <a moz-do-not-send="true" href="mailto:awt-dev@openjdk.java.net">awt-dev@openjdk.java.net</a>; Sergey Bylokhov; <o:p></o:p></pre>
              <pre>Philip Race<o:p></o:p></pre>
              <pre>Subject: Re: <AWT Dev> <Swing Dev>[9] Review Request JDK-8151787 <o:p></o:p></pre>
              <pre>Unify the HiDPI splash screen image naming convention<o:p></o:p></pre>
              <pre><o:p> </o:p></pre>
              <pre><o:p> </o:p></pre>
              <pre>Hi,<o:p></o:p></pre>
              <pre><o:p> </o:p></pre>
              <blockquote style="margin-top:5.0pt;margin-bottom:5.0pt">
                <pre>Hello Kumar,<o:p></o:p></pre>
                <pre><o:p> </o:p></pre>
                <pre>I have further updated launcher.properties.<o:p></o:p></pre>
                <pre><a moz-do-not-send="true" href="http://cr.openjdk.java.net/%7Erchamyal/8151787/webrev.07/">http://cr.openjdk.java.net/~rchamyal/8151787/webrev.07/</a><o:p></o:p></pre>
                <pre><o:p> </o:p></pre>
                <pre>For documentation update I have already raise a bug.<o:p></o:p></pre>
                <pre><a moz-do-not-send="true" href="https://bugs.openjdk.java.net/browse/JDK-8165009">https://bugs.openjdk.java.net/browse/JDK-8165009</a><o:p></o:p></pre>
              </blockquote>
              <pre>Ok.<o:p></o:p></pre>
              <pre><o:p> </o:p></pre>
              <pre>so what about ?<o:p></o:p></pre>
              <pre><o:p> </o:p></pre>
              <pre><a moz-do-not-send="true" href="http://hg.openjdk.java.net/jdk9/dev/jdk/file/1c28399f1b50/src/java.de">http://hg.openjdk.java.net/jdk9/dev/jdk/file/1c28399f1b50/src/java.de</a><o:p></o:p></pre>
              <pre>s ktop/share/classes/java/awt/SplashScreen.java<o:p></o:p></pre>
              <pre><o:p> </o:p></pre>
              <pre><o:p> </o:p></pre>
              <pre>This is the SplashScreen specification, is it not ? But there seems to be no effort to clarify this specification ?<o:p></o:p></pre>
              <pre><o:p> </o:p></pre>
              <pre>Kumar<o:p></o:p></pre>
              <pre><o:p> </o:p></pre>
              <blockquote style="margin-top:5.0pt;margin-bottom:5.0pt">
                <pre>Regards,<o:p></o:p></pre>
                <pre>Rajeev Chamyal<o:p></o:p></pre>
                <pre><o:p> </o:p></pre>
                <pre>-----Original Message-----<o:p></o:p></pre>
                <pre>From: Kumar Srinivasan<o:p></o:p></pre>
                <pre>Sent: 01 September 2016 19:17<o:p></o:p></pre>
                <pre>To: Rajeev Chamyal<o:p></o:p></pre>
                <pre>Cc: Alexander Scherbatiy; <a moz-do-not-send="true" href="mailto:awt-dev@openjdk.java.net">awt-dev@openjdk.java.net</a>; Sergey Bylokhov; <o:p></o:p></pre>
                <pre>Philip Race<o:p></o:p></pre>
                <pre>Subject: Re: <AWT Dev> <Swing Dev>[9] Review Request JDK-8151787 <o:p></o:p></pre>
                <pre>Unify the HiDPI splash screen image naming convention<o:p></o:p></pre>
                <pre><o:p> </o:p></pre>
                <pre><o:p> </o:p></pre>
                <pre>Hello Rajeev,<o:p></o:p></pre>
                <pre><o:p> </o:p></pre>
                <pre>IMHO, this really belongs here:<o:p></o:p></pre>
                <pre><o:p> </o:p></pre>
                <pre><a moz-do-not-send="true" href="http://hg.openjdk.java.net/jdk9/dev/jdk/file/1c28399f1b50/src/java.d">http://hg.openjdk.java.net/jdk9/dev/jdk/file/1c28399f1b50/src/java.d</a><o:p></o:p></pre>
                <pre>e s ktop/share/classes/java/awt/SplashScreen.java<o:p></o:p></pre>
                <pre><o:p> </o:p></pre>
                <pre>and here:<o:p></o:p></pre>
                <pre><o:p> </o:p></pre>
                <pre><a moz-do-not-send="true" href="https://docs.oracle.com/javase/8/docs/technotes/tools/unix/java.html">https://docs.oracle.com/javase/8/docs/technotes/tools/unix/java.html</a><o:p></o:p></pre>
                <pre><o:p> </o:p></pre>
                <pre>If you can reduce the words in the launcher, it would be good, also please make sure the lines do not exceed 80 chars ie. the output of java -help.<o:p></o:p></pre>
                <pre><o:p> </o:p></pre>
                <pre><o:p> </o:p></pre>
                <pre>Kumar<o:p></o:p></pre>
                <pre><o:p> </o:p></pre>
                <pre><o:p> </o:p></pre>
                <pre><o:p> </o:p></pre>
                <blockquote style="margin-top:5.0pt;margin-bottom:5.0pt">
                  <pre>Hello Kumar,<o:p></o:p></pre>
                  <pre><o:p> </o:p></pre>
                  <pre>Can you please review the updated<o:p></o:p></pre>
                  <pre>src/java.base/share/classes/sun/launcher/resources/launcher.propert<o:p></o:p></pre>
                  <pre>i e s <a moz-do-not-send="true" href="http://cr.openjdk.java.net/%7Erchamyal/8151787/webrev.06/">http://cr.openjdk.java.net/~rchamyal/8151787/webrev.06/</a><o:p></o:p></pre>
                  <pre><o:p> </o:p></pre>
                  <pre>Regards,<o:p></o:p></pre>
                  <pre>Rajeev Chamyal<o:p></o:p></pre>
                  <pre><o:p> </o:p></pre>
                  <pre>-----Original Message-----<o:p></o:p></pre>
                  <pre>From: Philip Race<o:p></o:p></pre>
                  <pre>Sent: 27 August 2016 03:10<o:p></o:p></pre>
                  <pre>To: Rajeev Chamyal<o:p></o:p></pre>
                  <pre>Cc: Alexander Scherbatiy; <a moz-do-not-send="true" href="mailto:awt-dev@openjdk.java.net">awt-dev@openjdk.java.net</a>; Sergey Bylokhov<o:p></o:p></pre>
                  <pre>Subject: Re: <AWT Dev> <Swing Dev>[9] Review Request JDK-8151787 <o:p></o:p></pre>
                  <pre>Unify the HiDPI splash screen image naming convention<o:p></o:p></pre>
                  <pre><o:p> </o:p></pre>
                  <pre>Seems fine now.<o:p></o:p></pre>
                  <pre>1) Please do a CCC for this<o:p></o:p></pre>
                  <pre>2) Please file a doc. bug so docs team can update the HTML man page <o:p></o:p></pre>
                  <pre>and also perhaps <o:p></o:p></pre>
                  <pre><a moz-do-not-send="true" href="https://docs.oracle.com/javase/tutorial/uiswing/misc/splashscreen.h">https://docs.oracle.com/javase/tutorial/uiswing/misc/splashscreen.h</a><o:p></o:p></pre>
                  <pre>t<o:p></o:p></pre>
                  <pre>m<o:p></o:p></pre>
                  <pre>l<o:p></o:p></pre>
                  <pre><o:p> </o:p></pre>
                  <pre>-phil<o:p></o:p></pre>
                  <pre><o:p> </o:p></pre>
                  <pre>On 8/25/16, 11:49 PM, Rajeev Chamyal wrote:<o:p></o:p></pre>
                  <blockquote
                    style="margin-top:5.0pt;margin-bottom:5.0pt">
                    <pre>Hello Alexandr,<o:p></o:p></pre>
                    <pre><o:p> </o:p></pre>
                    <pre>Please review the updated webrev.<o:p></o:p></pre>
                    <pre><a moz-do-not-send="true" href="http://cr.openjdk.java.net/%7Erchamyal/8151787/webrev.05/">http://cr.openjdk.java.net/~rchamyal/8151787/webrev.05/</a><o:p></o:p></pre>
                    <pre><o:p> </o:p></pre>
                    <pre>Regards,<o:p></o:p></pre>
                    <pre>Rajeev Chamyal<o:p></o:p></pre>
                    <pre><o:p> </o:p></pre>
                    <pre>-----Original Message-----<o:p></o:p></pre>
                    <pre>From: Alexandr Scherbatiy<o:p></o:p></pre>
                    <pre>Sent: 25 August 2016 22:07<o:p></o:p></pre>
                    <pre>To: Rajeev Chamyal; Philip Race<o:p></o:p></pre>
                    <pre>Cc: <a moz-do-not-send="true" href="mailto:awt-dev@openjdk.java.net">awt-dev@openjdk.java.net</a>; Sergey Bylokhov<o:p></o:p></pre>
                    <pre>Subject: Re:<AWT Dev>  <Swing Dev>[9] Review Request JDK-8151787 <o:p></o:p></pre>
                    <pre>Unify the HiDPI splash screen image naming convention<o:p></o:p></pre>
                    <pre><o:p> </o:p></pre>
                    <pre>On 8/22/2016 2:13 PM, Rajeev Chamyal wrote:<o:p></o:p></pre>
                    <blockquote
                      style="margin-top:5.0pt;margin-bottom:5.0pt">
                      <pre>Hello Phil,<o:p></o:p></pre>
                      <pre><o:p> </o:p></pre>
                      <pre>Thanks for the review,<o:p></o:p></pre>
                      <pre>Please review updated webrev.<o:p></o:p></pre>
                      <pre><a moz-do-not-send="true" href="http://cr.openjdk.java.net/%7Erchamyal/8151787/webrev.04/">http://cr.openjdk.java.net/~rchamyal/8151787/webrev.04/</a><o:p></o:p></pre>
                      <pre>Updated files:<o:p></o:p></pre>
                      <pre>src/java.base/share/classes/sun/launcher/resources/launcher.prope<o:p></o:p></pre>
                      <pre>r<o:p></o:p></pre>
                      <pre>t<o:p></o:p></pre>
                      <pre>i<o:p></o:p></pre>
                      <pre>e s<o:p></o:p></pre>
                      <pre>src/java.desktop/macosx/native/libsplashscreen/splashscreen_sys.m<o:p></o:p></pre>
                      <pre>src/java.desktop/macosx/native/libsplashscreen/splashscreen_config.<o:p></o:p></pre>
                      <pre>h<o:p></o:p></pre>
                    </blockquote>
                    <pre>         The findScaledImageName(...) method is only used in splashscreen_sys.m file. Is it possible to not declare it in splashscreen_config.h?<o:p></o:p></pre>
                    <pre><o:p> </o:p></pre>
                    <pre>        Thanks,<o:p></o:p></pre>
                    <pre>        Alexandr.<o:p></o:p></pre>
                    <pre><o:p> </o:p></pre>
                    <blockquote
                      style="margin-top:5.0pt;margin-bottom:5.0pt">
                      <pre>Regards,<o:p></o:p></pre>
                      <pre>Rajeev Chamyal<o:p></o:p></pre>
                      <pre><o:p> </o:p></pre>
                      <pre>-----Original Message-----<o:p></o:p></pre>
                      <pre>From: Phil Race<o:p></o:p></pre>
                      <pre>Sent: 20 August 2016 01:47<o:p></o:p></pre>
                      <pre>To: Rajeev Chamyal<o:p></o:p></pre>
                      <pre>Cc: <a moz-do-not-send="true" href="mailto:awt-dev@openjdk.java.net">awt-dev@openjdk.java.net</a>; Sergey Bylokhov; Alexander <o:p></o:p></pre>
                      <pre>Scherbatiy<o:p></o:p></pre>
                      <pre>Subject: Re:<AWT Dev>  <Swing Dev>[9] Review Request JDK-8151787 <o:p></o:p></pre>
                      <pre>Unify the HiDPI splash screen image naming convention<o:p></o:p></pre>
                      <pre><o:p> </o:p></pre>
                      <pre>I recall that in order to be consistent we concluded that @200pct and @300pct needed to be supported in addition to the @2x and @3x syntax.<o:p></o:p></pre>
                      <pre><o:p> </o:p></pre>
                      <pre>-phil.<o:p></o:p></pre>
                      <pre><o:p> </o:p></pre>
                      <pre>On 8/19/2016 3:41 AM, Rajeev Chamyal wrote:<o:p></o:p></pre>
                      <blockquote
                        style="margin-top:5.0pt;margin-bottom:5.0pt">
                        <pre>Hello Phil,<o:p></o:p></pre>
                        <pre><o:p> </o:p></pre>
                        <pre>Please review the updated webrev.<o:p></o:p></pre>
                        <pre><o:p> </o:p></pre>
                        <pre><a moz-do-not-send="true" href="http://cr.openjdk.java.net/%7Erchamyal/8151787/webrev.03/">http://cr.openjdk.java.net/~rchamyal/8151787/webrev.03/</a><o:p></o:p></pre>
                        <pre><a moz-do-not-send="true" href="http://cr.openjdk.java.net/%7Erchamyal/8151787/webrev.03/"><http://cr.openjdk.java.net/%7Erchamyal/8151787/webrev.03/></a><o:p></o:p></pre>
                        <pre><o:p> </o:p></pre>
                        <pre>Updated file<o:p></o:p></pre>
                        <pre>src/java.base/share/classes/sun/launcher/resources/launcher.prop<o:p></o:p></pre>
                        <pre>e<o:p></o:p></pre>
                        <pre>r<o:p></o:p></pre>
                        <pre>t<o:p></o:p></pre>
                        <pre>i<o:p></o:p></pre>
                        <pre>e<o:p></o:p></pre>
                        <pre>s<o:p></o:p></pre>
                        <pre><o:p> </o:p></pre>
                        <pre>Added all other supported name extensions.<o:p></o:p></pre>
                        <pre><o:p> </o:p></pre>
                        <pre>Regards,<o:p></o:p></pre>
                        <pre><o:p> </o:p></pre>
                        <pre>Rajeev Chamyal<o:p></o:p></pre>
                        <pre><o:p> </o:p></pre>
                        <pre>*From:*Philip Race<o:p></o:p></pre>
                        <pre>*Sent:* 19 August 2016 04:48<o:p></o:p></pre>
                        <pre>*To:* Rajeev Chamyal<o:p></o:p></pre>
                        <pre>*Cc:* <a moz-do-not-send="true" href="mailto:awt-dev@openjdk.java.net">awt-dev@openjdk.java.net</a>; Sergey Bylokhov; Alexander <o:p></o:p></pre>
                        <pre>Scherbatiy<o:p></o:p></pre>
                        <pre>*Subject:* Re:<AWT Dev>  <Swing Dev>[9] Review Request<o:p></o:p></pre>
                        <pre>JDK-8151787 Unify the HiDPI splash screen image naming <o:p></o:p></pre>
                        <pre>convention<o:p></o:p></pre>
                        <pre><o:p> </o:p></pre>
                        <pre>Better, although it still does not document the supported set of <o:p></o:p></pre>
                        <pre>scale name extensions that we discussed/proposed off-line.<o:p></o:p></pre>
                        <pre><o:p> </o:p></pre>
                        <pre>-phil.<o:p></o:p></pre>
                        <pre><o:p> </o:p></pre>
                        <pre>On 8/18/16, 5:39 AM, Rajeev Chamyal wrote:<o:p></o:p></pre>
                        <pre><o:p> </o:p></pre>
                        <pre>          Hello Phil,<o:p></o:p></pre>
                        <pre><o:p> </o:p></pre>
                        <pre>          Thanks for the review.<o:p></o:p></pre>
                        <pre><o:p> </o:p></pre>
                        <pre>          Please review the updated webrev.<o:p></o:p></pre>
                        <pre><o:p> </o:p></pre>
                        <pre>          <o:p></o:p></pre>
                        <pre><a moz-do-not-send="true" href="http://cr.openjdk.java.net/%7Erchamyal/8151787/webrev.02/">http://cr.openjdk.java.net/~rchamyal/8151787/webrev.02/</a><o:p></o:p></pre>
                        <pre>          <o:p></o:p></pre>
                        <pre><a moz-do-not-send="true" href="http://cr.openjdk.java.net/%7Erchamyal/8151787/webrev.02/"><http://cr.openjdk.java.net/%7Erchamyal/8151787/webrev.02/></a><o:p></o:p></pre>
                        <pre><o:p> </o:p></pre>
                        <pre>          Updated file<o:p></o:p></pre>
                        <pre><o:p> </o:p></pre>
                        <pre>src/java.base/share/classes/sun/launcher/resources/launcher.prop<o:p></o:p></pre>
                        <pre>e<o:p></o:p></pre>
                        <pre>r<o:p></o:p></pre>
                        <pre>t<o:p></o:p></pre>
                        <pre>i<o:p></o:p></pre>
                        <pre>e<o:p></o:p></pre>
                        <pre>s<o:p></o:p></pre>
                        <pre><o:p> </o:p></pre>
                        <pre>          Regards,<o:p></o:p></pre>
                        <pre><o:p> </o:p></pre>
                        <pre>          Rajeev Chamyal<o:p></o:p></pre>
                        <pre><o:p> </o:p></pre>
                        <pre>          *From:*Phil Race<o:p></o:p></pre>
                        <pre>          *Sent:* 16 August 2016 22:28<o:p></o:p></pre>
                        <pre>          *To:* Alexandr Scherbatiy<o:p></o:p></pre>
                        <pre>          *Cc:* Rajeev Chamyal; <a moz-do-not-send="true" href="mailto:awt-dev@openjdk.java.net">awt-dev@openjdk.java.net</a><o:p></o:p></pre>
                        <pre>          <a moz-do-not-send="true" href="mailto:awt-dev@openjdk.java.net"><mailto:awt-dev@openjdk.java.net></a>; Sergey Bylokhov<o:p></o:p></pre>
                        <pre>          *Subject:* Re:<AWT Dev>  <Swing Dev>[9] Review Request JDK-8151787<o:p></o:p></pre>
                        <pre>          Unify the HiDPI splash screen image naming convention<o:p></o:p></pre>
                        <pre><o:p> </o:p></pre>
                        <pre>          On 08/16/2016 09:41 AM, Alexandr Scherbatiy wrote:<o:p></o:p></pre>
                        <pre><o:p> </o:p></pre>
                        <pre><o:p> </o:p></pre>
                        <pre>              The fix looks good to me.<o:p></o:p></pre>
                        <pre><o:p> </o:p></pre>
                        <pre>              It would be better if a native speaker look at the<o:p></o:p></pre>
                        <pre>              documentation change in the launcher.properties file.<o:p></o:p></pre>
                        <pre><o:p> </o:p></pre>
                        <pre><o:p> </o:p></pre>
                        <pre>          That documentation seems to cover only *some* of the extensions we<o:p></o:p></pre>
                        <pre>          discussed.<o:p></o:p></pre>
                        <pre>          It ought to cite all of them if it does so at all. How else are<o:p></o:p></pre>
                        <pre>          people supposed<o:p></o:p></pre>
                        <pre>          to know what they can use ? Where else are you documenting it?<o:p></o:p></pre>
                        <pre>          Perhaps the launcher "man" page should be updated too<o:p></o:p></pre>
                        <pre>          find . -name java.1<o:p></o:p></pre>
                        <pre>          ./src/linux/doc/man/java.1<o:p></o:p></pre>
                        <pre>          ./src/linux/doc/man/ja/java.1<o:p></o:p></pre>
                        <pre>          ./src/bsd/doc/man/java.1<o:p></o:p></pre>
                        <pre>          ./src/bsd/doc/man/ja/java.1<o:p></o:p></pre>
                        <pre>          ./src/solaris/doc/sun/man/man1/java.1<o:p></o:p></pre>
                        <pre>          ./src/solaris/doc/sun/man/man1/ja/java.1<o:p></o:p></pre>
                        <pre><o:p> </o:p></pre>
                        <pre>          .. although I think there is also some HTML version maintained by<o:p></o:p></pre>
                        <pre>          the pubs/docs team<o:p></o:p></pre>
                        <pre>          that is not in OpenJDK - the above does not include Windows or Mac.<o:p></o:p></pre>
                        <pre>          I don't know offhand what is recommended these days. We'll have to<o:p></o:p></pre>
                        <pre>          find someone<o:p></o:p></pre>
                        <pre>          who does more with the launcher to help point to where to do the<o:p></o:p></pre>
                        <pre>          documentation.<o:p></o:p></pre>
                        <pre><o:p> </o:p></pre>
                        <pre>          And the doc does not really explain what is going on here. Reading<o:p></o:p></pre>
                        <pre>          that I might<o:p></o:p></pre>
                        <pre>          think I am supposed to pass -splash:image@2x.ext if I want a<o:p></o:p></pre>
                        <pre>          hi-dpi image<o:p></o:p></pre>
                        <pre>          and that is not the idea at all, is it ?<o:p></o:p></pre>
                        <pre>          The idea is you would still specify -splash:image.ext and the<o:p></o:p></pre>
                        <pre>          *implementation*<o:p></o:p></pre>
                        <pre>          will look for the most appropriate scaled image for the current<o:p></o:p></pre>
                        <pre>          desktop.<o:p></o:p></pre>
                        <pre><o:p> </o:p></pre>
                        <pre>          I think we should also have a CCC cover this (somehow).<o:p></o:p></pre>
                        <pre><o:p> </o:p></pre>
                        <pre>          -phil.<o:p></o:p></pre>
                        <pre><o:p> </o:p></pre>
                        <pre><o:p> </o:p></pre>
                        <pre><o:p> </o:p></pre>
                        <pre><o:p> </o:p></pre>
                        <pre>              Thanks,<o:p></o:p></pre>
                        <pre>              Alexandr.<o:p></o:p></pre>
                        <pre><o:p> </o:p></pre>
                        <pre>              On 8/16/2016 8:26 AM, Rajeev Chamyal wrote:<o:p></o:p></pre>
                        <pre><o:p> </o:p></pre>
                        <pre>                  Hello Alexandr,<o:p></o:p></pre>
                        <pre><o:p> </o:p></pre>
                        <pre>                  Please review the updated webrev.<o:p></o:p></pre>
                        <pre><o:p> </o:p></pre>
                        <pre>                  <o:p></o:p></pre>
                        <pre><a moz-do-not-send="true" href="http://cr.openjdk.java.net/%7Erchamyal/8151787/webrev.01/">http://cr.openjdk.java.net/~rchamyal/8151787/webrev.01/</a><o:p></o:p></pre>
                        <pre><o:p> </o:p></pre>
                        <pre><a moz-do-not-send="true" href="http://cr.openjdk.java.net/%7Erchamyal/8151787/webrev.01/"><http://cr.openjdk.java.net/%7Erchamyal/8151787/webrev.01/></a><o:p></o:p></pre>
                        <pre><o:p> </o:p></pre>
                        <pre>                  Updates :<o:p></o:p></pre>
                        <pre><o:p> </o:p></pre>
                        <pre>                  1)Updated the consition as suggested  if(*scaleFactor -<o:p></o:p></pre>
                        <pre>                  (int)*scaleFactor<  0.000001)<o:p></o:p></pre>
                        <pre><o:p> </o:p></pre>
                        <pre>                  2)Includes the changes of<o:p></o:p></pre>
                        <pre><o:p> </o:p></pre>
                        <pre>src/java.desktop/unix/native/libsplashscreen/splashscreen_sys.c<o:p></o:p></pre>
                        <pre><o:p> </o:p></pre>
                        <pre>                  3)+        //map the splash co-ordinates as per system scale<o:p></o:p></pre>
                        <pre>                  +        splash->x /= splash->scaleFactor;<o:p></o:p></pre>
                        <pre>                  +        splash->y /= splash->scaleFactor;<o:p></o:p></pre>
                        <pre><o:p> </o:p></pre>
                        <pre><o:p> </o:p></pre>
                        <pre><o:p> </o:p></pre>
                        <pre>                  This change is required only for windows and linux. As we<o:p></o:p></pre>
                        <pre>                  use absolute system resolution for centring the splash on<o:p></o:p></pre>
                        <pre>                  screen on these.<o:p></o:p></pre>
                        <pre><o:p> </o:p></pre>
                        <pre>                  i.e. if system resolution is 1920 X 1080(i.e. unscaled<o:p></o:p></pre>
                        <pre>                  resolution) on windows and linux we use this for centring<o:p></o:p></pre>
                        <pre>                  the splash on screen. For mac scaled resolution is used<o:p></o:p></pre>
                        <pre>                  directly.<o:p></o:p></pre>
                        <pre><o:p> </o:p></pre>
                        <pre>                  Regards,<o:p></o:p></pre>
                        <pre><o:p> </o:p></pre>
                        <pre>                  Rajeev Chamyal<o:p></o:p></pre>
                        <pre><o:p> </o:p></pre>
                        <pre>                  *From:*Alexander Scherbatiy<o:p></o:p></pre>
                        <pre>                  *Sent:* 11 August 2016 14:44<o:p></o:p></pre>
                        <pre>                  *To:* Rajeev Chamyal; <a moz-do-not-send="true" href="mailto:awt-dev@openjdk.java.net">awt-dev@openjdk.java.net</a><o:p></o:p></pre>
                        <pre>                  <a moz-do-not-send="true" href="mailto:awt-dev@openjdk.java.net"><mailto:awt-dev@openjdk.java.net></a>; Philip Race; Sergey<o:p></o:p></pre>
                        <pre>                  Bylokhov<o:p></o:p></pre>
                        <pre>                  *Subject:* Re:<AWT Dev>  <Swing Dev>[9] Review Request<o:p></o:p></pre>
                        <pre>                  JDK-8151787 Unify the HiDPI splash screen image naming<o:p></o:p></pre>
                        <pre>                  convention<o:p></o:p></pre>
                        <pre><o:p> </o:p></pre>
                        <pre>                  On 10/08/16 19:24, Alexandr Scherbatiy wrote:<o:p></o:p></pre>
                        <pre><o:p> </o:p></pre>
                        <pre><o:p> </o:p></pre>
                        <pre><o:p> </o:p></pre>
                        <pre>                      On 8/9/2016 11:18 AM, Rajeev Chamyal wrote:<o:p></o:p></pre>
                        <pre><o:p> </o:p></pre>
                        <pre>                          Hello All,<o:p></o:p></pre>
                        <pre><o:p> </o:p></pre>
                        <pre>                          Please review the following webrev.<o:p></o:p></pre>
                        <pre><o:p> </o:p></pre>
                        <pre>                          Bug:<o:p></o:p></pre>
                        <pre><a moz-do-not-send="true" href="https://bugs.openjdk.java.net/browse/JDK-8151787">https://bugs.openjdk.java.net/browse/JDK-8151787</a><o:p></o:p></pre>
                        <pre><o:p> </o:p></pre>
                        <pre>                          Webrev:<o:p></o:p></pre>
                        <pre><o:p> </o:p></pre>
                        <pre><a moz-do-not-send="true" href="http://cr.openjdk.java.net/%7Erchamyal/8151787/webrev.00/">http://cr.openjdk.java.net/~rchamyal/8151787/webrev.00/</a><o:p></o:p></pre>
                        <pre><o:p> </o:p></pre>
                        <pre><a moz-do-not-send="true" href="http://cr.openjdk.java.net/%7Erchamyal/8151787/webrev.00/"><http://cr.openjdk.java.net/%7Erchamyal/8151787/webrev.00/></a><o:p></o:p></pre>
                        <pre><o:p> </o:p></pre>
                        <pre><o:p> </o:p></pre>
                        <pre>                          Issue: Currently different naming conventions are<o:p></o:p></pre>
                        <pre>                          used for Hidpi image on different platforms.<o:p></o:p></pre>
                        <pre><o:p> </o:p></pre>
                        <pre>                          With this change the names will be unified across<o:p></o:p></pre>
                        <pre>                          all platforms.<o:p></o:p></pre>
                        <pre><o:p> </o:p></pre>
                        <pre>                          For a unscaled image image.ext following naming<o:p></o:p></pre>
                        <pre>                          convention will be followed.<o:p></o:p></pre>
                        <pre><o:p> </o:p></pre>
                        <pre>                          Unscaled name: image.ext<o:p></o:p></pre>
                        <pre><o:p> </o:p></pre>
                        <pre>                          Supported Scaled Names:<o:p></o:p></pre>
                        <pre><o:p> </o:p></pre>
                        <pre>                          If screen scale is integer number e.g. 2:<o:p></o:p></pre>
                        <pre>                          <a moz-do-not-send="true" href="mailto:image@2x.ext">image@2x.ext</a><a moz-do-not-send="true" href="mailto:image@2x.ext"><mailto:image@2x.ext></a><o:p></o:p></pre>
                        <pre><o:p> </o:p></pre>
                        <pre>                          If screen scale is float value like 1.25:<o:p></o:p></pre>
                        <pre>                          <o:p></o:p></pre>
                        <pre><a moz-do-not-send="true" href="mailto:image@125pct.ext">image@125pct.ext</a><a moz-do-not-send="true" href="mailto:image@125pct.ext"><mailto:image@125pct.ext></a><o:p></o:p></pre>
                        <pre><o:p> </o:p></pre>
                        <pre><o:p> </o:p></pre>
                        <pre>                      The fix should be reviewed on the awt-dev alias.<o:p></o:p></pre>
                        <pre><o:p> </o:p></pre>
                        <pre>                      + if(*scaleFactor - (int)*scaleFactor<<o:p></o:p></pre>
                        <pre>0.000001)<o:p></o:p></pre>
                        <pre><o:p> </o:p></pre>
                        <pre>                      Should there be so high precision there? Could only<o:p></o:p></pre>
                        <pre>                      percent values be compared like<o:p></o:p></pre>
                        <pre>                       if ((*scaleFactor *100) !=<o:p></o:p></pre>
                        <pre>((int)(*scaleFactor))<o:p></o:p></pre>
                        <pre>*<o:p></o:p></pre>
                        <pre>100)<o:p></o:p></pre>
                        <pre><o:p> </o:p></pre>
                        <pre><o:p> </o:p></pre>
                        <pre>                      +        //map the splash co-ordinates as per system scale<o:p></o:p></pre>
                        <pre>                      +        splash->x /= splash->scaleFactor;<o:p></o:p></pre>
                        <pre>                      +        splash->y /= splash->scaleFactor;<o:p></o:p></pre>
                        <pre><o:p> </o:p></pre>
                        <pre>                      It looks like the splash coordinates and sizes are<o:p></o:p></pre>
                        <pre>                      rescaled in different places. Is it possible to do<o:p></o:p></pre>
                        <pre>                      that in the same place? May be in<o:p></o:p></pre>
                        <pre>                      java_awt_SplashScreen.c file getBounds() function?<o:p></o:p></pre>
                        <pre><o:p> </o:p></pre>
                        <pre><o:p> </o:p></pre>
                        <pre>                  src/java.desktop/unix/native/libsplashscreen/splashscreen_sys.c<o:p></o:p></pre>
                        <pre>                         *scaleFactor = getNativeScaleFactor();<o:p></o:p></pre>
                        <pre><o:p> </o:p></pre>
                        <pre>                  Could you also include the change which requires to add<o:p></o:p></pre>
                        <pre>                  some default output screen name to the<o:p></o:p></pre>
                        <pre>                  getNativeScaleFactor() function on Linux. There is the<o:p></o:p></pre>
                        <pre>                  discussion about that:<o:p></o:p></pre>
                        <pre><o:p> </o:p></pre>
                        <pre><a moz-do-not-send="true" href="http://mail.openjdk.java.net/pipermail/awt-dev/2016-August/011766">http://mail.openjdk.java.net/pipermail/awt-dev/2016-August/011766</a>.<o:p></o:p></pre>
                        <pre>h<o:p></o:p></pre>
                        <pre>t<o:p></o:p></pre>
                        <pre>m<o:p></o:p></pre>
                        <pre>l<o:p></o:p></pre>
                        <pre><o:p> </o:p></pre>
                        <pre>                  Thanks,<o:p></o:p></pre>
                        <pre>                  Alexandr.<o:p></o:p></pre>
                        <pre><o:p> </o:p></pre>
                        <pre><o:p> </o:p></pre>
                        <pre><o:p> </o:p></pre>
                        <pre><o:p> </o:p></pre>
                        <pre>                      Thanks,<o:p></o:p></pre>
                        <pre>                      Alexandr.<o:p></o:p></pre>
                        <pre><o:p> </o:p></pre>
                        <pre><o:p> </o:p></pre>
                        <pre><o:p> </o:p></pre>
                        <pre><o:p> </o:p></pre>
                        <pre>                          Regards,<o:p></o:p></pre>
                        <pre><o:p> </o:p></pre>
                        <pre>                          Rajeev Chamyal<o:p></o:p></pre>
                        <pre><o:p> </o:p></pre>
                      </blockquote>
                    </blockquote>
                  </blockquote>
                </blockquote>
              </blockquote>
            </blockquote>
          </blockquote>
          <pre><o:p> </o:p></pre>
        </blockquote>
      </div>
    </blockquote>
    <br>
  </body>
</html>