Fix review comments

master
Simon Stürz 2020-01-16 11:32:22 +01:00
parent bd03cbf6db
commit fb719a6ba6
2 changed files with 5 additions and 4 deletions

View File

@ -1,4 +1,5 @@
# usbrelay
--------------------------------
Description of the plugin...
This plugin eneblas you to control low-cost USB HID relays. The plugin creates a power device for each relay and can be controlled using the `power?? interface.
Examples of USB relay hardware can be found [here](http://vusb.wikidot.com/project:driver-less-usb-relays-hid-interface).

View File

@ -90,7 +90,7 @@ void DevicePluginUsbRelay::deviceRemoved(Device *device)
UsbRelay *relay = m_relayDevices.key(device);
if (!relay) return;
m_relayDevices.remove(relay);
relay->deleteLater();
delete relay;
}
}
@ -189,7 +189,7 @@ void DevicePluginUsbRelay::setupDevice(DeviceSetupInfo *info)
UsbRelay *relay = new UsbRelay(path, relayCount, this);
m_relayDevices.insert(relay, device);
connect(relay, &UsbRelay::connectedChanged, [this, device, relay](bool connected) {
connect(relay, &UsbRelay::connectedChanged, this, [this, device, relay](bool connected) {
qCDebug(dcUsbRelay()) << "Device" << device->name() << (connected ? "connected" : "disconnected");
device->setStateValue(usbRelayConnectorConnectedStateTypeId, connected);
@ -204,7 +204,7 @@ void DevicePluginUsbRelay::setupDevice(DeviceSetupInfo *info)
}
});
connect(relay, &UsbRelay::relayPowerChanged, [this, device](int relayNumber, bool power) {
connect(relay, &UsbRelay::relayPowerChanged, this, [this, device](int relayNumber, bool power) {
Device *relayDevice = getRelayDevice(device, relayNumber);
if (!relayDevice) {
// Note: probably not set up yet