Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix UTF encoding issues for wav sample and MSVC build #1300

Merged

Conversation

baconpaul
Copy link
Contributor

Reliably maintain unicode encoding on all strings for dragging a wav file from the browser into the engine and storing it in a multi. This fixes that simple case but we need a top to bottom review of all the other stream-a-path gestures in the codebase (although I think this is the majority of them)

Also MSVC has a bug with double-this-binding in lambdas which this commit fixes

@baconpaul baconpaul merged commit f1bbf71 into surge-synthesizer:main Sep 10, 2024
12 checks passed
@baconpaul baconpaul deleted the utf-means-ur-totally-effed branch September 10, 2024 23:13
Reliably maintain unicode encoding on all strings for dragging
a wav file from the browser into the engine and storing it in
a multi. This fixes that simple case but we need a top to bottom
review of all the other stream-a-path gestures in the codebase
(although I think this is the majority of them)

Also MSVC has a bug with double-this-binding in lambdas which
this commit fixes
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.

1 participant