RFR: 8329178: Clean up jdk.accessibility native compilation
Erik Joelsson
erikj at openjdk.org
Wed Mar 27 17:51:23 UTC 2024
On Wed, 27 Mar 2024 12:00:28 GMT, Magnus Ihse Bursie <ihse at openjdk.org> wrote:
> This is a follow-up on JDK-8328680, making the same kind of cleanup to jdk.accessibility. The code here needed more work than for the other modules, so I wanted have it in a separate PR to get a more thorough review.
make/modules/jdk.accessibility/Launcher.gmk line 70:
> 68: LIBS_windows := advapi32.lib user32.lib, \
> 69: VERSIONINFO_RESOURCE := \
> 70: $(ACCESSIBILITY_SRCDIR)/jaccessinspector/jaccessinspectorWindow.rc, \
Should we increase relative indentation to 4?
make/modules/jdk.accessibility/Launcher.gmk line 95:
> 93: LIBS_windows := advapi32.lib comctl32.lib gdi32.lib user32.lib, \
> 94: VERSIONINFO_RESOURCE := \
> 95: $(ACCESSIBILITY_SRCDIR)/jaccesswalker/jaccesswalkerWindow.rc, \
Indentation 4?
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/18510#discussion_r1541573987
PR Review Comment: https://git.openjdk.org/jdk/pull/18510#discussion_r1541575150
More information about the build-dev
mailing list