RFR: 8295017: Remove Windows specific workaround in JLI_Snprintf [v4]

Julian Waters jwaters at openjdk.org
Wed Oct 12 13:36:36 UTC 2022


> The C99 snprintf is available with Visual Studio 2015 and above, alongside Windows 10 and the UCRT, and is no longer identical to the outdated Windows _snprintf. Since support for the Visual C++ 2017 compiler was removed a while ago, we can now safely remove the compatibility workaround on Windows and have JLI_Snprintf simply delegate to snprintf.

Julian Waters has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains six additional commits since the last revision:

 - Merge branch 'openjdk:master' into patch-1
 - Comment documenting change isn't required
 - Merge branch 'openjdk:master' into patch-1
 - Comment formatting
 - Remove Windows specific JLI_Snprintf implementation
 - Remove Windows JLI_Snprintf definition

-------------

Changes:
  - all: https://git.openjdk.org/jdk/pull/10625/files
  - new: https://git.openjdk.org/jdk/pull/10625/files/9149aae1..8ac9b519

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=jdk&pr=10625&range=03
 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=10625&range=02-03

  Stats: 7257 lines in 143 files changed: 5279 ins; 912 del; 1066 mod
  Patch: https://git.openjdk.org/jdk/pull/10625.diff
  Fetch: git fetch https://git.openjdk.org/jdk pull/10625/head:pull/10625

PR: https://git.openjdk.org/jdk/pull/10625


More information about the core-libs-dev mailing list