diff options
author | Alex Trotsenko <alex1973tr@gmail.com> | 2020-12-17 18:28:54 +0200 |
---|---|---|
committer | Alex Trotsenko <alex1973tr@gmail.com> | 2020-12-17 21:33:17 +0200 |
commit | 7a4e68624094216844134fe86a167444b25d2894 (patch) | |
tree | 555bbdaff40c57382392af6a5dcd2b266e907255 /tests/auto/corelib/kernel/qsocketnotifier | |
parent | 1649137b9311a5ee3d12bc8824f5453294bd6fe9 (diff) |
QSocketNotifier: hone setSocket() API
Remove a second argument to the setSocket(qintptr, bool) function as
it makes the API harder to understand.
Change-Id: Ib1852a4e9d96adde35bfbf0fe03b386d9ded395a
Reviewed-by: Oswald Buddenhagen <oswald.buddenhagen@gmx.de>
Diffstat (limited to 'tests/auto/corelib/kernel/qsocketnotifier')
-rw-r--r-- | tests/auto/corelib/kernel/qsocketnotifier/tst_qsocketnotifier.cpp | 10 |
1 files changed, 5 insertions, 5 deletions
diff --git a/tests/auto/corelib/kernel/qsocketnotifier/tst_qsocketnotifier.cpp b/tests/auto/corelib/kernel/qsocketnotifier/tst_qsocketnotifier.cpp index 79ee7da5b6..13ec6a7982 100644 --- a/tests/auto/corelib/kernel/qsocketnotifier/tst_qsocketnotifier.cpp +++ b/tests/auto/corelib/kernel/qsocketnotifier/tst_qsocketnotifier.cpp @@ -102,12 +102,12 @@ void tst_QSocketNotifier::constructing() notifier.setEnabled(true); QVERIFY(!notifier.isEnabled()); - notifier.setSocket(fd, true); + notifier.setSocket(fd); QVERIFY(notifier.isValid()); QCOMPARE(notifier.socket(), fd); - QVERIFY(notifier.isEnabled()); - notifier.setEnabled(false); QVERIFY(!notifier.isEnabled()); + notifier.setEnabled(true); + QVERIFY(notifier.isEnabled()); } // Test constructing with the notifications enabled by default. @@ -119,12 +119,12 @@ void tst_QSocketNotifier::constructing() QCOMPARE(notifier.type(), QSocketNotifier::Write); QVERIFY(notifier.isEnabled()); - notifier.setSocket(fd, false); + notifier.setSocket(fd); QVERIFY(!notifier.isEnabled()); notifier.setEnabled(true); QVERIFY(notifier.isEnabled()); - notifier.setSocket(-1, true); + notifier.setSocket(-1); QVERIFY(!notifier.isValid()); QCOMPARE(notifier.socket(), Q_INT64_C(-1)); QVERIFY(!notifier.isEnabled()); |