blumia/dde-application-manager
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix: should't move singelSlot

Browse Source 
add some comment

Signed-off-by: ComixHe <heyuming@deepin.org>
This commit is contained in:
ComixHe 2023-09-19 13:26:08 +08:00 committed by Comix
parent efb1212945
commit 2b5fc291d4
1 changed files with 3 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
src/dbus/applicationmanager1service.cpp
View File

@ -112,7 +112,7 @@ void ApplicationManager1Service::initService(QDBusConnection &connection) noexce
watcher->deleteLater();
};
*sigCon = connect(watcher, &QDBusServiceWatcher::serviceRegistered, std::move(singleSlot));
*sigCon = connect(watcher, &QDBusServiceWatcher::serviceRegistered, singleSlot);
auto msg =
QDBusMessage::createMethodCall("org.freedesktop.DBus", "/org/freedesktop/DBus", "org.freedesktop.DBus", "NameHasOwner");
@ -121,7 +121,8 @@ void ApplicationManager1Service::initService(QDBusConnection &connection) noexce
auto reply = QDBusConnection::sessionBus().call(msg);
if (reply.type() != QDBusMessage::ReplyMessage) {
qWarning() << "call org.freedesktop.DBus::NameHasOwner failed, skip autostart:" << reply.errorMessage();
// ...
// The connection should not be deleted, failure to call org.freedesktop.DBus::NameHasOwner does not mean that the
// XSettings service is invalid.
return;
}