Minor: added debug output in error case

src/com.cpp

@@ -47,7 +47,7 @@ void T_com::writeToSerial(const QByteArray &data, uint16_t sendLength)
             int bytesToWrite = buffer.size();
             while (bytesToWrite > 0 && !buffer.isEmpty()) {
                 int bytesWritten = CatSerial->write(buffer);
-                if (bytesWritten != -1 && CatSerial->waitForBytesWritten(1000)) {
+                if (bytesWritten != -1) {
                     bytesToWrite -= bytesWritten;
                     buffer = buffer.right(bytesWritten);
                 } else {
@@ -67,13 +67,14 @@ void T_com::writeToSerial(const QByteArray &data, uint16_t sendLength)
                         << "ERROR sending" << data.toHex(':') << "port is not open";
         }
     } else {
-        qCritical() << __func__ << "" << __LINE__ << "ERROR detected active read cmd" << readCmds[0];
+        qCritical() << __func__ << "" << __LINE__ << errorCnt << "ERROR about to send cmd" << QString("0x%1").arg((unsigned char)data.constData()[2], 0, 16);
+        qCritical() << __func__ << "" << __LINE__ << errorCnt << "ERROR detected active read cmds" << readCmds;
         if (CatSerial->isOpen()) {
             int availableBytes = CatSerial->bytesAvailable();
             qCritical() << __func__ << "" << __LINE__ << "ERROR available bytes" << availableBytes;
             if (availableBytes == 0) {
                 errorCnt += 1;
-                if (errorCnt > 100) {
+                if (errorCnt > 20) {
                     readCmds.clear();
                     errorCnt = 0;
                 }

src/datIf.cpp

@@ -556,23 +556,27 @@ char T_datif::loadRecDataFromFrame()
             if (readSource == myDCIF->getReadSource() && readSource == readCmds[0]) {
                 // maybe we have sent a explicit request for a test-response
                 if (waitForTestResponse) {
+                    qCritical() << __func__ << ":" << __LINE__ << "turn on auto-request";
                     if (readCmds[0] == CMD2DC_TestSerial) {
                         if (QString(QByteArray((char const *)receivedData, RdDleng)) == "< SlaveResponse") {
                             waitForTestResponse = false;
                             qCritical() << __func__ << ":" << __LINE__ << "turn on auto-request";
                             ((hwapi *)parent())->dc_autoRequest(true); // return autorequest to true
                         } else {
-                            qCritical() << __func__ << ":" << __LINE__ << "received wrong test-response"
+                            qCritical() << __func__ << ":" << __LINE__ << "ERROR received wrong test-response"
                                         << QString(QByteArray((char const *)receivedData, RdDleng));
                         }
                     }
                 } else {
                     // usual handling of response
+                    // qCritical() << __func__ << ":" << __LINE__ << ":" << readSource << myDCIF->getReadSource() << readCmds[0];
+                    // qCritical() << __func__ << ":" << __LINE__ << ":" << QByteArray((char const *)receivedData, RdDleng).toHex(':');
                 }
                 readCmds.clear();
             } else { // error
-                qCritical() << __func__ << ":" << __LINE__ << ":" << readSource << myDCIF->getReadSource() << readCmds[0];
-                qCritical() << __func__ << ":" << __LINE__ << ":" << QByteArray((char const *)receivedData, RdDleng).toHex(':');
+                qCritical() << __func__ << ":" << __LINE__ << ": ERROR " << readSource << myDCIF->getReadSource() << readCmds[0];
+                qCritical() << __func__ << ":" << __LINE__ << ": ERROR length" << RdDleng << QString(", ignore data for cmd 0x%1").arg(readCmds[0], 0, 16)
+                            << QByteArray((char const *)receivedData, RdDleng).toHex(':');
                 readCmds.clear();
                 return 0;
             }
@@ -583,8 +587,8 @@ char T_datif::loadRecDataFromFrame()
             // again.
             readCmds.clear();
             if (parent()) {
-                qCritical() << __func__ << ":" << __LINE__ << "turn off auto-request";
-                qCritical() << __func__ << ":" << __LINE__ << "send request for test-response";
+                qCritical() << __func__ << ":" << __LINE__ << "ERROR turn off auto-request";
+                qCritical() << __func__ << ":" << __LINE__ << "ERROR send request for test-response";
                 waitForTestResponse = true;
                 ((hwapi *)parent())->dc_autoRequest(false);
                 ((hwapi *)parent())->dc_requTestResponse();