On Mon, 3 Mar 2025 12:22:01 GMT, Kevin Walls <kev...@openjdk.org> wrote:
> Following on from JDK-8350820, which backed out the _snprintf to snprintf > change (JDK-8336289) in OperatingSystemImpl.c on Windows, because the counter > names were being truncated (so CPU monitoring was not possible). > > This change moves to snprintf again, but the counter names are not truncated. > > snprintf must need the null terminator to fit inside the buffer length given. > It does not, and snprintf truncates (and always add the null terminator). Thanks for reviews. Reran with edited problemlist to run relevant tests, all good. Hope we can clear problemlist in near future!... ------------- PR Comment: https://git.openjdk.org/jdk/pull/23861#issuecomment-2703704150
