On Fri, 30 Oct 2026 13:17:35 GMT, Volkan Yazici <[email protected]> wrote:
> Fix `RandomAccessFile#skipBytes(int)` and overhaul associated tests.
>
> ### Testing notes
>
> `SkipBytes` has gone through a major facelift.
>
> - Past EOF `getFilePointer()` check was broken, fixed in f15ccfaf.
> - `new RandomAccessFile("input.txt" , "rw")` was not working at all. File
> needed to be passed via `new File(System.getProperty("test.src", "."),
> "input.txt")`. Fixed in f4300dbb.
> - `input.txt` doesn't need to be VCS-tracked. Removed it in f4300dbb.
> - Extended the tests to reproduce the issue, see 049ae7c5. New cases are
> cloned from 46d34868 delivered in [JDK-8394316].
> - Migrated the tests to JUnit, see 02ac947c.
>
> [JDK-8394316]: https://bugs.openjdk.org/browse/JDK-8394316
>
> ---------
> - [X] I confirm that I make this contribution in accordance with the [OpenJDK
> Interim AI Policy](https://openjdk.org/legal/ai).
This pull request has now been integrated.
Changeset: 5cde5077
Author: Volkan Yazici <[email protected]>
URL:
https://git.openjdk.org/jdk/commit/5cde5077ec2b490dd209bf9b44fbb3f0a3ccf1aa
Stats: 265 lines in 4 files changed: 137 ins; 122 del; 6 mod
8396040: RandomAccessFile.skipBytes can skip backwards when n > 0
Reviewed-by: alanb
-------------
PR: https://git.openjdk.org/jdk/pull/33649
