You should probably fold the two if statements into one.
Also one line if statements, while they are concise are often problematic 1) because it's easy to overlook that it's even there and 2) because there are no braces to protect against someone adding another statement mistakenly. I have a slight preference to not have one-line or brace-less if's anywhere in Mixxx if possible since they can be a source of bugs.
Go ahead and merge to trunk -- we haven't been strict at all about the feature freeze this time around and this only affects controllers with it explicitly enabled so I don't see a problem.
Looks good, thanks Sean
Minor nitpick in midiscriptengin e.cpp
if(cot != NULL) { newValue) ; ignore( group,name, newValue) ) cot->slotSet( newValue) ;
- cot->slotSet(
+ if (!m_st.
}
You should probably fold the two if statements into one.
Also one line if statements, while they are concise are often problematic 1) because it's easy to overlook that it's even there and 2) because there are no braces to protect against someone adding another statement mistakenly. I have a slight preference to not have one-line or brace-less if's anywhere in Mixxx if possible since they can be a source of bugs.
Go ahead and merge to trunk -- we haven't been strict at all about the feature freeze this time around and this only affects controllers with it explicitly enabled so I don't see a problem.
thanks again,
rj