<AWT Dev> [9] Review request for JDK-8145173 HiDPI splash screen support on Windows

Sergey Bylokhov Sergey.Bylokhov at oracle.com
Thu Mar 3 10:07:23 UTC 2016


In general it looks fine to me. I will try to patch my local ws and run 
some sanity test.

On 03.03.16 8:50, Rajeev Chamyal wrote:
> Hello All,
>
> Please review the updated webrev.
>
> Added a free call for duplicate file name in splashscreen_sys.c ::
> SplashGetScaledImageName
>
> http://cr.openjdk.java.net/~rchamyal/8145173/webrev.02/
>
> Regards,
>
> Rajeev Chamyal
>
> *From:*Rajeev Chamyal
> *Sent:* 01 March 2016 15:45
> *To:* awt-dev at openjdk.java.net; Sergey Bylokhov; Alexander Scherbatiy
> *Subject:* RE: <AWT Dev> [9] Review request for JDK-8145173 HiDPI splash
> screen support on Windows
>
> Hello All,
>
> Gentle reminder.
>
> Please review the updated webrev.
>
>
> http://cr.openjdk.java.net/~rchamyal/8145173/webrev.01/
>
> Regards,
>
> Rajeev Chamyal
>
> *From:*Rajeev Chamyal
> *Sent:* 16 February 2016 16:01
> *To:* awt-dev at openjdk.java.net <mailto:awt-dev at openjdk.java.net>; Sergey
> Bylokhov; Alexander Scherbatiy
> *Subject:* <AWT Dev> [9] Review request for JDK-8145173 HiDPI splash
> screen support on Windows
>
> Hello All,
>
> Could you please review the following fix.
>
> Bug : https://bugs.openjdk.java.net/browse/JDK-8145173
>
> Webrev : http://cr.openjdk.java.net/~rchamyal/8145173/webrev.00/
>
> This is an enhancement to support HiDPI splash screen on windows.
>
> As a part of this enhancement implementation to
> splashscreen_sys.c::SplashGetScaledImageName method has been provided.
>
> System dpi and scale factor are used to determine the scaled image name.
> Dpi value is read using GetDpiForMonitor API on Windows 8 and
> GetDesktopDpi API on Windows 7.
>
> Scale factor is calculated from the dpi value.
>
> The naming convention followed for scaled image name is as follows:
>
> Refer :
> https://msdn.microsoft.com/en-us/library/windows/apps/xaml/hh965325.aspx
>
> Unscaled image name : image.ext
>
> Scaled image name : image.scale-<dpi value>./ext/
>
> Regards,
>
> Rajeev Chamyal
>


-- 
Best regards, Sergey.


More information about the awt-dev mailing list