Changeset View
Changeset View
Standalone View
Standalone View
ksldapp.cpp
Show First 20 Lines • Show All 603 Lines • ▼ Show 20 Line(s) | 601 | if (!m_lockWindow) { | |||
---|---|---|---|---|---|
604 | } | 604 | } | ||
605 | if (m_isWayland) { | 605 | if (m_isWayland) { | ||
606 | m_lockWindow = new WaylandLocker(m_waylandDisplay); | 606 | m_lockWindow = new WaylandLocker(m_waylandDisplay); | ||
607 | } | 607 | } | ||
608 | if (!m_lockWindow) { | 608 | if (!m_lockWindow) { | ||
609 | return; | 609 | return; | ||
610 | } | 610 | } | ||
611 | m_lockWindow->setGlobalAccel(m_globalAccel); | 611 | m_lockWindow->setGlobalAccel(m_globalAccel); | ||
612 | connect(m_lockWindow, &AbstractLocker::userActivity, this, | 612 | connect(m_lockWindow, &AbstractLocker::userActivity, m_lockWindow, | ||
ivan: This is equivalent to connect(m_lockWindow, signal-ptr, lambda) - adding m_lockWindows as the… | |||||
no it's not the same. It's using Qt::QueuedConnection instead of Qt::AutoConnection. graesslin: no it's not the same. It's using Qt::QueuedConnection instead of Qt::AutoConnection. | |||||
613 | [this]() { | 613 | [this]() { | ||
614 | if (isGraceTime()) { | 614 | if (isGraceTime()) { | ||
615 | unlock(); | 615 | unlock(); | ||
616 | } | 616 | } | ||
617 | }, | 617 | }, | ||
618 | Qt::QueuedConnection | 618 | Qt::QueuedConnection | ||
619 | ); | 619 | ); | ||
620 | connect(m_lockWindow, &AbstractLocker::lockWindowShown, this, &KSldApp::lockScreenShown, Qt::QueuedConnection); | 620 | connect(m_lockWindow, &AbstractLocker::lockWindowShown, this, &KSldApp::lockScreenShown, Qt::QueuedConnection); | ||
▲ Show 20 Lines • Show All 102 Lines • Show Last 20 Lines |
This is equivalent to connect(m_lockWindow, signal-ptr, lambda) - adding m_lockWindows as the third argument basically lies that we are using the 'safe' version of connect.