Do not continue waitForReadyRead() if an error occurred

This patch improves workaround implemented in the previous commit
d34fce4a5d12789ded107631e22cb6ef54d35eee

Change-Id: I013b19b47ee343ab00f242d7ee1021b92c07d18f
Reviewed-by: Peter Kümmel <syntheticpp@gmx.net>
Reviewed-by: Sergey Belyashov <Sergey.Belyashov@gmail.com>
Reviewed-by: Denis Shienkov <denis.shienkov@gmail.com>

1 files changed, 3 insertions, 10 deletions

diff --git a/src/serialport/qserialport_unix.cpp b/src/serialport/qserialport_unix.cpp
index f7d2af9..29822d5 100644
--- a/src/serialport/qserialport_unix.cpp
+++ b/src/serialport/qserialport_unix.cpp
@@ -424,7 +424,6 @@ bool QSerialPortPrivate::waitForReadyRead(int msecs)
 
     stopWatch.start();
 
-    error = QSerialPort::NoError;
     do {
         bool readyToRead = false;
         bool readyToWrite = false;
@@ -436,17 +435,11 @@ bool QSerialPortPrivate::waitForReadyRead(int msecs)
             return false;
         }
 
-        if (readyToRead) {
-            if (readNotification())
-                return true;
-        }
-
-        if (readyToWrite)
-            completeAsyncWrite();
+        if (readyToRead)
+            return readNotification();
 
-        if (error != QSerialPort::NoError)
+        if (readyToWrite && !completeAsyncWrite())
             return false;
-
     } while (msecs == -1 || timeoutValue(msecs, stopWatch.elapsed()) > 0);
     return false;
 }