diff options
| author | Marc Mutz <marc.mutz@kdab.com> | 2017-03-19 12:42:41 +0100 |
|---|---|---|
| committer | Marc Mutz <marc.mutz@kdab.com> | 2017-03-22 09:22:48 +0000 |
| commit | 287c43c9c35e900dbf8e820c60d0b71cbe9099ef (patch) | |
| tree | 5ee22dee1f253e46d26d862eda4ab5ed034a30f9 /examples/serialbus | |
| parent | a25755509faaedab51fdc3800b6b464d9aa78da0 (diff) | |
modbus master example: fix nonsensical code
Pointed out by GCC 7:
writeregistermodel.cpp:117:48: warning: enum constant in boolean context [-Wint-in-bool-context]
if (index.column() == HoldingColumn && Qt::EditRole) { // holding registers
^~~~~~~~
Change-Id: I5d0f19c0480b027109c6a751f69852114cb634b2
Reviewed-by: Denis Shienkov <denis.shienkov@gmail.com>
Reviewed-by: David Faure <david.faure@kdab.com>
Diffstat (limited to 'examples/serialbus')
| -rw-r--r-- | examples/serialbus/modbus/master/writeregistermodel.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/examples/serialbus/modbus/master/writeregistermodel.cpp b/examples/serialbus/modbus/master/writeregistermodel.cpp index a664482..60684c4 100644 --- a/examples/serialbus/modbus/master/writeregistermodel.cpp +++ b/examples/serialbus/modbus/master/writeregistermodel.cpp @@ -114,7 +114,7 @@ bool WriteRegisterModel::setData(const QModelIndex &index, const QVariant &value return true; } - if (index.column() == HoldingColumn && Qt::EditRole) { // holding registers + if (index.column() == HoldingColumn && role == Qt::EditRole) { // holding registers bool result = false; quint16 newValue = value.toString().toUShort(&result, 16); if (result) |