File handling tidy up #1749
Merged
File handling tidy up #1749
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
emericklaw
added a commit
to emericklaw/Bruce
that referenced
this pull request
Oct 23, 2025
…one in BruceDevices#1749 for rest of codebase
emericklaw
force-pushed
the
file-handling-tidy-up
branch
from
71c4110 to
f1302ba
Compare
emericklaw
added a commit
to emericklaw/Bruce
that referenced
this pull request
Oct 23, 2025
…one in BruceDevices#1749 for rest of codebase
emericklaw
force-pushed
the
file-handling-tidy-up
branch
from
261ad7d to
f1302ba
Compare
Merged
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Proposed Changes
openNextFiletogetNextFileNamefor speedUpdate storageReaddir function to accept an missing additional parameter, theNow in Interpreter reorganisation + new functions #1763optionsparameter was never passed to the function sowithFileTypes=truewould have never workedTypes of Changes
New Feature/Bugfix
Verification
The biggest change related to
getNextFileName, I have tested each place it was changed and everything still works.This change was also made in Launcher bmorcelli/Launcher#230
Testing
None
Linked Issues
User-Facing Change
Further Comments