RFR: 8341735: Rewrite the build/AbsPathsInImage.java test to not load the entire file at once [v7]
Daniel Hu
duke at openjdk.org
Thu Oct 23 18:36:37 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 14 additional commits since the last revision:
- Merge branch 'openjdk:master' into 8341735
- fix incorrect use of inputstream
- remove extraneous variables/imports
- 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
- ... and 4 more: https://git.openjdk.org/jdk/compare/4018a24f...9990811e
-------------
Changes:
- all: https://git.openjdk.org/jdk/pull/26030/files
- new: https://git.openjdk.org/jdk/pull/26030/files/6a72a163..9990811e
Webrevs:
- full: https://webrevs.openjdk.org/?repo=jdk&pr=26030&range=06
- incr: https://webrevs.openjdk.org/?repo=jdk&pr=26030&range=05-06
Stats: 23774 lines in 688 files changed: 14355 ins; 5561 del; 3858 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