-
Notifications
You must be signed in to change notification settings - Fork 2.5k
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
[MU4] Porting 3.5.2_backend fixes to master #6818
[MU4] Porting 3.5.2_backend fixes to master #6818
Conversation
#ifdef ZITA_REVERB | ||
{ | ||
0, "Zita1" | ||
}, | ||
#endif | ||
{ | ||
2, "0.1" | ||
}, | ||
{ 3, "440" }, | ||
{ 4, "1" }, | ||
{ 5, "1" } |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why only change the first two and not all others?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Because the code style check only wanted those. IMHO bad enough...
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is the problem of introducing automatic check. Do we still get to vote for its expulsion from MuseScore? :)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It comes with a price tag for sure...
No description provided.