Visualize distance to a key when pitch shifting

src/engine/keycontrol.cpp

@@ -1,4 +1,5 @@
 #include <QtDebug>
+#include <QPair>
 
 #include "engine/keycontrol.h"
 
@@ -38,6 +39,8 @@ KeyControl::KeyControl(const char* pGroup,
             this, SLOT(slotSetEngineKey(double)),
             Qt::DirectConnection);
 
+    m_pEngineKeyDistance = new ControlObject(ConfigKey(pGroup, "visual_key_distance"));
+
     m_pRateSlider = ControlObject::getControl(ConfigKey(pGroup, "rate"));
     connect(m_pRateSlider, SIGNAL(valueChanged(double)),
             this, SLOT(slotRateChanged()),
@@ -141,10 +144,11 @@ void KeyControl::slotFileKeyChanged(double value) {
         pitch_adjust += KeyUtils::powerOf2ToOctaveChange(m_dOldRate);
     }
 
-    mixxx::track::io::key::ChromaticKey adjusted =
+    QPair<mixxx::track::io::key::ChromaticKey, int> adjusted =
             KeyUtils::scaleKeyOctaves(key, pitch_adjust);
 
-    m_pEngineKey->set(KeyUtils::keyToNumericValue(adjusted));
+    m_pEngineKey->set(KeyUtils::keyToNumericValue(adjusted.first));
+    m_pEngineKeyDistance->set(adjusted.second);
 }
 
 void KeyControl::slotSetEngineKey(double key) {

src/engine/keycontrol.h

@@ -50,6 +50,7 @@ class KeyControl : public EngineControl {
 
     /** The current effective key of the engine */
     ControlObject* m_pEngineKey;
+    ControlObject* m_pEngineKeyDistance;
 
     TrackPointer m_pTrack;
 };

src/skin/legacyskinparser.cpp

@@ -440,7 +440,7 @@ QList<QWidget*> LegacySkinParser::parseNode(QDomElement node) {
     } else if (nodeName == "Library") {
         result = wrapWidget(parseLibrary(node));
     } else if (nodeName == "Key") {
-        result = wrapWidget(parseLabelWidget<WKey>(node));
+        result = wrapWidget(parseEngineKey(node));
     } else if (nodeName == "SetVariable") {
         m_pContext->updateVariable(node);
     } else if (nodeName == "Template") {
@@ -882,6 +882,14 @@ QWidget* LegacySkinParser::parseNumberPos(QDomElement node) {
     return p;
 }
 
+QWidget* LegacySkinParser::parseEngineKey(QDomElement node) {
+    QString channelStr = lookupNodeGroup(node);
+    const char* pSafeChannelStr = safeChannelString(channelStr);
+    WKey* pEngineKey = new WKey(pSafeChannelStr, m_pParent);
+    setupLabelWidget(node, pEngineKey);
+    return pEngineKey;
+}
+
 QWidget* LegacySkinParser::parseSpinny(QDomElement node) {
     QString channelStr = lookupNodeGroup(node);
     const char* pSafeChannelStr = safeChannelString(channelStr);

src/skin/legacyskinparser.h

@@ -69,6 +69,7 @@ class LegacySkinParser : public QObject, public SkinParser {
     QWidget* parseTrackProperty(QDomElement node);
     QWidget* parseNumberRate(QDomElement node);
     QWidget* parseNumberPos(QDomElement node);
+    QWidget* parseEngineKey(QDomElement node);
 
     // Legacy pre-1.12.0 skin support.
     QWidget* parseBackground(QDomElement node, QWidget* pOuterWidget, QWidget* pInnerWidget);

src/skin/tooltips.cpp

@@ -1,4 +1,3 @@
-
 #include "skin/tooltips.h"
 
 Tooltips::Tooltips() {

src/track/keyutils.cpp

@@ -1,7 +1,8 @@
 #include <QtDebug>
 #include <QMap>
-#include <QRegExp>
 #include <QMutexLocker>
+#include <QPair>
+#include <QRegExp>
 
 #include "track/keyutils.h"
 #include "mathstuff.h"
@@ -233,15 +234,17 @@ double KeyUtils::keyToNumericValue(ChromaticKey key) {
 }
 
 // static
-ChromaticKey KeyUtils::scaleKeyOctaves(ChromaticKey key, double octave_change) {
+QPair<ChromaticKey, int> KeyUtils::scaleKeyOctaves(ChromaticKey key, double octave_change) {
     // Convert the octave_change from percentage of octave to the nearest
     // integer of key changes. We need the rounding to be in the same direction
     // so that a -1.0 and 1.0 scale of C makes it back to C.
     double key_changes_scaled = octave_change * 12;
-    int key_changes = int(key_changes_scaled +
+    int key_changes = static_cast<int>(key_changes_scaled +
                           (key_changes_scaled > 0 ? 0.5 : -0.5));
 
-    return scaleKeySteps(key, key_changes);
+    // Distance to the nearest key in cents
+    int cents = static_cast<int>((key_changes_scaled - key_changes) * 100);
+    return QPair<ChromaticKey, int>(scaleKeySteps(key, key_changes), cents);
 }
 
 // static

src/track/keyutils.h

@@ -46,7 +46,7 @@ class KeyUtils {
 
     static double keyToNumericValue(mixxx::track::io::key::ChromaticKey key);
 
-    static mixxx::track::io::key::ChromaticKey scaleKeyOctaves(
+    static QPair<mixxx::track::io::key::ChromaticKey, int> scaleKeyOctaves(
         mixxx::track::io::key::ChromaticKey key, double scale);
 
     static mixxx::track::io::key::ChromaticKey scaleKeySteps(

src/widget/wkey.cpp

@@ -3,27 +3,30 @@
 #include "track/keyutils.h"
 #include "track/keys.h"
 
-WKey::WKey(QWidget* pParent)
+WKey::WKey(const char* group, QWidget* pParent)
         : WLabel(pParent),
           m_dOldValue(0),
-          m_preferencesUpdated(ConfigKey("[Preferences]", "updated")) {
+          m_preferencesUpdated(ConfigKey("[Preferences]", "updated")),
+          m_engineKeyDistance(ConfigKey(group, "visual_key_distance")) {
     setValue(m_dOldValue);
     connect(&m_preferencesUpdated, SIGNAL(valueChanged(double)),
             this, SLOT(preferencesUpdated(double)));
+    connect(&m_engineKeyDistance, SIGNAL(valueChanged(double)),
+            this, SLOT(setCents()));
 }
 
 WKey::~WKey() {
 }
 
 void WKey::onConnectedControlValueChanged(double v) {
     setValue(v);
+    setCents();
 }
 
 void WKey::setValue(double dValue) {
     m_dOldValue = dValue;
     mixxx::track::io::key::ChromaticKey key =
             KeyUtils::keyFromNumericValue(dValue);
-
     if (key != mixxx::track::io::key::INVALID) {
         // Render this key with the user-provided notation.
         setText(KeyUtils::keyToString(key));
@@ -32,6 +35,23 @@ void WKey::setValue(double dValue) {
     }
 }
 
+void WKey::setCents() {
+    int diff_cents = m_engineKeyDistance.get();
+    char sign;
+    if (diff_cents < 0) {
+        sign = ' ';
+    } else {
+        sign = '+';
+    }
+    // Remove the previous cent difference
+    QString old = text();
+    if (old.contains('(')) {
+        old = old.section('(', 0, 0);
+    }
+    setText(old + QString("(%1%2 c)").arg(sign).arg(diff_cents));
+}
+
+
 void WKey::preferencesUpdated(double dValue) {
     if (dValue > 0) {
         setValue(m_dOldValue);

src/widget/wkey.h

@@ -9,18 +9,20 @@
 class WKey : public WLabel  {
     Q_OBJECT
   public:
-    WKey(QWidget* pParent=NULL);
+    WKey(const char* group, QWidget* pParent=NULL);
     virtual ~WKey();
 
     virtual void onConnectedControlValueChanged(double v);
 
   private slots:
     void setValue(double dValue);
     void preferencesUpdated(double dValue);
+    void setCents();
 
   private:
     double m_dOldValue;
     ControlObjectThread m_preferencesUpdated;
+    ControlObjectThread m_engineKeyDistance;
 };
 
 #endif /* WKEY_H */