[11u] RFR: jnf_removal, series of patches
Vladimir Kempik
vkempik at azul.com
Fri Apr 16 12:12:15 UTC 2021
The collated webrev:
http://cr.openjdk.java.net/~vkempik/jnf_removal/webrev.01/ <http://cr.openjdk.java.net/~vkempik/jnf_removal/webrev.01/>
Testing - tier1, will soon update on tier2.
> 16 апр. 2021 г., в 14:39, Vladimir Kempik <vkempik at azul.com> написал(а):
>
> Hello
> Please review this backport of 11 patches from jdk17 to jdk11, they remove dependendy on JavaNativeFoundation.framework on macos
> It’s 7 original patches fo jnf removal and 4 postmortem items to fix bugs.
> I have created one collated webrev for 11 changesets, please let me know if it’s not the best way to do it.
> Brief summary for backports:
> 1_8257858 & 8257860 - Applies almost clean, few gmk files had different filename and few copyright year issues, context difference in gmk files.
> 2_8257853 - Not clean, most of issues were in CPrinterJob and JavaAccessibility due to changes in jdk12+. Rest was mostly context code difference.
> 3_8259343 - Clean, only gmk files renames were needed
> 4_8259651 - Not clean, have to remove some changes to parts which are missing in jdk11: CMenuBar: ActivateDefaultMenuBar, Accessebility: titleChanged, getMultiClickTime
> 5_8259869 - Almost Clean, slight Context code difference in CGLLayer.h
> 6_8260616 & 8259729 - not clean, Missing CommonTextAccessibility.m in jdk11. Some Context code difference. Two changes in one patch to not break the build
> 7_8257988 - Clean
> 8_8259232 - clean
> 9_8259585 - clean
> 10_8261198 - Clean
> 11_8263846 - clean
>
> So I need review for patches #1 #2 #4 #5 #6 and maybe (not sure) #3
>
> #1 the bug - https://bugs.openjdk.java.net/browse/JDK-8257858
> Original changeset - https://github.com/openjdk/jdk/commit/4a8b5c16
>
> #2 the bug - https://bugs.openjdk.java.net/browse/JDK-8257853
> Original changeset - https://github.com/openjdk/jdk/commit/fa50877c
>
> #3 the bug - https://bugs.openjdk.java.net/browse/JDK-8259343
> Original changeset - https://git.openjdk.java.net/jdk/commit/d6a2105b
>
> #4 the bug - https://bugs.openjdk.java.net/browse/JDK-8259651
> Original changeset - https://github.com/openjdk/jdk/commit/5855d52a
>
> #5 the bug - https://bugs.openjdk.java.net/browse/JDK-8259869
> Original changeset - https://github.com/openjdk/jdk/commit/92c2f084
>
> #6 the bug - https://bugs.openjdk.java.net/browse/JDK-8260616
> Original changeset - https://github.com/openjdk/jdk/commit/8760688d
>
> Regards, Vladimir
>
More information about the jdk-updates-dev
mailing list