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

Renamed GOSetter::m_swell to m_CrescendoCtrl #2014

Merged
merged 2 commits into from
Oct 1, 2024

Conversation

oleg68
Copy link
Contributor

@oleg68 oleg68 commented Sep 28, 2024

This is a next PR related to the crescendo.

It just renames m_swell to m_CrescendoEnclosure.

No GO behavior should be changed.

@larspalo
Copy link
Contributor

@oleg68 Do you realize that a crescendo pedal has nothing at all with an enclosure to do?

@oleg68
Copy link
Contributor Author

oleg68 commented Sep 29, 2024

@oleg68 Do you realize that a crescendo pedal has nothing at all with an enclosure to do?

In the real life - yes. But GrandOrgue uses the same GOEnclosure class for both swell/enclosures and crescendo.

Earlier GOSetter had the m_swell member for handling crescendo pedal. It was not clear. So I rename it to m_CrescendoEnclosure. Crescendo - because it is used for handling crescendo, Enclosure - because it has the GOEnclosure * datatype. If you disagree, I may rename it to m_CrescendoPedalCtl.

@larspalo
Copy link
Contributor

I may rename it to m_CrescendoPedalCtl

I like that much better, yes.

The visual (GUI) expression pedal rendering of swell/enclosure (enclosed swell box) and the crescendo (register crescendo) might very well be of the same type even if the crescendo in real life also might be done as a Rollschweller type. The backend functionality and model is vastly different for them though. Perhaps it could be a future area of improvement to more clearly differentiate between them...

@oleg68 oleg68 changed the title Renamed GOSetter::m_swell to m_CrescendoEnclosure Renamed GOSetter::m_swell to m_CrescendoCtrl Sep 30, 2024
@oleg68
Copy link
Contributor Author

oleg68 commented Sep 30, 2024

I like that much better, yes.

Renamed to m_CrescendoCtrl

@oleg68 oleg68 merged commit 8e250a1 into GrandOrgue:master Oct 1, 2024
1 check passed
@oleg68 oleg68 deleted the refactor/go-setter-swell branch October 1, 2024 08:14
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