frodovdr/hyperion.ng
1
0
Fork 0
mirror of https://github.com/hyperion-project/hyperion.ng.git synced 2025-03-01 10:33:28 +00:00
Code Issues Packages Projects Releases Wiki Activity

fix: Nanoleaf - Udp Network init was missing (#698)

Browse Source 
* Fix Nanoleaf - Udp Network init was missing

* ProviderUdp - Add init error handling
This commit is contained in:
LordGrey
2020-02-23 23:18:34 +01:00
committed by GitHub
parent a2dbbcdd0d
commit f955af29fd
3 changed files with 27 additions and 16 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
libsrc/leddevice/LedDevice.cpp
View File

@@ -166,7 +166,7 @@ int LedDevice::updateLeds(const std::vector<ColorRgb>& ledValues)
int retval = 0; int retval = 0;
if ( !_deviceReady || _deviceInError) if ( !_deviceReady || _deviceInError)
{ {
std::cout << "LedDevice::updateLeds(), LedDevice NOT ready!" << std::endl; //std::cout << "LedDevice::updateLeds(), LedDevice NOT ready!" << std::endl;
return -1; return -1;
} }
else else

15
libsrc/leddevice/dev_net/LedDeviceNanoleaf.cpp
View File

@@ -256,11 +256,18 @@ int LedDeviceNanoleaf::open()
if ( init(_devConfig) ) if ( init(_devConfig) )
{ {
if ( initLeds() ) if ( !initNetwork() )
{ {
_deviceReady = true; this->setInError( "UDP Network error!" );
setEnable(true); }
retval = 0; else
{
if ( initLeds() )
{
_deviceReady = true;
setEnable(true);
retval = 0;
}
} }
} }
return retval; return retval;

26
libsrc/leddevice/dev_net/ProviderUdp.cpp
View File

@@ -76,10 +76,21 @@ bool ProviderUdp::init(const QJsonObject &deviceConfig)
bool ProviderUdp::initNetwork() bool ProviderUdp::initNetwork()
{ {
bool isInitOK = true; bool isInitOK = false;
_udpSocket =new QUdpSocket(this);
if ( _udpSocket != nullptr)
{
QHostAddress localAddress = QHostAddress::Any;
quint16 localPort = 0;
if ( !_udpSocket->bind(localAddress, localPort) )
{
Warning ( _log, "Could not bind local address: %s", strerror(errno));
}
isInitOK = true;
}
// TODO: Add Network-Error handling
_udpSocket = new QUdpSocket(this);
return isInitOK; return isInitOK;
} }
@@ -93,17 +104,10 @@ int ProviderUdp::open()
{ {
if ( ! initNetwork()) if ( ! initNetwork())
{ {
this->setInError( "Network error!" ); this->setInError( "UDP Network error!" );
} }
else else
{ {
QHostAddress localAddress = QHostAddress::Any;
quint16 localPort = 0;
if ( !_udpSocket->bind(localAddress, localPort) )
{
Warning ( _log, "Could not bind local address: %s", strerror(errno));
}
// Everything is OK -> enable device // Everything is OK -> enable device
_deviceReady = true; _deviceReady = true;
setEnable(true); setEnable(true);