Skip to content

Fix issue with spaces in paths on open with#1194

Merged
chreden merged 1 commit intomasterfrom
bug/1192
Oct 12, 2023
Merged

Fix issue with spaces in paths on open with#1194
chreden merged 1 commit intomasterfrom
bug/1192

Conversation

@chreden
Copy link
Copy Markdown
Owner

@chreden chreden commented Oct 12, 2023

When trview is associated with level files and you open a level in explorer if the path has spaces in then the CommandLineToArgvW will split on spaces - we now stick them back together to form the original path.
This means flags have to come first but they aren't really used yet anyway.
Also fixes an issue in the map renderer where it would try to use imgui before it was ready in some situations.
Closes #1192

When trview is associated with level files and you open a level in explorer if the path has spaces in then the `CommandLineToArgvW` will split on spaces - we now stick them back together to form the original path.
This means flags have to come first but they aren't really used yet anyway.
Also fixes an issue in the map renderer where it would try to use imgui before it was ready in some situations.
Closes #1192
@chreden chreden added the bug label Oct 12, 2023
@chreden chreden added this to the Next milestone Oct 12, 2023
@chreden chreden requested review from lahm86 and makotocchi October 12, 2023 01:03
@chreden chreden self-assigned this Oct 12, 2023
@chreden chreden merged commit 5f92bab into master Oct 12, 2023
@chreden chreden deleted the bug/1192 branch October 12, 2023 23:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Paths with spaces can't be opened from explorer

2 participants