diff options
author | hjk <hjk@qt.io> | 2019-07-23 12:03:27 +0200 |
---|---|---|
committer | hjk <hjk@qt.io> | 2019-07-30 09:34:36 +0200 |
commit | 4fa0b415237040d806758ab5374055ab2c32d84c (patch) | |
tree | 7e1ce49a0f5e2d0e264cb21064c34ef7dc75ab85 /tests/auto/tools | |
parent | 31753adebe0f19b90f332e81e1a9b063b40f982d (diff) | |
download | qtbase-4fa0b415237040d806758ab5374055ab2c32d84c.tar.gz |
uic: Avoid use of Q_UNUSED in the generated code
Instead, use (void)x; directly.
The current use of Q_UNUSED(x); generates warnings for an
empty statement the expansion of Q_UNUSED contains a semicolon
already.
Emitting Q_UNUSED(x) without the extra semicolon would be
an option, too, but as the future of Q_UNUSED's embedded
semicolon seems unclear right now, avoid its use altogether.
The change affects only generated code that's barely ever
read by a human, so the overall utility of "improved readability"
of Q_UNUSED in that place is questionable anyway.
Change-Id: I332527ed7c202f779bd82290517837e3ecf09a08
Reviewed-by: Jarek Kobus <jaroslaw.kobus@qt.io>
Diffstat (limited to 'tests/auto/tools')
-rw-r--r-- | tests/auto/tools/uic/baseline/qfiledialog.ui.h | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/tests/auto/tools/uic/baseline/qfiledialog.ui.h b/tests/auto/tools/uic/baseline/qfiledialog.ui.h index 9e51ee6a7c..52f8d25c75 100644 --- a/tests/auto/tools/uic/baseline/qfiledialog.ui.h +++ b/tests/auto/tools/uic/baseline/qfiledialog.ui.h @@ -291,7 +291,7 @@ public: detailModeButton->setToolTip(QCoreApplication::translate("QFileDialog", "Detail View", nullptr)); #endif // QT_CONFIG(tooltip) fileTypeLabel->setText(QCoreApplication::translate("QFileDialog", "Files of type:", nullptr)); - Q_UNUSED(QFileDialog); + (void)QFileDialog; } // retranslateUi }; |