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

Eliminated writting default value of midi setup to the cmb file https://github.com/GrandOrgue/grandorgue/issues/1199 #2120

Merged
merged 1 commit into from
Feb 23, 2025

Conversation

oleg68
Copy link
Contributor

@oleg68 oleg68 commented Feb 15, 2025

A next PR relating to #1199

Most of MIDI objects do not have any MIDI events assigned to them. Earlier when a user saved organ settings to a preset (*.cmb) file, the default values of MIDI object properties also were written to the file.

Now the default values are not written to .cmb, so only objects with any midi assignments appear in the saved files.

It changes nothing in loading the preset files, but it will be used for exporting MIDI settings to separate files in the future.

Copy link
Contributor

@larspalo larspalo left a comment

Choose a reason for hiding this comment

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

Seems to do what it should and respects settings from initial MIDI.

@oleg68
Copy link
Contributor Author

oleg68 commented Feb 22, 2025

@rousseldenis Could you approve this PR?

@oleg68 oleg68 merged commit 4779a50 into GrandOrgue:master Feb 23, 2025
1 check passed
@oleg68 oleg68 deleted the feature/save-not-default branch February 23, 2025 11: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.

3 participants