Skip to content
GitLab
Menu
Projects
Groups
Snippets
/
Help
Help
Support
Community forum
Keyboard shortcuts
?
Submit feedback
Contribute to GitLab
Sign in
Toggle navigation
Menu
Open sidebar
Plasma
Discover
Commits
0d728583
Commit
0d728583
authored
Apr 09, 2021
by
Aleix Pol Gonzalez
🐧
Browse files
pk: small code simplifcation
Don't connect to a signal from the same object that we just emit once.
parent
5b8d1ba4
Changes
1
Hide whitespace changes
Inline
Side-by-side
libdiscover/backends/PackageKitBackend/PackageKitResource.cpp
View file @
0d728583
...
@@ -31,10 +31,6 @@ PackageKitResource::PackageKitResource(QString packageName, QString summary, Pac
...
@@ -31,10 +31,6 @@ PackageKitResource::PackageKitResource(QString packageName, QString summary, Pac
,
m_name
(
std
::
move
(
packageName
))
,
m_name
(
std
::
move
(
packageName
))
{
{
setObjectName
(
m_name
);
setObjectName
(
m_name
);
connect
(
this
,
&
PackageKitResource
::
dependenciesFound
,
this
,
[
this
](
const
QJsonObject
&
obj
)
{
setDependenciesCount
(
obj
.
size
());
});
}
}
QString
PackageKitResource
::
name
()
const
QString
PackageKitResource
::
name
()
const
...
@@ -361,6 +357,7 @@ void PackageKitResource::fetchDependencies()
...
@@ -361,6 +357,7 @@ void PackageKitResource::fetchDependencies()
});
});
connect
(
trans
,
&
PackageKit
::
Transaction
::
finished
,
this
,
[
this
,
packageDependencies
](
PackageKit
::
Transaction
::
Exit
/*status*/
)
{
connect
(
trans
,
&
PackageKit
::
Transaction
::
finished
,
this
,
[
this
,
packageDependencies
](
PackageKit
::
Transaction
::
Exit
/*status*/
)
{
Q_EMIT
dependenciesFound
(
*
packageDependencies
);
Q_EMIT
dependenciesFound
(
*
packageDependencies
);
setDependenciesCount
(
packageDependencies
->
size
());
});
});
}
}
...
...
Write
Preview
Supports
Markdown
0%
Try again
or
attach a new file
.
Attach a file
Cancel
You are about to add
0
people
to the discussion. Proceed with caution.
Finish editing this message first!
Cancel
Please
register
or
sign in
to comment