hotplug: emit a plugin configuration change only when actually needed

For now a patch reconnection always succeed, to avoid that annoying error popup

engine/src/inputoutputmap.cpp

@@ -676,15 +676,15 @@ void InputOutputMap::slotPluginConfigurationChanged(QLCIOPlugin* plugin)
         if (op != NULL && op->plugin() == plugin)
         {
             QMutexLocker locker(&m_universeMutex);
-            success = op->reconnect();
+            /*success = */ op->reconnect();
         }
 
         InputPatch* ip = m_universeArray.at(i)->inputPatch();
 
         if (ip != NULL && ip->plugin() == plugin)
         {
             QMutexLocker locker(&m_universeMutex);
-            success = ip->reconnect();
+            /*success = */ ip->reconnect();
         }
     }
 

plugins/dmxusb/src/dmxusb.cpp

@@ -57,6 +57,7 @@ int DMXUSB::capabilities() const
 
 bool DMXUSB::rescanWidgets()
 {
+    int linesCount = m_inputs.count() + m_outputs.count();
     m_inputs.clear();
     m_outputs.clear();
 
@@ -74,6 +75,9 @@ bool DMXUSB::rescanWidgets()
             m_inputs.append(widget);
     }
 
+    if (m_inputs.count() + m_outputs.count() != linesCount)
+        emit configurationChanged();
+
     return true;
 }
 
@@ -310,7 +314,6 @@ void DMXUSB::slotDeviceAdded(uint vid, uint pid)
     }
 
     rescanWidgets();
-    emit configurationChanged();
 }
 
 void DMXUSB::slotDeviceRemoved(uint vid, uint pid)

plugins/hid/hidplugin.cpp

@@ -245,6 +245,7 @@ void HIDPlugin::rescanDevices()
     /* Treat all devices as dead first, until we find them again. Those
        that aren't found, get destroyed at the end of this function. */
     QList <HIDDevice*> destroyList(m_devices);
+    int devCount = m_devices.count();
 
     struct hid_device_info *devs, *cur_dev;
     quint32 line = 0;
@@ -306,7 +307,8 @@ void HIDPlugin::rescanDevices()
         delete dev;
     }
 
-    emit configurationChanged();
+    if (m_devices.count() != devCount)
+        emit configurationChanged();
 }
 
 HIDDevice* HIDPlugin::device(const QString& path)