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

Add more MIDI definitions #1539

Merged
merged 3 commits into from
Aug 31, 2024
Merged

Conversation

jlucfarias
Copy link
Contributor

All definitions are commented out because none of them (unless the early added definitions and the current required) are really necessary right now.
I did not found these definitions in any Apple OSS repository, then I get all from Apple's Documentation.

Trying to fix #1533

Copy link
Contributor

@CuriousTommy CuriousTommy 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!

One optional suggestion I have is to add a note in MIDIServices.m about how it is intentional for some of these values to be nil.

Usually, we want to avoid lazily setting values as "nil". However, there are some instances where Apple would set these values as "nil".
@CuriousTommy CuriousTommy merged commit 7d3041a into darlinghq:master Aug 31, 2024
2 checks passed
@CuriousTommy
Copy link
Contributor

Thanks for your contribution!

@jlucfarias jlucfarias deleted the add-midi-symbols branch September 2, 2024 19:06
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.

Propellerhead reason dyld: Symbol not found: _kMIDIPropertyAdvanceScheduleTimeMuSec
2 participants