Clazy fixes

diff --git a/import-mylist/main.cpp b/import-mylist/main.cpp
index d0167613fe56f592df93ac52ffdf599d7cc8ee57..2310ccbf4334093ea0ef447dfd6148121ec22661 100644 (file)
--- a/import-mylist/main.cpp
+++ b/import-mylist/main.cpp
@@ -13,7 +13,7 @@ int main(int argc, char *argv[])
        QTextStream cout(stdout);
        if (a.arguments().count() < 2)
        {
-               cout << "Usage: " << a.arguments()[0] << " FILE";
+               cout << "Usage: " << a.arguments().at(0) << " FILE";
                return 1;
        }
 
@@ -25,7 +25,7 @@ int main(int argc, char *argv[])
        }
 
        AbstractTask *t =
-               LocalMyList::instance()->importMyList(QFileInfo{a.arguments()[1]});
+               LocalMyList::instance()->importMyList(QFileInfo{a.arguments().at(1)});
        QObject::connect(t, SIGNAL(finished()), &a, SLOT(quit()));
 
        return a.exec();

diff --git a/import-titles/main.cpp b/import-titles/main.cpp
index 875f2ba246250f49b799e1f1964e9bf29688067d..46af92a78e625ab46b97779debf6cb6927808764 100644 (file)
--- a/import-titles/main.cpp
+++ b/import-titles/main.cpp
@@ -30,7 +30,8 @@ int main(int argc, char *argv[])
        if (a.arguments().count() == 1)
                t = LocalMyList::instance()->importTitlesFromUrl(QUrl::fromUserInput(LocalMyList::instance()->settings()->get("anidbTitleUrl").toString()));
        else
-               t = LocalMyList::instance()->importTitles(QFileInfo{a.arguments()[1]});
+               t = LocalMyList::instance()->importTitles(
+                       QFileInfo{a.arguments().at(1)});
        QObject::connect(t, SIGNAL(finished()), &a, SLOT(quit()));
 
        return a.exec();

diff --git a/localmylist-management/dynamicmodelview.cpp b/localmylist-management/dynamicmodelview.cpp
index b4d05c1ff0c798cce8807cd29122e5c404fb770a..2d21f451753b60cf726b3daf26129e2b4a060d12 100644 (file)
--- a/localmylist-management/dynamicmodelview.cpp
+++ b/localmylist-management/dynamicmodelview.cpp
@@ -194,7 +194,8 @@ void DynamicModelView::markSelectionWatched()
        RaiiTransaction t{MyList::instance()->database()};\r
        t.commit();\r
 \r
