RFR: 8341735: Rewrite the build/AbsPathsInImage.java test to not load the entire file at once [v5]

Daniel Hu duke at openjdk.org
Tue Oct 14 19:59:54 UTC 2025


> These changes should prevent entire binary files from being loaded into memory for build/AbsPathsInImage.java test. I chose a default buffer size of 8KB since BufferedInputStream uses that, but open to alternative solutions. GHA passes and test passes on linux x64.

Daniel Hu 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 11 additional commits since the last revision:

 - Merge branch 'openjdk:master' into 8341735
 - syntax fixes
 - implement KMP
 - refactor logic
 - speedup scanBytes
 - Merge branch 'openjdk:master' into 8341735
 - redid byte matching which should fix cross chunk strings
 - style improvements
 - remove 8341024
 - remove trailing whitespace
 - ... and 1 more: https://git.openjdk.org/jdk/compare/25296754...82a4cc91

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

Changes:
  - all: https://git.openjdk.org/jdk/pull/26030/files
  - new: https://git.openjdk.org/jdk/pull/26030/files/c3d7a77b..82a4cc91

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

  Stats: 323209 lines in 5120 files changed: 228779 ins; 64158 del; 30272 mod
  Patch: https://git.openjdk.org/jdk/pull/26030.diff
  Fetch: git fetch https://git.openjdk.org/jdk.git pull/26030/head:pull/26030

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


More information about the build-dev mailing list