diff options
| author | Denis Shienkov <denis.shienkov@gmail.com> | 2017-09-27 19:47:38 +0300 |
|---|---|---|
| committer | Denis Shienkov <denis.shienkov@gmail.com> | 2017-10-06 20:17:15 +0000 |
| commit | c513cbaee4d8819a2ebb27396be6c056b1f5856f (patch) | |
| tree | 0a3fe4811ebe62b811a3f4f95ebd8d9602f5af13 /examples/serialport/blockingslave/slavethread.h | |
| parent | 466705128376bd5d60ec23eec7af37d3bf6649fe (diff) | |
| download | qtserialport-c513cbaee4d8819a2ebb27396be6c056b1f5856f.tar.gz | |
Revamp the Blocking Slave Example
* Use the 'const' keywords more.
* Split very long lines.
* Use the 'm_' prefixes for all class members.
* Use the QOverload in signal/slot casting.
* Use more member class initializations.
* Remove QT_USE_NAMESPACE macro.
* Use from(to)Utf8 instead of from(to)Local8Bit.
Task-number: QTBUG-60652
Change-Id: I64bfaf93672dc69f4d10bcaaa0c1c6dd87267d3e
Reviewed-by: André Hartmann <aha_1980@gmx.de>
Reviewed-by: Alex Blasche <alexander.blasche@qt.io>
Diffstat (limited to 'examples/serialport/blockingslave/slavethread.h')
| -rw-r--r-- | examples/serialport/blockingslave/slavethread.h | 15 |
1 files changed, 8 insertions, 7 deletions
diff --git a/examples/serialport/blockingslave/slavethread.h b/examples/serialport/blockingslave/slavethread.h index 9a15e24..737e7cb 100644 --- a/examples/serialport/blockingslave/slavethread.h +++ b/examples/serialport/blockingslave/slavethread.h @@ -51,8 +51,8 @@ #ifndef SLAVETHREAD_H #define SLAVETHREAD_H -#include <QThread> #include <QMutex> +#include <QThread> #include <QWaitCondition> //! [0] @@ -65,7 +65,6 @@ public: ~SlaveThread(); void startSlave(const QString &portName, int waitTimeout, const QString &response); - void run() Q_DECL_OVERRIDE; signals: void request(const QString &s); @@ -73,11 +72,13 @@ signals: void timeout(const QString &s); private: - QString portName; - QString response; - int waitTimeout; - QMutex mutex; - bool quit; + void run() override; + + QString m_portName; + QString m_response; + int m_waitTimeout = 0; + QMutex m_mutex; + bool m_quit = false; }; //! [0] |