-
-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
glwaveformrenderbackground #11523
glwaveformrenderbackground #11523
Conversation
…sing glClear to clear waveform widget
Thank you. I can confirm that this fixes #11164. |
Maintaining 3 differing major branches is a PITA for everyone involved and waste of scarce resources. The project should be aware of the consequences of its decisions. Last time that I build 2.3. |
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.
@Swiftb0y Works for me.
Thank you. 2.3 will be retired soon. 2.3.5 will likely be our last 2.3 release before 2.4 |
I can confirm that the waveforms are still working @60 Hz on Ubuntu Focal and |
I have made a minor change based on @Swiftb0y review (aesthetic only, code execution should be identical). |
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.
LGTM. I highly thank you for helping us fix the issue. I highly doubt anyone else would've identified and contributed a potential fix as fast as you did in this PR.
@m0dB You contributions are greatly appreciated and needed to keep Mixxx alive. |
Thanks guys :-) It sure helped that I have been deep down the waveform renderer rabbit hole! |
glwaveformrenderbackground is a replacement for waveformrenderbackground, which uses glClear to clear the waveform widget. this is a potential fix for #11164