From 63bc298fb336e285c5abf5946eb790b4ba7df372 Mon Sep 17 00:00:00 2001 From: Mitch Curtis Date: Mon, 17 Sep 2012 17:11:16 +0200 Subject: [PATCH] Pass correct argument to QShortcutEvent ctor in tst_qabstractbutton.cpp The QShortcutEvent constructor takes "int shortcutId" as its second argument, not a bool. Since the default shortcutId is 0, this test passed, since false == 0. Change-Id: I43bbae4613f3badb1578dccec76dcdd3c96a3a2f Reviewed-by: Marc Mutz --- .../widgets/widgets/qabstractbutton/tst_qabstractbutton.cpp | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/tests/auto/widgets/widgets/qabstractbutton/tst_qabstractbutton.cpp b/tests/auto/widgets/widgets/qabstractbutton/tst_qabstractbutton.cpp index 85cd2a8d92..56578b0336 100644 --- a/tests/auto/widgets/widgets/qabstractbutton/tst_qabstractbutton.cpp +++ b/tests/auto/widgets/widgets/qabstractbutton/tst_qabstractbutton.cpp @@ -544,7 +544,8 @@ void tst_QAbstractButton::shortcutEvents() for (int i = 0; i < 4; ++i) { QKeySequence sequence; - QShortcutEvent event(sequence, false); + // Default shortcutId for QAbstractButton is 0, so the shortcut event will work. + QShortcutEvent event(sequence, /*shortcutId*/ 0); QApplication::sendEvent(&button, &event); if (i < 2) QTest::qWait(500);