Skip to content

Refactoring: Use Paths.get() to avoid concatenation with File.separator#677

Merged
Flowdalic merged 2 commits into
igniterealtime:masterfrom
stokito:path
Oct 12, 2025
Merged

Refactoring: Use Paths.get() to avoid concatenation with File.separator#677
Flowdalic merged 2 commits into
igniterealtime:masterfrom
stokito:path

Conversation

@stokito

@stokito stokito commented Oct 4, 2025

Copy link
Copy Markdown
Member

loadFormFieldRegistryEntries(): use try with close

@Flowdalic

Copy link
Copy Markdown
Member

Thanks for your contribution. Could you elaborate on the motivation for this change?

@stokito

stokito commented Oct 5, 2025

Copy link
Copy Markdown
Member Author

This is more a code style and modernization type of change.
The Paths.get() makes the concatenation with the File.separator and makes code shorter and more stable. In many API places now used the Path instead of a string.
As a downside it may work slower in some places.

@Flowdalic Flowdalic added this pull request to the merge queue Oct 12, 2025
Merged via the queue into igniterealtime:master with commit 7e5413b Oct 12, 2025
3 checks passed
@stokito stokito deleted the path branch October 12, 2025 14:09
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.

2 participants