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

made glwaveformrenderer a subclass of waveformrenderersignalbase #10964

Merged
merged 1 commit into from
Oct 17, 2022

Conversation

m0dB
Copy link
Contributor

@m0dB m0dB commented Oct 14, 2022

Improving the class hierarchy for waverenderers, as discussed here: https://mixxx.zulipchat.com/#narrow/stream/109171-development/topic/class.20hierarchy.20refactoring (includes before and after diagrams)

@m0dB m0dB force-pushed the glwaveformrenderer_hierarchy branch from ee2c646 to 7d4de24 Compare October 14, 2022 18:05
@m0dB m0dB force-pushed the glwaveformrenderer_hierarchy branch from 7d4de24 to 15a0d78 Compare October 14, 2022 21:36
@m0dB
Copy link
Contributor Author

m0dB commented Oct 17, 2022

ping @daschuer :-)

Copy link
Member

@daschuer daschuer left a comment

Choose a reason for hiding this comment

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

LGTM, Thank you.

@daschuer daschuer merged commit e9def4d into mixxxdj:main Oct 17, 2022
@daschuer
Copy link
Member

All waveforms and the V-Sync-test behaves like before.

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.

2 participants