Merge branch '2.3' of git@github.com:mixxxdj/mixxx.git

4 files changed, 14 insertions, 9 deletions

diff --git a/src/control/control.cpp b/src/control/control.cpp
index 99f72ce37f..9b0ae2f9f9 100644
--- a/src/control/control.cpp
+++ b/src/control/control.cpp
@@ -101,9 +101,9 @@ QSharedPointer<ControlDoublePrivate> ControlDoublePrivate::getControl(
         bool bTrack,
         bool bPersist,
         double defaultValue) {
-    VERIFY_OR_DEBUG_ASSERT(!key.isEmpty()) {
-        qWarning() << "ControlDoublePrivate::getControl returning NULL"
-                   << "for empty ConfigKey.";
+    VERIFY_OR_DEBUG_ASSERT(key.isValid()) {
+        qWarning() << "ControlDoublePrivate::getControl returning nullptr"
+                   << "for invalid ConfigKey" << key;
         return nullptr;
     }
 
diff --git a/src/control/control.h b/src/control/control.h
index 5ff49bcd4b..fb7c5abfc3 100644
--- a/src/control/control.h
+++ b/src/control/control.h
@@ -15,8 +15,13 @@ class ControlObject;
 
 enum class ControlFlag {
     None = 0,
-    AllowEmptyKey = 1,
+    /// Do not throw an assertion if the key is invalid. Needed for controller
+    /// mappings and skins.
+    AllowInvalidKey = 1,
+    /// Don't throw an assertion when trying to access a non-existing CO.
+    /// Needed for controller mappings and skins.
     NoAssertIfMissing = 1 << 1,
+    /// Don't log a warning when trying to access a non-existing CO.
     NoWarnIfMissing = (1 << 2) | NoAssertIfMissing,
 };
 
diff --git a/src/control/controlobject.cpp b/src/control/controlobject.cpp
index 988681cc50..543046e576 100644
--- a/src/control/controlobject.cpp
+++ b/src/control/controlobject.cpp
@@ -31,8 +31,8 @@ ControlObject::ControlObject() {
 ControlObject::ControlObject(ConfigKey key, bool bIgnoreNops, bool bTrack,
                              bool bPersist, double defaultValue)
         : m_key(key) {
-    // Don't bother looking up the control if key is NULL. Prevents log spew.
-    if (!m_key.isNull()) {
+    // Don't bother looking up the control if key is invalid. Prevents log spew.
+    if (m_key.isValid()) {
         m_pControl = ControlDoublePrivate::getControl(m_key,
                 ControlFlag::None,
                 this,
diff --git a/src/control/controlproxy.cpp b/src/control/controlproxy.cpp
index f0b93961a5..5ca1f04630 100644
--- a/src/control/controlproxy.cpp
+++ b/src/control/controlproxy.cpp
@@ -14,10 +14,10 @@ ControlProxy::ControlProxy(const char* g, const char* i, QObject* pParent, Contr
 ControlProxy::ControlProxy(const ConfigKey& key, QObject* pParent, ControlFlags flags)
         : QObject(pParent),
           m_pControl(nullptr) {
-    DEBUG_ASSERT(!key.isNull() || flags.testFlag(ControlFlag::AllowEmptyKey));
+    DEBUG_ASSERT(key.isValid() || flags.testFlag(ControlFlag::AllowInvalidKey));
     m_key = key;
 
-    if (!m_key.isNull()) {
+    if (m_key.isValid()) {
         initialize(flags);
     }
 }
@@ -27,7 +27,7 @@ void ControlProxy::initialize(ControlFlags flags) {
     DEBUG_ASSERT(!m_pControl);
 
     // Prevent empty keys
-    VERIFY_OR_DEBUG_ASSERT(!m_key.isNull() || flags.testFlag(ControlFlag::AllowEmptyKey)) {
+    VERIFY_OR_DEBUG_ASSERT(m_key.isValid() || flags.testFlag(ControlFlag::AllowInvalidKey)) {
         return;
     }