ubuntu-download-manager

Code review comment for lp:~michael-sheldon/ubuntu-download-manager/snap

Revision history for this message

Lukáš Tinkl (lukas-kde) wrote on 2016-11-29:

+ if (QDir::homePath().contains(QString("snap%1ubuntu-download-manager").arg(QDir::separator()))) {

The usage of QDir::separator() is really discouraged, you could as well write a much simpler version:

+ if (QDir::homePath().contains(QStringLiteral("snap/ubuntu-download-manager"))) {

review: Needs Fixing