RFR: JDK-8295710 : unify os::dll_file_extension
David Holmes
dholmes at openjdk.org
Thu Oct 20 12:18:48 UTC 2022
On Thu, 20 Oct 2022 08:12:04 GMT, Matthias Baesken <mbaesken at openjdk.org> wrote:
> The os::dll_file_extension() function could be unified for the different OS platforms.
This looks good but I think we can do better :) There is only a single use of `os::dll_file_extension`:
./share/compiler/disassembler.cpp: int written = jio_snprintf(&buf[offset], sz, "%s%s", hsdis_library_name, os::dll_file_extension());
so we can remove it altogether and just replace with `JNI_LIB_SUFFIX`.
Thanks.
-------------
Changes requested by dholmes (Reviewer).
PR: https://git.openjdk.org/jdk/pull/10782
More information about the hotspot-runtime-dev
mailing list