diff options
author | Jan Holthuis <jan.holthuis@ruhr-uni-bochum.de> | 2020-10-27 22:58:00 +0100 |
---|---|---|
committer | Jan Holthuis <jan.holthuis@ruhr-uni-bochum.de> | 2020-10-27 23:07:32 +0100 |
commit | 758341f6b30a51e28e3052a1a3f8850ddcf4ec20 (patch) | |
tree | ad709910a989b7ed5e89b257c2602faf89db47c6 /src/control | |
parent | c60e8e727dcbd878c8aea08240c7df7defec7c4e (diff) |
Use more comparisons instead of implicit casts to double
Diffstat (limited to 'src/control')
-rw-r--r-- | src/control/controlbehavior.cpp | 12 | ||||
-rw-r--r-- | src/control/controllinpotmeter.cpp | 4 |
2 files changed, 8 insertions, 8 deletions
diff --git a/src/control/controlbehavior.cpp b/src/control/controlbehavior.cpp index 720fa7399e..f6de005a90 100644 --- a/src/control/controlbehavior.cpp +++ b/src/control/controlbehavior.cpp @@ -187,7 +187,7 @@ double ControlAudioTaperPotBehavior::valueToParameter(double dValue) { // m_minDB = 0 // 0 dB = m_neutralParameter double overlay = m_offset * (1 - dValue); - if (m_minDB) { + if (m_minDB != 0) { dParam = (ratio2db(dValue + overlay) - m_minDB) / m_minDB * m_neutralParameter * -1; } else { dParam = dValue * m_neutralParameter; @@ -211,7 +211,7 @@ double ControlAudioTaperPotBehavior::parameterToValue(double dParam) { // db + linear overlay to reach // m_minDB = 0 // 0 dB = m_neutralParameter; - if (m_minDB) { + if (m_minDB != 0) { double db = (dParam * m_minDB / (m_neutralParameter * -1)) + m_minDB; dValue = (db2ratio(db) - m_offset) / (1 - m_offset) ; } else { @@ -232,7 +232,7 @@ double ControlAudioTaperPotBehavior::parameterToValue(double dParam) { double ControlAudioTaperPotBehavior::midiToParameter(double midiValue) { double dParam; - if (m_neutralParameter && m_neutralParameter != 1.0) { + if (m_neutralParameter != 0 && m_neutralParameter != 1.0) { double neutralTest = (midiValue - m_midiCorrection) / 127.0; if (neutralTest < m_neutralParameter) { dParam = midiValue / @@ -255,7 +255,7 @@ double ControlAudioTaperPotBehavior::valueToMidiParameter(double dValue) { // always on a full Midi integer double dParam = valueToParameter(dValue); double dMidiParam = dParam * 127.0; - if (m_neutralParameter && m_neutralParameter != 1.0) { + if (m_neutralParameter != 0 && m_neutralParameter != 1.0) { if (dParam < m_neutralParameter) { dMidiParam += m_midiCorrection * dParam / m_neutralParameter; } else { @@ -316,8 +316,8 @@ void ControlPushButtonBehavior::setValueFromMidi( auto* timer = getTimer(); if (pressed) { // Toggle on press - double value = pControl->get(); - pControl->set(!value, NULL); + bool value = (pControl->get() != 0); + pControl->set(!value, nullptr); timer->setSingleShot(true); timer->start(kPowerWindowTimeMillis); } else if (!timer->isActive()) { diff --git a/src/control/controllinpotmeter.cpp b/src/control/controllinpotmeter.cpp index e3f16d44b4..c3ab24cca5 100644 --- a/src/control/controllinpotmeter.cpp +++ b/src/control/controllinpotmeter.cpp @@ -9,10 +9,10 @@ ControlLinPotmeter::ControlLinPotmeter(ConfigKey key, m_pControl->setBehavior( new ControlLinPotmeterBehavior(dMinValue, dMaxValue, allowOutOfBounds)); } - if (dStep) { + if (dStep != 0) { setStepCount(static_cast<int>((dMaxValue - dMinValue) / dStep)); } - if (dSmallStep) { + if (dSmallStep != 0) { setSmallStepCount(static_cast<int>((dMaxValue - dMinValue) / dSmallStep)); } } |