Skip to content

A collection of bugfixes and performance improvements#4

Merged
gus33000 merged 33 commits intoOSTooling:masterfrom
hounsell:upstream
Nov 15, 2021
Merged

A collection of bugfixes and performance improvements#4
gus33000 merged 33 commits intoOSTooling:masterfrom
hounsell:upstream

Conversation

@hounsell
Copy link
Copy Markdown
Contributor

@hounsell hounsell commented Sep 8, 2021

Some of these are rather opinionated and happy to exclude from this branch.

Not yet ready to merge because there's some bugs and regressions that I'm aware of that need fixing first.

Don't reload the WIM for each index, just load the once and then switch between indexes as needed.
This is to try to prevent an overly-long filename, it adds "-multi" to distinguish these files
* Missing readme.txt on placeholder ISO-9660
* WIM XML doesn't contain description
@hounsell hounsell marked this pull request as ready for review November 14, 2021 23:29
@hounsell
Copy link
Copy Markdown
Contributor Author

I can't remember everything I wanted to do, but I think this covers most and includes a number of nice fixes to include in here.

Let me know if there's something I've missed.

@gus33000 gus33000 self-requested a review November 15, 2021 07:46
Copy link
Copy Markdown
Collaborator

@gus33000 gus33000 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me. Thanks again :)

@gus33000 gus33000 merged commit d9a967e into OSTooling:master Nov 15, 2021
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