-
-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
Macros Backend respin #13172
base: main
Are you sure you want to change the base?
Macros Backend respin #13172
Conversation
-Ignoring return value of function declared with 'nodiscard' attribute -Missing include
…ter clear, but clear intentional leaves a user specified label untouched
9ac8800
to
d61749c
Compare
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.
I tried the feature but could only get a single hotcue to work. It could be that I'm not using properly tho. Do you know if there is any wiki or doc on how to use this feature beside the GSoC wiki page?
@@ -227,6 +227,18 @@ EngineBuffer::EngineBuffer(const QString& group, | |||
m_pCueControl = new CueControl(group, pConfig); | |||
addControl(m_pCueControl); | |||
|
|||
for (int i = 1; i <= kMacrosPerChannel; ++i) { | |||
auto* pMacroControl = new MacroControl(group, pConfig, i); |
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.
Looks like this might currently be leaked. Perhaps should be replaced by a unique_ptr
I think there are only the three GSoC pages in our wiki, and the original PRs of cause. One aim of this feature was to achieve compatibility to Serato Flip, to allow data import from Serato. Therefore I guess, that the Serato Flip feature is similar to use: https://support.serato.com/hc/en-us/articles/203157550-Serato-Flip-User-Guide |
This is #4528 with fixed CI and solved merge conflicts.
It implements the following COs without GUI: https://github.com/mixxxdj/mixxx/wiki/Macros