> 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).

Volkan Yazici has updated the pull request incrementally with one additional 
commit since the last revision:

  Improve code style

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

Changes:
  - all: https://git.openjdk.org/jdk/pull/33649/files
  - new: https://git.openjdk.org/jdk/pull/33649/files/02ac947c..03d0a092

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=jdk&pr=33649&range=01
 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=33649&range=00-01

  Stats: 12 lines in 1 file changed: 6 ins; 2 del; 4 mod
  Patch: https://git.openjdk.org/jdk/pull/33649.diff
  Fetch: git fetch https://git.openjdk.org/jdk.git pull/33649/head:pull/33649

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

Reply via email to