>  8218914: Handle the case when fonts are installed into user registry key. This is the default behaviour since Windows 10 1809.

When you get to the point of preparing a changeset, this line should have the bug synopsis.
The text you have here is better placed on the "Summary:" line.

You seem to have lines > 80 chars. Please fix.

What does Windows do if a user installs a different version of a font already installed on the system ?
- Refuse to install it ?
- Use the system one ?
- Use the user one ?

If it refuses to install it, we can ignore that problem. If it prefers one, we should make sure
we do the same.

I think the comment

/* Starting from Windows 10 Preview Build 17704 fonts are installed into user's home folder by default,


can be misconstrued. It could be read as ALL fonts are installed into a user folder and
there is no more system location. I think you actually mean

/* Starting from Windows 10 Preview Build 17704, when a user installs non-system fonts, * then by default they are installed in a new per-user location as specified in a * per user registry entry. */


Have you tested this on a machine with at least several user fonts installed and verified we still get ALL the same system fonts as well as the new user fonts ?

Have you verified what this does on older OS versions ?

-phil.

On 2/15/19 6:23 AM, Mikhail Filippov wrote:
Hi. Please review the fix.

patch: attached to message.
bug: https://bugs.openjdk.java.net/browse/JDK-8218914
webrev: http://cr.openjdk.java.net/~dbatrak/8218914/webrev.01/

Description:
Starting from Windows 10 Preview Build 17704 fonts are installed into the user's home folder by default, and are listed in user's registry section. This is Microsoft blog post about it: "https://blogs.windows.com/windowsexperience/2018/06/27/announcing-windows-10-insider-preview-build-17704/"; I this patch I extract function for registry access and call it for two keys: HKEY_LOCAL_MACHINE and HKEY_CURRENT_USER. In original code fonts loading only from HKEY_LOCAL_MACHINE.



--
Mikhail Filippov
Software Developer
JetBrains
http://jetbrains.com
“The Drive To Develop"


Reply via email to