Bladeren bron

Widgets: Fix Qt5 "unused-result" build warnings associated with qWaitForWindowActive

This commit fixes warnings like the following:

/path/to/CTK/Libs/Widgets/Testing/Cpp/ctkColorDialogTest1.cpp: In function ‘int ctkColorDialogTest1(int, char**)’:
/path/to/CTK/Libs/Widgets/Testing/Cpp/ctkColorDialogTest1.cpp:94:46: warning: ignoring return value of ‘bool QTest::qWaitForWindowActive(QWidget*, int)’, declared with attribute warn_unused_result [-Wunused-result]
     QTest::qWaitForWindowActive(&colorDialog);
                                              ^
Jean-Christophe Fillion-Robin 7 jaren geleden
bovenliggende
commit
aaea3894e5

+ 2 - 1
Libs/Widgets/Testing/Cpp/ctkColorDialogTest1.cpp

@@ -91,7 +91,8 @@ int ctkColorDialogTest1(int argc, char * argv [] )
   if (argc < 2 || QString(argv[1]) != "-I" )
     {
 #if (QT_VERSION >= 0x50000)
-    QTest::qWaitForWindowActive(&colorDialog);
+    bool result = QTest::qWaitForWindowActive(&colorDialog);
+    Q_UNUSED(result);
 #else
     QTest::qWaitForWindowShown(&colorDialog);
 #endif

+ 2 - 1
Libs/Widgets/Testing/Cpp/ctkConsoleTest.cpp

@@ -45,7 +45,8 @@ void ctkConsoleTester::testShow()
   ctkConsole console;
   console.show();
 #if (QT_VERSION >= 0x50000)
-  QTest::qWaitForWindowActive(&console);
+  bool result = QTest::qWaitForWindowActive(&console);
+  Q_UNUSED(result);
 #else
   QTest::qWaitForWindowShown(&console);
 #endif

+ 2 - 1
Libs/Widgets/Testing/Cpp/ctkDoubleRangeSliderTest.cpp

@@ -46,7 +46,8 @@ void ctkDoubleRangeSliderTester::testUI()
   ctkDoubleRangeSlider slider;
   slider.show();
 #if (QT_VERSION >= 0x50000)
-  QTest::qWaitForWindowActive(&slider);
+  bool result = QTest::qWaitForWindowActive(&slider);
+  Q_UNUSED(result);
 #else
   QTest::qWaitForWindowShown(&slider);
 #endif

+ 2 - 1
Libs/Widgets/Testing/Cpp/ctkDoubleSliderTest.cpp

@@ -46,7 +46,8 @@ void ctkDoubleSliderTester::testUI()
   ctkDoubleSlider slider;
   slider.show();
 #if (QT_VERSION >= 0x50000)
-  QTest::qWaitForWindowActive(&slider);
+  bool result = QTest::qWaitForWindowActive(&slider);
+  Q_UNUSED(result);
 #else
   QTest::qWaitForWindowShown(&slider);
 #endif

+ 2 - 1
Libs/Widgets/Testing/Cpp/ctkDoubleSpinBoxTest.cpp

@@ -85,7 +85,8 @@ void ctkDoubleSpinBoxTester::testUI()
   spinBox.setSetMode(ctkDoubleSpinBox::SetAlways);
   spinBox.show();
 #if (QT_VERSION >= 0x50000)
-  QTest::qWaitForWindowActive(&spinBox);
+  bool result = QTest::qWaitForWindowActive(&spinBox);
+  Q_UNUSED(result);
 #else
   QTest::qWaitForWindowShown(&spinBox);
 #endif

+ 4 - 2
Libs/Widgets/Testing/Cpp/ctkMatrixWidgetTest.cpp

@@ -46,7 +46,8 @@ void ctkMatrixWidgetTester::testUI()
 
   matrix.show();
 #if (QT_VERSION >= 0x50000)
-  QTest::qWaitForWindowActive(&matrix);
+  bool result = QTest::qWaitForWindowActive(&matrix);
+  Q_UNUSED(result);
 #else
   QTest::qWaitForWindowShown(&matrix);
 #endif
@@ -65,7 +66,8 @@ void ctkMatrixWidgetTester::testDecimals()
   matrix.show();
 
 #if (QT_VERSION >= 0x50000)
-  QTest::qWaitForWindowActive(&matrix);
+  bool result = QTest::qWaitForWindowActive(&matrix);
+  Q_UNUSED(result);
 #else
   QTest::qWaitForWindowShown(&matrix);
 #endif

+ 2 - 1
Libs/Widgets/Testing/Cpp/ctkPathListWidgetWithButtonsTest.cpp

@@ -68,7 +68,8 @@ void ctkPathListWidgetWithButtonsTester::testButtons()
 
   topLevel.show();
 #if (QT_VERSION >= 0x50000)
-  QTest::qWaitForWindowActive(&topLevel);
+  bool result = QTest::qWaitForWindowActive(&topLevel);
+  Q_UNUSED(result);
 #else
   QTest::qWaitForWindowShown(&topLevel);
 #endif

+ 2 - 1
Libs/Widgets/Testing/Cpp/ctkPushButtonTest.cpp

@@ -56,7 +56,8 @@ void ctkPushButtonTester::testDefaults()
 
   button.show();
 #if (QT_VERSION >= 0x50000)
-  QTest::qWaitForWindowActive(&button);
+  bool result = QTest::qWaitForWindowActive(&button);
+  Q_UNUSED(result);
 #else
   QTest::qWaitForWindowShown(&button);
 #endif

+ 2 - 1
Libs/Widgets/Testing/Cpp/ctkRangeSliderTest.cpp

@@ -164,7 +164,8 @@ void ctkRangeSliderTester::testHandleMouseEvents()
 
   rangeSlider.show();
 #if (QT_VERSION >= 0x50000)
-  QTest::qWaitForWindowActive(&rangeSlider);
+  bool result = QTest::qWaitForWindowActive(&rangeSlider);
+  Q_UNUSED(result);
 #else
   QTest::qWaitForWindowShown(&rangeSlider);
 #endif

+ 2 - 1
Libs/Widgets/Testing/Cpp/ctkRangeWidgetTest.cpp

@@ -59,7 +59,8 @@ void ctkRangeWidgetTester::testUI()
   ctkRangeWidget rangeWidget;
   rangeWidget.show();
 #if (QT_VERSION >= 0x50000)
-  QTest::qWaitForWindowActive(&rangeWidget);
+  bool result = QTest::qWaitForWindowActive(&rangeWidget);
+  Q_UNUSED(result);
 #else
   QTest::qWaitForWindowShown(&rangeWidget);
 #endif

+ 2 - 1
Libs/Widgets/Testing/Cpp/ctkSearchBoxTest.cpp

@@ -42,7 +42,8 @@ void ctkSearchBoxTester::testSignals()
   ctkSearchBox searchBox;
   searchBox.show();
 #if (QT_VERSION >= 0x50000)
-  QTest::qWaitForWindowActive(&searchBox);
+  bool result = QTest::qWaitForWindowActive(&searchBox);
+  Q_UNUSED(result);
 #else
   QTest::qWaitForWindowShown(&searchBox);
 #endif

+ 6 - 3
Libs/Widgets/Testing/Cpp/ctkSliderWidgetTest.cpp

@@ -67,7 +67,8 @@ void ctkSliderWidgetTester::testUI()
   slider.setPrefix("A: ");
   slider.show();
 #if (QT_VERSION >= 0x50000)
-  QTest::qWaitForWindowActive(&slider);
+  bool result = QTest::qWaitForWindowActive(&slider);
+  Q_UNUSED(result);
 #else
   QTest::qWaitForWindowShown(&slider);
 #endif
@@ -165,7 +166,8 @@ void ctkSliderWidgetTester::testDecimalsByShortcuts()
 
   slider.show();
 #if (QT_VERSION >= 0x50000)
-  QTest::qWaitForWindowActive(&slider);
+  bool result = QTest::qWaitForWindowActive(&slider);
+  Q_UNUSED(result);
 #else
   QTest::qWaitForWindowShown(&slider);
 #endif
@@ -190,7 +192,8 @@ void ctkSliderWidgetTester::testValueChangedWithNoTracking()
 
   slider.show();
 #if (QT_VERSION >= 0x50000)
-  QTest::qWaitForWindowActive(&slider);
+  bool result = QTest::qWaitForWindowActive(&slider);
+  Q_UNUSED(result);
 #else
   QTest::qWaitForWindowShown(&slider);
 #endif