Would it not make sense to have an early return?
It seems that you are going to create a new launcher icon even if you are not going to show the launcher. Is this what you expect?
Also, (!path.empty() && !path.compare("software-center-agent")) is not very readable... how about (path == "software-centre=agent")
« Back to merge proposal
Would it not make sense to have an early return?
It seems that you are going to create a new launcher icon even if you are not going to show the launcher.
Is this what you expect?
Also, "software- center- agent") ) centre= agent")
(!path.empty() && !path.compare(
is not very readable... how about
(path == "software-