Skip to content

Experiments#819

Draft
garydgregory wants to merge 376 commits into
fix/read-onlyfrom
master
Draft

Experiments#819
garydgregory wants to merge 376 commits into
fix/read-onlyfrom
master

Conversation

@garydgregory

Copy link
Copy Markdown
Member

No description provided.

@github-advanced-security

Copy link
Copy Markdown

This pull request sets up GitHub code scanning for this repository. Once the scans have completed and the checks have passed, the analysis results for this pull request branch will appear on this overview. Once you merge this pull request, the 'Security' tab will show more code scanning analysis results (for example, for the default branch). Depending on your configuration and choice of analysis tool, future pull requests will be annotated with code scanning analysis results. For more information about GitHub code scanning, check out the documentation.

@garydgregory garydgregory marked this pull request as draft December 14, 2025 15:29
This reverts commit 4cf10f9.
assumption about read return value

- Add test IOUtilsTest.testContentEquals_Reader_Reader_unevenReads()
- Revert unreleased code to previous implementation
garydgregory and others added 30 commits June 11, 2026 14:08
Replace Java 26-ea with 27-ea for experimental builds
… by (#858)

default.

To get the old behavior back, call
org.apache.commons.io.output.DeferredFileOutputStream.Builder.setDeleteTempFileOnClose(false)

- Add DeferredFileOutputStream.Builder.setDeleteTempFileOnClose(boolean)
- Add PathUtils.clear[IfExists](Path)
- Add PathUtils.deleteIfExists(Path)
- Update generic type of RandomAccessFileMode.accept(Path, IOConsumer)
from IOConsumer<RandomAccessFile> to IOConsumer<IORandomAccessFile>
- Harden tests for NPEs on failures, happy path unchanged.
default (#858)

To get the old behavior back, call
org.apache.commons.io.output.DeferredFileOutputStream.Builder.setDeleteTempFileOnClose(false)

- Add DeferredFileOutputStream.Builder.setDeleteTempFileOnClose(boolean)
- Add PathUtils.clear[IfExists](Path)
- Add PathUtils.deleteIfExists(Path)
- Update generic type of RandomAccessFileMode.accept(Path, IOConsumer)
from IOConsumer<RandomAccessFile> to IOConsumer<IORandomAccessFile>
- Harden tests for NPEs on failures, happy path unchanged.
…860)

* [IO-891] Fix BoundedReader skip accounting
Limit skip requests to the remaining bound and update charsRead with the actual skipped count.

* [IO-891] Refine BoundedReader skip bounds fix
         Keep skip(long) focused on maxCharsFromTargetReader and actual skipped count, without treating mark read-ahead as a hard limit.

* Fix Checkstyle build failure

---------

Co-authored-by: Sarankumar Baskar <sbaskar@redhat.com>
Co-authored-by: Gary Gregory <garydgregory@users.noreply.github.com>
- FilenameUtils.requireNonNullChars():
Reduce wordiness in exception message
- Use longer lines
- Less whitepsace in expressions
- Use Math.min().
- Use Math.max().
…862)

- Add tests to validate owner permission on POSIX.
- Refactor test to check a specific folder for POSIX instead of the
default FS.
- DeferredFileOutputStream.thresholdReached(): Throws an NPE with a
useful message.
- DeferredFileOutputStream.thresholdReached(): Better local variable
name.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

8 participants