Merged
Merge Request #7
·
created by 
Fix/pgroen/error handling improvement
Added logging and sensible return points if a problem occurs. After merge :
- it will be tagged as version 0.9.4
- it will be integrated into the Main IPZC SDK ( Aarch64 ) @ Priva
From
fix/pgroen/error-handling-improvement
into
master
2 participants
-
the 'test' directory does not seem to hold any substance. I'd say it can be removed for now.
-
is there any reason for using the for(;;) construction over while(true) besides compiler satisfaction?
-
It is in essence the same as a while( true ) -loop. As a for-clause has three elements :
for (initialization; condition; update) { // body of-loop }By using it this way, it will just create an infinite loop There is not much difference as a while loop. ( compiled, they produce the exact same hex-pattern
-
can be merged after comments are resolved.
-
mentioned in commit a19279232f12b3d0c946d46510c823b9eddb1a41
-
Status changed to merged
started a discussion
on the diff
src/mqttclient.cpp
| 531 | 543 | |
| 532 | 544 | void MqttClient::eventHandler() |
| 533 | 545 | { |
| 534 | - // InfoLogToFIle ("MqttClient", "%1 - starting event handler", m_clientId); | |
| 535 | - for (;;) { | |
| 546 | + LogInfo("[MqttClient::eventHandler]", std::string( m_clientId + " - starting event handler." ) ); | |
| 547 | + for (;;) | |
| 2 |
|
|