-       for (const auto &index : lastSelection) {\r
+       for (const auto &index : std::as_const(lastSelection))\r
+       {\r
                DynamicModel::Node *node = dynamicModelFilterModel()->node(index);\r
                markEpisodeWatched(node);\r
                markFileWatched(node);\r

diff --git a/localmylist-management/main.cpp b/localmylist-management/main.cpp
index ce95907fe37b37127b321d97fcae3a3c22493b46..4321e1e1602d364b6adf744aa38330a5847c84ba 100644 (file)
--- a/localmylist-management/main.cpp
+++ b/localmylist-management/main.cpp
@@ -11,7 +11,7 @@ int main(int argc, char *argv[])
        {
                if (a.arguments().count() < 2)
                        return 0;
-               return a.sendMessage(QString("ADD %1").arg(a.arguments()[1]));
+               return a.sendMessage(QString("ADD %1").arg(a.arguments().at(1)));
        }
 
        MainWindow w;
@@ -20,7 +20,7 @@ int main(int argc, char *argv[])
        w.show();
 
        if (a.arguments().count() >= 2)
-               LocalMyList::instance()->addFile(QFileInfo{a.arguments()[1]});
+               LocalMyList::instance()->addFile(QFileInfo{a.arguments().at(1)});
 
        return a.exec();
 }

diff --git a/localmylist-management/mainwindow.cpp b/localmylist-management/mainwindow.cpp
index a02f21e4a34fbb4623c8fe706dc73cd612a163a1..96d233fe996eb89556aee3b62e06f505279f96eb 100644 (file)
--- a/localmylist-management/mainwindow.cpp
+++ b/localmylist-management/mainwindow.cpp
@@ -313,7 +313,6 @@ void MainWindow::dragEnterEvent(QDragEnterEvent *event)
 void MainWindow::dropEvent(QDropEvent *event)
 {
        QList<QUrl> urls = event->mimeData()->urls();
-       QFileInfoList files;
 
        for (int i = 0; i < urls.count(); ++i)
        {

diff --git a/localmylist-management/settingsdialog.cpp b/localmylist-management/settingsdialog.cpp
index 5fb37faa63b15843363f8365294bc4d8f4f0e326..5c160c5a89ca779d799288cde5674b178a10e720 100644 (file)
--- a/localmylist-management/settingsdialog.cpp
+++ b/localmylist-management/settingsdialog.cpp
@@ -56,7 +56,7 @@ void SettingsDialog::generateSettings()
 
        QFormLayout *layout = new QFormLayout(ui->settingsBox);
 
-       for (const QString &setting : settings)
+       for (const QString &setting : std::as_const(settings))
        {
                QLabel *label = new QLabel(ui->settingsBox);
                QLineEdit *lineEdit = new QLineEdit(ui->settingsBox);

diff --git a/localmylist-management/tabs/mylisttab.cpp b/localmylist-management/tabs/mylisttab.cpp
index 1311ebf0c24450803b5cee69e4bf44dde02769f8..868087e714494ea1b0a08926655f433721897ce4 100644 (file)
--- a/localmylist-management/tabs/mylisttab.cpp
+++ b/localmylist-management/tabs/mylisttab.cpp
@@ -130,7 +130,6 @@ void MyListTab::on_myListView_renameFilesRequested(const QModelIndex &index)
        if (!node->id())
                return;
 
-       QString path;
        QSqlQuery q(MyList::instance()->database()->connection());
 
        QChar typeLetter;

diff --git a/localmylist-management/tabs/pathmappingtab.cpp b/localmylist-management/tabs/pathmappingtab.cpp
index bea1a12b75dbdfd13f734803c4d093d0011d54c2..b656792dac5711126cc3cb2a9ffdc2f60d404441 100644 (file)
--- a/localmylist-management/tabs/pathmappingtab.cpp
+++ b/localmylist-management/tabs/pathmappingtab.cpp
@@ -97,7 +97,8 @@ void PathMappingTab::removeSelected()
 {
        using namespace LocalMyList;
 
-       QModelIndexList selection = ui->pathMappingView->selectionModel()->selectedRows();
+       const QModelIndexList selection =
+               ui->pathMappingView->selectionModel()->selectedRows();
 
        for (const QModelIndex &idx : selection)
        {

diff --git a/localmylist-management/tabs/unknownfilestab.cpp b/localmylist-management/tabs/unknownfilestab.cpp
index 5e56d4e88b283560fe194c527117c244b2285476..f2e995c8d06552bb7277a714a87f93cf31a22a89 100644 (file)
--- a/localmylist-management/tabs/unknownfilestab.cpp
+++ b/localmylist-management/tabs/unknownfilestab.cpp
@@ -75,7 +75,8 @@ void UnknownFilesTab::removeSelected()
 {
        using namespace LocalMyList;
 
-       QModelIndexList selection = ui->unknownFilesView->selectionModel()->selectedRows();
+       const QModelIndexList selection =
+               ui->unknownFilesView->selectionModel()->selectedRows();
 
        RaiiTransaction t(MyList::instance()->database());
        for (const QModelIndex &idx : selection)

diff --git a/localmylist-management/tabs/watcheddirectoriestab.cpp b/localmylist-management/tabs/watcheddirectoriestab.cpp
index c2525d3610cafb1e2881084dc26604fabe1902dc..3fa297aeb7d6e91481b6bd2a00efb375659eef3b 100644 (file)
--- a/localmylist-management/tabs/watcheddirectoriestab.cpp
+++ b/localmylist-management/tabs/watcheddirectoriestab.cpp
@@ -89,7 +89,8 @@ void WatchedDirectoriesTab::removeSelected()
 {
        using namespace LocalMyList;
 
-       QModelIndexList selection = ui->watchedDirectoriesView->selectionModel()->selectedRows();
+       const QModelIndexList selection =
+               ui->watchedDirectoriesView->selectionModel()->selectedRows();
 
        for (const QModelIndex &idx : selection)
        {

diff --git a/localmylist-management/tabwidget.cpp b/localmylist-management/tabwidget.cpp
index df60c9dddd4657777ca73d310bfb9bc487d9592e..232bb48f480641f3d8fa4dc6c681e4fa032bc83d 100644 (file)
--- a/localmylist-management/tabwidget.cpp
+++ b/localmylist-management/tabwidget.cpp
@@ -32,7 +32,7 @@ void TabWidget::populateMenu(QMenu *menu)
 {
        qDeleteAll(menu->actions());
 
-       for (const QString &id : tabMenuOrder)
+       for (const QString &id : std::as_const(tabMenuOrder))
        {
                QAction *action = new QAction(registeredTabNames.value(id)(), menu);
                connect(action, &QAction::triggered, [this, id] { this->addTabRequested(id); });
@@ -67,7 +67,7 @@ void TabWidget::loadSettings(QSettings *s)
        setLockLayout(s->value("lockTabLayout", false).toBool());
        s->endGroup();
 
-       for (const QString &id : ids)
+       for (const QString &id : std::as_const(ids))
                addTab(id);
 
        setCurrentIndex(currentIndex);

diff --git a/localmylist-management/versiondialog.cpp b/localmylist-management/versiondialog.cpp
index e7ed57adb095967bb2cfe7bc01b9e53d17a227de..a3ff4dc9ab0577bf0f874969f87bcb9e9e6bc6d4 100644 (file)
--- a/localmylist-management/versiondialog.cpp
+++ b/localmylist-management/versiondialog.cpp
@@ -21,7 +21,8 @@ VersionDialog::VersionDialog(QWidget *parent) : QDialog(parent)
        if (LocalMyList::MyList::revision())
                revision = tr("from revision %1").arg(LocalMyList::MyList::revision());
 
-       const QString description = tr(R"(
+       const QString description =
+               tr(R"(
                <h3>%1 %2 <font size=\"3\">(%7 bit)</font></h3>
                <p>Built with Qt %3<br/>
                Running with Qt %4<br/>
@@ -34,17 +35,13 @@ VersionDialog::VersionDialog(QWidget *parent) : QDialog(parent)
                <p>The program is provided AS IS with NO WARRANTY OF ANY KIND,
                INCLUDING THE WARRANTY OF DESIGN, MERCHANTABILITY AND FITNESS FOR A
                PARTICULAR PURPOSE.</p>)")
-               .arg(qApp->applicationName())
-               .arg(qApp->applicationVersion())
-               .arg(QLatin1String(QT_VERSION_STR))
-               .arg(QLatin1String(qVersion()))
-               .arg(qApp->organizationName())
-               .arg(revision)
-               .arg(QSysInfo::WordSize)
-               .arg(__DATE__)
-               .arg(__TIME__)
-               // Format of __DATE__ is "Mmm dd yyyy" 16.8p1 [cpp.predefined]
-               .arg(__DATE__ + 7);
+                       .arg(qApp->applicationName(), qApp->applicationVersion(),
+                                QLatin1String(QT_VERSION_STR), QLatin1String(qVersion()),
+                                qApp->organizationName(), revision)
+                       .arg(QSysInfo::WordSize)
+                       .arg(__DATE__, __TIME__,
+                                // Format of __DATE__ is "Mmm dd yyyy" 16.8p1 [cpp.predefined]
+                                __DATE__ + 7);
 
        QLabel *copyrightLabel = new QLabel(description);
        copyrightLabel->setWordWrap(true);

diff --git a/localmylist/database.cpp b/localmylist/database.cpp
index 1bd819e7e6327ea79f1ad1b897687ea32cf3e075..c4a0b3881f83c26ba65eb1e97d79c2d02dcce216 100644 (file)
--- a/localmylist/database.cpp
+++ b/localmylist/database.cpp
@@ -2283,7 +2283,8 @@ bool Database::notifyRenameDataChanged()
 void Database::handleNotification(const QString &name, QSqlDriver::NotificationSource source, const QVariant &payload)
 {
        Q_UNUSED(source);
-       qDebug() << QString("[%1] Received notification %2, %3").arg(connectionName).arg(name).arg(payload.toString());
+       qDebug() << QString("[%1] Received notification %2, %3")
+                                       .arg(connectionName, name, payload.toString());
        if (name == "new_pending_request")
        {
                emit newPendingRequest();
@@ -2512,8 +2513,9 @@ void RaiiTransaction::commit()
 QString toSearchQuery(const QString &string)
 {
        const static QChar anyChar = QChar('%');
+       static QRegularExpression rx("\\s+");
        QString ret = string.trimmed();
-       ret.replace(QRegularExpression("\\s+"), anyChar);
+       ret.replace(rx, anyChar);
        ret = ret.append(anyChar).prepend(anyChar);
        return ret;
 }

diff --git a/localmylist/database.h b/localmylist/database.h
index 677c3095c86b9f0ba8011bdb0cbcfeb60bda91ac..0b8755a871cb07fdfc0ca55f993cec1d4627b25d 100644 (file)
--- a/localmylist/database.h
+++ b/localmylist/database.h
@@ -43,7 +43,8 @@ public:
 public slots:
        bool isConnected() const;
 
-       void setConnectionSettings(const DatabaseConnectionSettings &dbs);
+       void
+       setConnectionSettings(const LocalMyList::DatabaseConnectionSettings &dbs);
        LocalMyList::DatabaseConnectionSettings connectionSettings() const;
 
        bool transaction();
@@ -110,7 +111,8 @@ public slots:
        bool setEpisode(const LocalMyList::Episode &episode);
        bool setFile(const LocalMyList::File &file);
 
-       bool updateFilesFromPendingMyListUpdate(const PendingMyListUpdate &request);
+       bool updateFilesFromPendingMyListUpdate(
+               const LocalMyList::PendingMyListUpdate &request);
 
        bool addTitle(const LocalMyList::AnimeTitle &title);
        bool addAnime(const LocalMyList::Anime &anime);
@@ -119,17 +121,19 @@ public slots:
        bool addFileEpisodeRel(const LocalMyList::FileEpisodeRel &fileEpisodeRel);
 
        bool addUnknownFile(const LocalMyList::UnknownFile &file);
-       UnknownFile getUnknownFile(const QByteArray &ed2k, qint64 size);
-       UnknownFile getUnknownFileByPath(const QString &path);
+       LocalMyList::UnknownFile getUnknownFile(const QByteArray &ed2k,
+                                                                                       qint64 size);
+       LocalMyList::UnknownFile getUnknownFileByPath(const QString &path);
        bool removeUnknownFile(const QByteArray &ed2k, qint64 size);
 
        bool addRequest(const LocalMyList::PendingRequest &request);
-       QList<PendingRequest> getRequestBatch(int limit = 10);
+       QList<LocalMyList::PendingRequest> getRequestBatch(int limit = 10);
        bool clearRequest(const LocalMyList::PendingRequest &request);
 
        bool addPendingMyListUpdate(const LocalMyList::PendingMyListUpdate &request);
        LocalMyList::PendingMyListUpdate getPendingMyListUpdate(qint64 updateId);
-       QList<PendingMyListUpdate> getPendingMyListUpdateBatch(int limit = 10);
+       QList<LocalMyList::PendingMyListUpdate>
+       getPendingMyListUpdateBatch(int limit = 10);
        bool clearPendingMyListUpdate(const LocalMyList::PendingMyListUpdate &request);
 
        QStringList getWatchedDirectories(int hostId, bool onlyEnabled = true);
@@ -160,7 +164,7 @@ public slots:
        bool truncateMyListData();
        bool truncateDatabase();
 
-       bool removeKnownUnknownFile(const File &file);
+       bool removeKnownUnknownFile(const LocalMyList::File &file);
 
        bool log(const QString &message, int type = 1);
 
@@ -183,14 +187,30 @@ public slots:
        void disconnect();
        bool reconnect();
 
-       static void readAnimeTitleData(const SqlResultIteratorInterface &result, AnimeTitle &data, int offset = 0);
-       static void readAnimeData(const SqlResultIteratorInterface &result, Anime &data, int offset = 0);
-       static void readEpisodeData(const SqlResultIteratorInterface &result, Episode &data, int offset = 0);
-       static void readFileData(const SqlResultIteratorInterface &result, File &data, int offset = 0);
-       static void readFileLocationData(const SqlResultIteratorInterface &result, FileLocation &data, int offset = 0);
-       static void readOpenFileData(const SqlResultIteratorInterface &result, OpenFileData &data, int offset = 0);
-       static void readUnknownFileData(const SqlResultIteratorInterface &result, UnknownFile &data, int offset = 0);
-       static void readPendingMyListUpdateData(const SqlResultIteratorInterface &result, PendingMyListUpdate &data, int offset = 0);
+       static void
+       readAnimeTitleData(const LocalMyList::SqlResultIteratorInterface &result,
+                                          LocalMyList::AnimeTitle &data, int offset = 0);
+       static void
+       readAnimeData(const LocalMyList::SqlResultIteratorInterface &result,
+                                 LocalMyList::Anime &data, int offset = 0);
+       static void
+       readEpisodeData(const LocalMyList::SqlResultIteratorInterface &result,
+                                       LocalMyList::Episode &data, int offset = 0);
+       static void
+       readFileData(const LocalMyList::SqlResultIteratorInterface &result,
+                                LocalMyList::File &data, int offset = 0);
+       static void
+       readFileLocationData(const LocalMyList::SqlResultIteratorInterface &result,
+                                                LocalMyList::FileLocation &data, int offset = 0);
+       static void
+       readOpenFileData(const LocalMyList::SqlResultIteratorInterface &result,
+                                        LocalMyList::OpenFileData &data, int offset = 0);
+       static void
+       readUnknownFileData(const LocalMyList::SqlResultIteratorInterface &result,
+                                               LocalMyList::UnknownFile &data, int offset = 0);
+       static void readPendingMyListUpdateData(
+               const LocalMyList::SqlResultIteratorInterface &result,
+               LocalMyList::PendingMyListUpdate &data, int offset = 0);
 
        static QString animeTitleFields();
        static QString animeFields();

diff --git a/localmylist/directorywatcher.cpp b/localmylist/directorywatcher.cpp
index cc7b31aa8c4c6929383e17777b627750753b2540..a4aa83bba7b9723efd99126114224024b361d121 100644 (file)
--- a/localmylist/directorywatcher.cpp
+++ b/localmylist/directorywatcher.cpp
@@ -82,7 +82,8 @@ void DirectoryWatcher::watchedDirectoryChanged(const QString &path)
 
 void DirectoryWatcher::checkWatchedDirectories()
 {
-       for (const QString &dir : watcher->directories())
+       const auto &dirs = watcher->directories();
+       for (const QString &dir : dirs)
                watchedDirectoryChanged(dir);
 }
 

diff --git a/localmylist/dynamicmodel/data.cpp b/localmylist/dynamicmodel/data.cpp
index 455a860dc7b99823ef7c018598b87f0d5a208f48..e79163a149f179d7b27717ba86cb59fbb3530655 100644 (file)
--- a/localmylist/dynamicmodel/data.cpp
+++ b/localmylist/dynamicmodel/data.cpp
@@ -94,7 +94,7 @@ void Data::deref(Node *node)
 
 void Data::updated(Data *oldData)
 {
-       for (Node *node : references)
+       for (Node *node : std::as_const(references))
        {
                Q_ASSERT_X(node->parent(), "dynamicmodel/updated", "Updating node without parent");
                node->updated(oldData);
@@ -183,13 +183,17 @@ QVariant AnimeData::data(int column, int role) const
                                case TitleColumn:
                                        return animeData.titleRomaji;
                                case EpCountColumn:
-                                       return (animeData.totalEpisodeCount ? epCountString : unknownEpCountString)
-                                                       .arg(episodesInMyList)
-                                                       .arg(animeData.totalEpisodeCount
-                                                                ? animeData.totalEpisodeCount
-                                                                : qMax(animeData.highestEpno, episodesInMyList))
-                                                       .arg(specialsInMyList ? specialsCountString.arg(specialsInMyList) : "")
-                                                       .arg("");
+                                       return (animeData.totalEpisodeCount ? epCountString
+                                                                                                               : unknownEpCountString)
+                                               .arg(episodesInMyList)
+                                               .arg(
+                                                       animeData.totalEpisodeCount
+                                                               ? animeData.totalEpisodeCount
+                                                               : qMax(animeData.highestEpno, episodesInMyList))
+                                               .arg(specialsInMyList
+                                                                ? specialsCountString.arg(specialsInMyList)
+                                                                : "",
+                                                        "");
                                case RatingColumn:
                                        if (animeData.rating < 1)
                                                return "n/a";
@@ -204,9 +208,13 @@ QVariant AnimeData::data(int column, int role) const
                                        return QString::number(animeData.myVote, 'f', 2);
                                case WatchedColumn:
                                        return epCountString.arg(watchedEpisodes)
-                                                       .arg(episodesInMyList)
-                                                       .arg(specialsInMyList ? specialsCountString.arg(watchedSpecials) : "")
-                                                       .arg(specialsInMyList ? specialsCountString.arg(specialsInMyList) : "");
+                                               .arg(episodesInMyList)
+                                               .arg(specialsInMyList
+                                                                ? specialsCountString.arg(watchedSpecials)
+                                                                : "",
+                                                        specialsInMyList
+                                                                ? specialsCountString.arg(specialsInMyList)
+                                                                : "");
                                case StateColumn:
                                        return stateIdToState(myState);
                        }
@@ -216,8 +224,8 @@ QVariant AnimeData::data(int column, int role) const
                        {
                                case TitleColumn:
                                        if (!animeData.titleEnglish.isEmpty() && !animeData.titleKanji.isEmpty())
-                                               return QString("%1 -- %2").arg(animeData.titleEnglish)
-                                                               .arg(animeData.titleKanji);
+                                               return QString("%1 -- %2")
+                                                       .arg(animeData.titleEnglish, animeData.titleKanji);
                                        if (!animeData.titleEnglish.isEmpty())
                                                return animeData.titleEnglish;
                                        if (!animeData.titleKanji.isEmpty())
@@ -333,8 +341,9 @@ QVariant EpisodeData::data(int column, int role) const
                        {
                                case 0:
                                        if (!episodeData.titleEnglish.isEmpty() && !episodeData.titleKanji.isEmpty())
-                                               return QString("%1 -- %2").arg(episodeData.titleKanji)
-                                                               .arg(episodeData.titleRomaji);
+                                               return QString("%1 -- %2")
+                                                       .arg(episodeData.titleKanji,
+                                                                episodeData.titleRomaji);
                                        if (!episodeData.titleEnglish.isEmpty())
                                                return episodeData.titleEnglish;
                                        if (!episodeData.titleKanji.isEmpty())

diff --git a/localmylist/dynamicmodel/datamodel.cpp b/localmylist/dynamicmodel/datamodel.cpp
index a7bc4e097ea6fbea55c9c9e542e34c2bee924423..8bd09486a21689be2d63410ba103db16cc40a89f 100644 (file)
--- a/localmylist/dynamicmodel/datamodel.cpp
+++ b/localmylist/dynamicmodel/datamodel.cpp
@@ -42,7 +42,8 @@ bool DataModel::registerTypeRelation(TypeRelation *typeRelation)
        if (!typeRelation->sourceType().isEmpty() && !dataTypeNames.contains(typeRelation->sourceType()))
                return false;
 
-       const auto destinationTypeIt = dataTypeNames.find(typeRelation->destinationType());
+       const auto destinationTypeIt =
+               dataTypeNames.constFind(typeRelation->destinationType());
        if (destinationTypeIt == dataTypeNames.constEnd())
                return false;
 
@@ -74,7 +75,7 @@ bool DataModel::hasDataType(const QString &name) const
 
 TypeRelation *DataModel::typeRelation(const QString &source, const QString &destiantion)
 {
-       const auto it = typeRelations.find(source);
+       const auto it = typeRelations.constFind(source);
 
        if (it == typeRelations.constEnd())
        {
@@ -82,7 +83,7 @@ TypeRelation *DataModel::typeRelation(const QString &source, const QString &dest
                return 0;
        }
 
-       const auto inner = it.value().find(destiantion);
+       const auto inner = it.value().constFind(destiantion);
 
        if (inner == it.value().constEnd())
        {

diff --git a/localmylist/dynamicmodel/datamodel.h b/localmylist/dynamicmodel/datamodel.h
index 7148a2f711a3739cbbd1ea4ffeef29a1c2c4664f..f65564ef625e12a8db21326ebc35eccb19c82021 100644 (file)
--- a/localmylist/dynamicmodel/datamodel.h
+++ b/localmylist/dynamicmodel/datamodel.h
@@ -35,7 +35,7 @@ public:
        void notifyEntryAdded(DataType *dataType, int id);
 
 signals:
-       void entryAdded(DataType *dataType, int id);
+       void entryAdded(LocalMyList::DynamicModel::DataType *dataType, int id);
 
 private slots:
 /*     void animeUpdate(int aid);

diff --git a/localmylist/dynamicmodel/datatype.cpp b/localmylist/dynamicmodel/datatype.cpp
index 1bc2252c893ae0d8f8f2c953ca00401d30ffb8ff..c484cd0858b34e566f8092155dadbf33532cae33 100644 (file)
--- a/localmylist/dynamicmodel/datatype.cpp
+++ b/localmylist/dynamicmodel/datatype.cpp
@@ -84,11 +84,8 @@ QString DataType::updateQuery() const
                                %5
                        WHERE %3.%4 = :id
        )"}
-               .arg(additionalColumns())
-               .arg(tableName())
-               .arg(alias())
-               .arg(primaryKeyName())
-               .arg(additionalJoins());
+               .arg(additionalColumns(), tableName(), alias(), primaryKeyName(),
+                        additionalJoins());
 }
 
 void DataType::update(Data *data)

diff --git a/localmylist/dynamicmodel/model.h b/localmylist/dynamicmodel/model.h
index 798843eb86179e0fe4f2f4b6957c24de40cc2f83..d9bb741882f6f9b880c83f04d83ead4ea6457916 100644 (file)
--- a/localmylist/dynamicmodel/model.h
+++ b/localmylist/dynamicmodel/model.h
@@ -58,11 +58,12 @@ public slots:
        void reload();
 
 private slots:
-       void entryAdded(DataType *dataType, int id);
+       void entryAdded(LocalMyList::DynamicModel::DataType *dataType, int id);
        void episodeInsert(int aid, int eid);
 
 signals:
-       void queryChanged(QueryParser query);
+       void queryChanged(LocalMyList::DynamicModel::QueryParser
+                                                 query); // clazy:exclude=overloaded-signal
        void queryChanged(QString query);
 
 private:

diff --git a/localmylist/dynamicmodel/node.cpp b/localmylist/dynamicmodel/node.cpp
index 9160899b27622cccb8cec3914c2b0ec1ab945bcf..08cd10ecc6a5263ce89382b14075848f8aa9ab79 100644 (file)
--- a/localmylist/dynamicmodel/node.cpp
+++ b/localmylist/dynamicmodel/node.cpp
@@ -317,10 +317,10 @@ int Node::level() const
 Node *Node::findChildByPrimaryValue(const QVariant &primaryValue)
 {
        // TODO this can be made logarithmic
-       // children of type column are ordered ascending by their valie
+       // children of type column are ordered ascending by their value
        // datatypes with ids can bee looked up in the DataType's dataStore,
        // followed by node lookup.
-       for (Node *child : m_children)
+       for (Node *child : std::as_const(m_children))
        {
                if (child->data()->primaryValue() == primaryValue)
                        return child;

diff --git a/localmylist/dynamicmodel/queryparser.cpp b/localmylist/dynamicmodel/queryparser.cpp
index d948c17718fb5f6d22e833ab39b4953075d8f92a..ed03c18a3433da625467fbd27ecd8fd537f88325 100644 (file)
--- a/localmylist/dynamicmodel/queryparser.cpp
+++ b/localmylist/dynamicmodel/queryparser.cpp
@@ -114,7 +114,7 @@ QDebug operator<<(QDebug dbg, const QueryParser::Level &l)
        if (l.column.isEmpty())\r
                dbg << QString("[%1:%2]").arg(l.type).arg(l.table);\r
        else\r
-               dbg << QString("[%1:%2.%3]").arg(l.type).arg(l.table).arg(l.column);\r
+               dbg << QString("[%1:%2.%3]").arg(l.type).arg(l.table, l.column);\r
        return dbg;\r
 }\r
 \r
@@ -215,8 +215,9 @@ bool QueryParser::parse(const QString &rawPath)
                {\r
                        if (!table_columns[currentLevel.table].contains(column))\r
                        {\r
-                               m_errorString = QObject::tr("Column %1 does not exist in table %2.")\r
-                                               .arg(column).arg(table);\r
+                               m_errorString =\r
+                                       QObject::tr("Column %1 does not exist in table %2.")\r
+                                               .arg(column, table);\r
                                m_valid = false;\r
                                return m_valid;\r
                        }\r
@@ -228,8 +229,10 @@ bool QueryParser::parse(const QString &rawPath)
                        && m_levels.last().table != currentLevel.table\r
                        && !m_dataModel->hasTypeRelation(m_levels.last().table, currentLevel.table))\r
                {\r
-                       m_errorString = QObject::tr("No relation defined between table %1 and table %2.")\r
-                                       .arg(m_levels.last().table).arg(currentLevel.table);\r
+                       m_errorString =\r
+                               QObject::tr(\r
+                                       "No relation defined between table %1 and table %2.")\r
+                                       .arg(m_levels.last().table, currentLevel.table);\r
                        m_valid = false;\r
                        return m_valid;\r
                }\r
@@ -256,8 +259,7 @@ QString QueryParser::buildSql(int currentLevel) const
 \r
        if (!lastLevel.column.isEmpty())\r
        {\r
-               columns += QString(", %2.%1")\r
-                               .arg(lastLevel.column).arg(dataType->alias());\r
+               columns += QString(", %2.%1").arg(lastLevel.column, dataType->alias());\r
        }\r
        else\r
        {\r
@@ -272,8 +274,8 @@ QString QueryParser::buildSql(int currentLevel) const
 \r
        if (lastLevel.type == ColumnEntry)\r
        {\r
-               QString column = QString("%2.%1")\r
-                               .arg(lastLevel.column).arg(dataType->alias());\r
+               QString column =\r
+                       QString("%2.%1").arg(lastLevel.column, dataType->alias());\r
                ret += QString("\n\tGROUP BY %1\n\tORDER BY %1")\r
                                .arg(column);\r
        }\r
@@ -398,8 +400,7 @@ QString QueryParser::buildEntrySqlInternal(int currentLevel) const
 \r
        if (!lastLevel.column.isEmpty())\r
        {\r
-               columns += QString(", %2.%1")\r
-                               .arg(lastLevel.column).arg(dataType->alias());\r
+               columns += QString(", %2.%1").arg(lastLevel.column, dataType->alias());\r
        }\r
        else\r
        {\r
@@ -411,9 +412,8 @@ QString QueryParser::buildEntrySqlInternal(int currentLevel) const
        }\r
 \r
        QString ret = buildSelect(currentLevel, columns, true, true);\r
-       ret += QString{"\n\t\tWHERE %1.%2 = :id"}\r
-                       .arg(dataType->alias())\r
-                       .arg(dataType->primaryKeyName());\r
+       ret += QString{"\n\t\tWHERE %1.%2 = :id"}.arg(dataType->alias(),\r
+                                                                                                 dataType->primaryKeyName());\r
        return ret;\r
 }\r
 \r
@@ -423,7 +423,7 @@ QString QueryParser::buildSelect(int currentLevel, const QString &columns, bool
        const DataType *dataType = m_dataModel->dataType(lastLevel.table);\r
        const QString joins = buildJoins(currentLevel - 1, willRequireAdditionalJoins, includeConditions, aliasSuffix);\r
        return QString("\nSELECT DISTINCT %4 FROM %1 %2%3")\r
-                       .arg(lastLevel.table).arg(dataType->alias() + aliasSuffix).arg(joins).arg(columns);\r
+               .arg(lastLevel.table, dataType->alias() + aliasSuffix, joins, columns);\r
 }\r
 \r
 QString QueryParser::buildJoins(int currentLevel, bool willRequireAdditionalJoins, bool includeConditions, const QString &aliasSuffix) const\r
@@ -459,12 +459,14 @@ QString QueryParser::buildJoins(int currentLevel, bool willRequireAdditionalJoin
                if (level(i).type == ColumnEntry)\r
                {\r
                        *it << QString("%1.%2 = %3")\r
-                                       .arg(dataType->alias() + aliasSuffix).arg(level(i).column).arg(nextPlaceHolder(i));\r
+                                          .arg(dataType->alias() + aliasSuffix, level(i).column,\r
+                                                       nextPlaceHolder(i));\r
                }\r
                else\r
                {\r
                        *it << QString("%1.%2 = %3")\r
-                                       .arg(dataType->alias() + aliasSuffix).arg(dataType->primaryKeyName()).arg(nextPlaceHolder(i));\r
+                                          .arg(dataType->alias() + aliasSuffix,\r
+                                                       dataType->primaryKeyName(), nextPlaceHolder(i));\r
                }\r
        }\r
        qDebug() << conditions;\r
@@ -478,7 +480,9 @@ QString QueryParser::buildJoins(int currentLevel, bool willRequireAdditionalJoin
                if (!addedTables.contains(l.table) && conditions.contains(l.table))\r
                {\r
                        const DataType *dataType = m_dataModel->dataType(l.table);\r
-                       ret += QString("\n\tJOIN %1 %2 ON %3\n").arg(l.table).arg(dataType->alias() + aliasSuffix).arg(conditions[l.table].join("\n\t\tAND "));\r
+                       ret += QString("\n\tJOIN %1 %2 ON %3\n")\r
+                                          .arg(l.table, dataType->alias() + aliasSuffix,\r
+                                                       conditions[l.table].join("\n\t\tAND "));\r
                        addedTables.insert(l.table);\r
                }\r
        }\r
@@ -503,12 +507,10 @@ QString QueryParser::valueColumn(int currentLevel, const QString &alias) const
        QString column = QString{"%2.%1"};\r
        if (lastLevel.type == ColumnEntry)\r
        {\r
-               return column\r
-                               .arg(lastLevel.column).arg(alias);\r
+               return column.arg(lastLevel.column, alias);\r
        }\r
        const DataType *dataType = m_dataModel->dataType(lastLevel.table);\r
-       return column\r
-                       .arg(dataType->primaryKeyName()).arg(alias);\r
+       return column.arg(dataType->primaryKeyName(), alias);\r
 }\r
 \r
 QString QueryParser::currentPlaceHolder(int currentLevel) const\r

diff --git a/localmylist/dynamicmodel/typerelation.cpp b/localmylist/dynamicmodel/typerelation.cpp
index 6aa5c9fbc2890deb35a727978349477518af09d9..99a45cd3e180002ecb5a2c7a9bf158472f19192b 100644 (file)
--- a/localmylist/dynamicmodel/typerelation.cpp
+++ b/localmylist/dynamicmodel/typerelation.cpp
@@ -50,8 +50,7 @@ QString ForeignKeyRelation::destinationType() const
 
 QString ForeignKeyRelation::joinCondition(const QString &leftAlias, const QString &rightAlias) const
 {
-       return QString{"%2.%3 = %1.%4"}
-               .arg(rightAlias).arg(leftAlias).arg(m_pk).arg(m_fk);
+       return QString{"%2.%3 = %1.%4"}.arg(rightAlias, leftAlias, m_pk, m_fk);
 }
 
 } // namespace DynamicModel

diff --git a/localmylist/dynamicmodel/types.cpp b/localmylist/dynamicmodel/types.cpp
index 4c1254c6b2a470caadc04ae0ec85ba2b1879ddb5..e0962f7b39dc31272cc045418f4177b83b368089 100644 (file)
--- a/localmylist/dynamicmodel/types.cpp
+++ b/localmylist/dynamicmodel/types.cpp
@@ -175,7 +175,7 @@ void AnimeType::animeAdded(int aid)
 
 void AnimeType::animeUpdated(int aid)
 {
-       const auto it = m_dataStore.find(aid);
+       const auto it = m_dataStore.constFind(aid);
 
        if (it == m_dataStore.constEnd())
                return;
@@ -186,7 +186,7 @@ void AnimeType::animeUpdated(int aid)
 void AnimeType::animeDeleted(int aid)
 {
        qDebug() << "animeDeleted" << aid;
-       const auto it = m_dataStore.find(aid);
+       const auto it = m_dataStore.constFind(aid);
 
        if (it == m_dataStore.constEnd())
                return;
@@ -350,7 +350,7 @@ void EpisodeType::episodeAdded(int eid, int aid)
 void EpisodeType::episodeUpdated(int eid, int aid)
 {
        Q_UNUSED(aid);
-       const auto it = m_dataStore.find(eid);
+       const auto it = m_dataStore.constFind(eid);
 
        if (it == m_dataStore.constEnd())
                return;
@@ -361,7 +361,7 @@ void EpisodeType::episodeUpdated(int eid, int aid)
 void EpisodeType::episodeDeleted(int eid, int aid)
 {
        Q_UNUSED(aid);
-       const auto it = m_dataStore.find(eid);
+       const auto it = m_dataStore.constFind(eid);
 
        if (it == m_dataStore.constEnd())
                return;
@@ -457,7 +457,7 @@ void FileType::fileUpdated(int fid, int eid, int aid)
 {
        Q_UNUSED(aid);
        Q_UNUSED(eid);
-       const auto it = m_dataStore.find(fid);
+       const auto it = m_dataStore.constFind(fid);
 
        if (it == m_dataStore.constEnd())
                return;
@@ -469,7 +469,7 @@ void FileType::fileDeleted(int fid, int eid, int aid)
 {
        Q_UNUSED(aid);
        Q_UNUSED(eid);
-       const auto it = m_dataStore.find(fid);
+       const auto it = m_dataStore.constFind(fid);
 
        if (it == m_dataStore.constEnd())
                return;
@@ -550,7 +550,7 @@ void FileLocationType::fileLocationAdded(int locationId, int fid)
 void FileLocationType::fileLocationUpdated(int locationId, int fid)
 {
        Q_UNUSED(fid);
-       const auto it = m_dataStore.find(locationId);
+       const auto it = m_dataStore.constFind(locationId);
 
        if (it == m_dataStore.constEnd())
                return;
@@ -561,7 +561,7 @@ void FileLocationType::fileLocationUpdated(int locationId, int fid)
 void FileLocationType::fileLocationDeleted(int locationId, int fid)
 {
        Q_UNUSED(fid);
-       const auto it = m_dataStore.find(locationId);
+       const auto it = m_dataStore.constFind(locationId);
 
        if (it == m_dataStore.constEnd())
                return;

diff --git a/localmylist/dynamicmodel/types.h b/localmylist/dynamicmodel/types.h
index a1f652504a34c93ec21f68c34f497b23eab0462d..aeeb85334512125bdba67c3aa395311ee04b6197 100644 (file)
--- a/localmylist/dynamicmodel/types.h
+++ b/localmylist/dynamicmodel/types.h
@@ -21,7 +21,7 @@ class LOCALMYLISTSHARED_EXPORT ColumnType : public DataType
        QString additionalColumns() const override;
 
 protected:
-       Data *readEntry(const SqlResultIteratorInterface &it);
+       Data *readEntry(const SqlResultIteratorInterface &it) override;
 };
 
 // =============================================================================================================
@@ -40,10 +40,10 @@ class LOCALMYLISTSHARED_EXPORT AnimeType : public DataType
 
        void update(Data *data) override;
 
-       NodeCompare nodeCompareFunction() const;
+       NodeCompare nodeCompareFunction() const override;
 
 protected:
-       Data *readEntry(const SqlResultIteratorInterface &it);
+       Data *readEntry(const SqlResultIteratorInterface &it) override;
 
 private slots:
        void animeAdded(int aid);
@@ -78,7 +78,7 @@ class LOCALMYLISTSHARED_EXPORT EpisodeType : public DataType
 
        void update(Data *data) override;
 
-       NodeCompare nodeCompareFunction() const;
+       NodeCompare nodeCompareFunction() const override;
 
 protected:
        Data *readEntry(const SqlResultIteratorInterface &it) override;
@@ -137,7 +137,7 @@ class LOCALMYLISTSHARED_EXPORT FileLocationType : public DataType
        QString tableName() const override;
        QString alias() const override;
        QString primaryKeyName() const override;
-       QString additionalColumns() const;
+       QString additionalColumns() const override;
        QString additionalJoins() const override;
 
        void registerd() override;

diff --git a/localmylist/filelocationchecktask.cpp b/localmylist/filelocationchecktask.cpp
index bba82864607255e455581f8d901694482f47adbe..554db4ea3a334b7e8ecc83558e67b627cfa94286 100644 (file)
--- a/localmylist/filelocationchecktask.cpp
+++ b/localmylist/filelocationchecktask.cpp
@@ -32,7 +32,7 @@ void FileLocationCheckTask::workUnit()
 
        for (FileLocation &fl : fileLocations)
        {
-               if (!QFileInfo(fl.path).exists())
+               if (!QFileInfo::exists(fl.path))
                {
                        db->log(QObject::tr("Removing file location %1: %2. File does not exist.")
                                        .arg(fl.locationId)

diff --git a/localmylist/mylist.h b/localmylist/mylist.h
index de3413925db3abaa7dcd3d86b3f725a82393a8a8..998399dad2a25a90b8e1d5a3fcaf7276025e3a76 100644 (file)
--- a/localmylist/mylist.h
+++ b/localmylist/mylist.h
@@ -74,12 +74,12 @@ public slots:
        bool playFile(const LocalMyList::OpenFileData &ofd,
                                  const QStringList &additionalArguments = QStringList());
 
-       AbstractTask *addFile(const QFileInfo &file);
-       AbstractTask *addDirectory(const QDir &directory);
-       AbstractTask *importTitles(const QFileInfo &file);
-       AbstractTask *importTitlesFromUrl(const QUrl &url);
-       AbstractTask *importMyList(const QFileInfo &file);
-       void executeTask(AbstractTask *task);
+       LocalMyList::AbstractTask *addFile(const QFileInfo &file);
+       LocalMyList::AbstractTask *addDirectory(const QDir &directory);
+       LocalMyList::AbstractTask *importTitles(const QFileInfo &file);
+       LocalMyList::AbstractTask *importTitlesFromUrl(const QUrl &url);
+       LocalMyList::AbstractTask *importMyList(const QFileInfo &file);
+       void executeTask(LocalMyList::AbstractTask *task);
 
        QString mapPathFromSource(const QString &path, int sourceHost);
        QString mapPathToSource(const QString &path, int *sourceHost = 0);

diff --git a/localmylist/mylistexportparsetask.cpp b/localmylist/mylistexportparsetask.cpp
index c27fcb40fd8671c24f4b60d812615292cd571b7b..9f61a7a305b26f0858160405b62ed10772dbdb13 100644 (file)
--- a/localmylist/mylistexportparsetask.cpp
+++ b/localmylist/mylistexportparsetask.cpp
@@ -498,7 +498,7 @@ void MyListExportParseTask::readFile()
                                f.releaseDate = QDateTime::fromString(readElementText(), dateTimeFormat);
                        }
                        else if (name() == "VersionName") {
-                               f.version = readElementText().mid(1).toInt();
+                               f.version = QStringView{readElementText()}.mid(1).toInt();
                        }
                        // f.censored;
                        else if (name() == "TypeName") {

diff --git a/localmylist/mylistmodel.h b/localmylist/mylistmodel.h
index 836862a81b38bfede8c4b2e5ce7318f3502eba21..7aa6ec9609f465c78af4fda9f07d315484ec4625 100644 (file)
--- a/localmylist/mylistmodel.h
+++ b/localmylist/mylistmodel.h
@@ -71,7 +71,7 @@ public:
 signals:
 
 public slots:
-       MyListNode *node(const QModelIndex &idx) const;
+       LocalMyList::MyListNode *node(const QModelIndex &idx) const;
 
        void reload();
 

diff --git a/localmylist/mylistnode.cpp b/localmylist/mylistnode.cpp
index cd57eb4ff3153297a5d21cee104eb1b8d4e44210..41947fde8b0384bd0ad37ccb5d943ee97f5372e0 100644 (file)
--- a/localmylist/mylistnode.cpp
+++ b/localmylist/mylistnode.cpp
@@ -415,8 +415,9 @@ QVariant MyListAnimeNode::data(int column, int role) const
                        {
                                case 0:
                                        if (!animeData.data.titleEnglish.isEmpty() && !animeData.data.titleKanji.isEmpty())
-                                               return QString("%1 -- %2").arg(animeData.data.titleEnglish)
-                                                               .arg(animeData.data.titleKanji);
+                                               return QString("%1 -- %2")
+                                                       .arg(animeData.data.titleEnglish,
+                                                                animeData.data.titleKanji);
                                        if (!animeData.data.titleEnglish.isEmpty())
                                                return animeData.data.titleEnglish;
                                        if (!animeData.data.titleKanji.isEmpty())
@@ -674,8 +675,9 @@ QVariant MyListEpisodeNode::data(int column, int role) const
                        {
                                case 0:
                                        if (!episodeData.data.titleEnglish.isEmpty() && !episodeData.data.titleKanji.isEmpty())
-                                               return QString("%1 -- %2").arg(episodeData.data.titleKanji)
-                                                               .arg(episodeData.data.titleRomaji);
+                                               return QString("%1 -- %2")
+                                                       .arg(episodeData.data.titleKanji,
+                                                                episodeData.data.titleRomaji);
                                        if (!episodeData.data.titleEnglish.isEmpty())
                                                return episodeData.data.titleEnglish;
                                        if (!episodeData.data.titleKanji.isEmpty())

diff --git a/localmylist/renamehandler.cpp b/localmylist/renamehandler.cpp
index 66029c40372ac2c1956d8fac95e410dbeccdc92b..3879860d07ab326ab39aac37186f9184a1e5b927 100644 (file)
--- a/localmylist/renamehandler.cpp
+++ b/localmylist/renamehandler.cpp
@@ -55,7 +55,8 @@ void RenameHandler::handleRename()
 
        t.commit();
 
-       QStringList watchedDirectories = db->getWatchedDirectories(MyList::instance()->hostId());
+       const QStringList watchedDirectories =
+               db->getWatchedDirectories(MyList::instance()->hostId());
 
        while (q.next())
        {
@@ -148,7 +149,9 @@ void RenameHandler::handleRename()
                if (!differenceInCaseOnly && newFile.exists())
                {
                        fl.failedRename = true;
-                       fl.renameError = tr("Rename: Failed to rename file <%1>. Destination <%2> exists.").arg(oldFile.filePath()).arg(newFileString);
+                       fl.renameError = tr("Rename: Failed to rename file <%1>. "
+                                                               "Destination <%2> exists.")
+                                                                .arg(oldFile.filePath(), newFileString);
                        db->setFileLocation(fl);
                        db->log(fl.renameError, 2);
                        continue;
@@ -159,7 +162,10 @@ void RenameHandler::handleRename()
                        if (!QDir().mkpath(newFilePath))
                        {
                                fl.failedRename = true;
-                               fl.renameError = tr("Rename: Failed to rename file <%1>. Destination path <%2> does not exist and could not be created.").arg(oldFile.filePath()).arg(newFile.canonicalPath());
+                               fl.renameError =
+                                       tr("Rename: Failed to rename file <%1>. Destination path "
+                                          "<%2> does not exist and could not be created.")
+                                               .arg(oldFile.filePath(), newFile.canonicalPath());
                                db->setFileLocation(fl);
                                db->log(fl.renameError, 2);
                                continue;
@@ -170,7 +176,10 @@ void RenameHandler::handleRename()
                if (!QFileInfo(newFilePath).isWritable())
                {
                        fl.failedRename = true;
-                       fl.renameError = tr("Rename: Failed to rename file <%1>. Destination directory <%2> is not writable.").arg(oldFile.canonicalFilePath()).arg(newFile.absolutePath());
+                       fl.renameError =
+                               tr("Rename: Failed to rename file <%1>. Destination directory "
+                                  "<%2> is not writable.")
+                                       .arg(oldFile.canonicalFilePath(), newFile.absolutePath());
                        db->setFileLocation(fl);
                        db->log(fl.renameError, 2);
                        continue;
@@ -199,7 +208,10 @@ void RenameHandler::handleRename()
                if (!renameFile(oldFile.canonicalFilePath(), newFileString, &exactFileName))
                {
                        fl.failedRename = true;
-                       fl.renameError = tr("Rename: Failed to rename file <%1>. Failed to rename file to <%2>").arg(oldFile.canonicalFilePath()).arg(newFileString);
+                       fl.renameError =
+                               tr("Rename: Failed to rename file <%1>. Failed to rename file "
+                                  "to <%2>")
+                                       .arg(oldFile.canonicalFilePath(), newFileString);
                        db->setFileLocation(fl);
                        db->log(fl.renameError, 2);
                        continue;
@@ -219,7 +231,8 @@ void RenameHandler::handleRename()
                fl.renameError = "";
                db->setFileLocation(fl);
 
-               db->log(tr("Rename: File <%1> was renamed to <%2>").arg(oldFile.canonicalFilePath()).arg(fl.path));
+               db->log(tr("Rename: File <%1> was renamed to <%2>")
+                                       .arg(oldFile.canonicalFilePath(), fl.path));
 
                if (settings->get("renameRemoveEmptyDirectories", false))
                {

diff --git a/localmylist/reportengine.h b/localmylist/reportengine.h
index 639b8ec4b8d3a1cf02c8471a6142a7970fff23b3..5bda11a0632ed811d82f2e6f47c9bd99b3d4534f 100644 (file)
--- a/localmylist/reportengine.h
+++ b/localmylist/reportengine.h
@@ -25,7 +25,7 @@ public:
 signals:
        
 public slots:
-       void run(const Report &report);
+       void run(const LocalMyList::Report &report);
        void run(const QString &report);
 
 private:

diff --git a/localmylist/scriptable.cpp b/localmylist/scriptable.cpp
index edf6d45883fc2aeaffe3d16259104913b850cb45..87d84478ba155a72e97d67687f776707fc9914a8 100644 (file)
--- a/localmylist/scriptable.cpp
+++ b/localmylist/scriptable.cpp
@@ -50,8 +50,8 @@ QString titleTypeToString(int titleType)
 QString AnimeTitle::toString() const
 {
        return QString("AnimeTitle('%1', %2, %3, %4)")
-                       .arg(read_title()).arg(read_language())
-                       .arg(titleTypeToString(read_type())).arg(read_aid());
+               .arg(read_title(), read_language(), titleTypeToString(read_type()))
+               .arg(read_aid());
 }
 
 // --------------------------------------------------------------------------------

diff --git a/localmylist/service.h b/localmylist/service.h
index 6cd2b6b84e0e4fb2bb3c50cf10e6502d97b1117d..2f7ff5778b1e52180e1649233a19859cd1a4e39f 100644 (file)
--- a/localmylist/service.h
+++ b/localmylist/service.h
@@ -9,7 +9,6 @@ namespace LocalMyList {
 class LOCALMYLISTSHARED_EXPORT Service : public QObject
 {
        Q_OBJECT
-       Q_ENUMS(ServiceState)
        Q_PROPERTY(ServiceState state READ state NOTIFY stateChanged)
        Q_PROPERTY(QString name READ name)
 
@@ -24,6 +23,7 @@ public:
                Error,         //<! Service failed to start
                Invalid = -1,  //<! This value represents the state of an invalid/nonexistant service. A service can't be in this state.
        };
+       Q_ENUM(ServiceState)
 
        ServiceState state() const;
        QString name() const;
@@ -47,7 +47,7 @@ public slots:
        void processRequests();
 
 signals:
-       void stateChanged(Service::ServiceState state);
+       void stateChanged(LocalMyList::Service::ServiceState state);
 
 protected:
        Service(const QString &name, QObject *parent = 0);

diff --git a/localmylist/servicemanager.cpp b/localmylist/servicemanager.cpp
index 5fd2fbf0239de87e7f2c577ffc081a20ef4095ae..bf0b9818e4c3f1d6661de3c2951787ca0673ce0d 100644 (file)
--- a/localmylist/servicemanager.cpp
+++ b/localmylist/servicemanager.cpp
@@ -12,7 +12,7 @@ ServiceManager::ServiceManager(QObject *parent) :
 
 ServiceManager::~ServiceManager()
 {
-       for (Service *s : m_registeredServices)
+       for (Service *s : std::as_const(m_registeredServices))
                callServiceSlot(s, "deleteLater");
 }
 
@@ -42,7 +42,7 @@ void ServiceManager::start(const QString &serviceName)
 
 void ServiceManager::startAll()
 {
-       for (Service *s : m_registeredServices)
+       for (Service *s : std::as_const(m_registeredServices))
                callServiceSlot(s, "start");
 }
 
@@ -57,7 +57,7 @@ void ServiceManager::stop(const QString &serviceName)
 
 void ServiceManager::stopAll()
 {
-       for (Service *s : m_registeredServices)
+       for (Service *s : std::as_const(m_registeredServices))
                callServiceSlot(s, "stop");
 }
 
@@ -72,7 +72,7 @@ void ServiceManager::processRequests(const QString &serviceName)
 
 void ServiceManager::processRequestsAll()
 {
-       for (Service *s : m_registeredServices)
+       for (Service *s : std::as_const(m_registeredServices))
                callServiceSlot(s, "processRequests");
 }
 

diff --git a/localmylist/servicemanager.h b/localmylist/servicemanager.h
index baafcceb29abe6e5f89cd0ae0f0c7002e1d5f145..add0d33aaed7ae46288880b90ac74d9f9f427025 100644 (file)
--- a/localmylist/servicemanager.h
+++ b/localmylist/servicemanager.h
@@ -28,7 +28,8 @@ public:
        bool addService(Service *service);
 
 signals:
-       void stateChanged(const QString &serviceName, Service::ServiceState state);
+       void stateChanged(const QString &serviceName,
+                                         LocalMyList::Service::ServiceState state);
 
 public slots:
        void start(const QString &serviceName);
@@ -38,7 +39,7 @@ public slots:
        void processRequests(const QString &serviceName);
        void processRequestsAll();
 
-       Service::ServiceState state(const QString &serviceName);
+       LocalMyList::Service::ServiceState state(const QString &serviceName);
 
        bool hasService(const QString &serviceName) const;
 
@@ -49,7 +50,7 @@ public slots:
        QList<QString> registeredServices() const;
 
 private slots:
-       void serviceStateChanged(Service::ServiceState state);
+       void serviceStateChanged(LocalMyList::Service::ServiceState state);
 
 private:
        void callServiceSlot(Service *s, const char *slot) const;

diff --git a/play-next/main.cpp b/play-next/main.cpp
index 0d823b54ffa272205c547203fed4d04519141a05..2f1094292c365ec6bfa10519691bd347afed084b 100644 (file)
--- a/play-next/main.cpp
+++ b/play-next/main.cpp
@@ -16,7 +16,7 @@ int main(int argc, char *argv[])
        QTextStream cout(stdout);
        if (a.arguments().count() < 2)
        {
-               cout << "Usage: " << a.arguments()[0] << " TITLE" << Qt::endl;
+               cout << "Usage: " << a.arguments().at(0) << " TITLE" << Qt::endl;
                return 1;
        }
 

diff --git a/runscript/main.cpp b/runscript/main.cpp
index 458043a7295efb63a78de4603c729a7bcb166e52..96373724fd8837f5eef3a7e1f7aec7d1ea6b86b3 100644 (file)
--- a/runscript/main.cpp
+++ b/runscript/main.cpp
@@ -39,8 +39,9 @@ int main(int argc, char *argv[])
        QTextStream cout(stdout);
        if (a.arguments().count() < 2)
        {
-               cout << "Usage: " << a.arguments()[0] << " FILE [ARGS...]" << Qt::endl;
-               cout << "       " << a.arguments()[0] << " -e SCRIPT [-- ARGS...]"
+               cout << "Usage: " << a.arguments().at(0) << " FILE [ARGS...]"
+                        << Qt::endl;
+               cout << "       " << a.arguments().at(0) << " -e SCRIPT [-- ARGS...]"
                         << Qt::endl;
                return 1;
        }