From 964acf69b6382f8ee8252c640c5048f8f4644622 Mon Sep 17 00:00:00 2001
From: Patrizio Bekerle <patrizio@bekerle.com>
Date: Mon, 1 Jan 2024 17:40:40 +0100
Subject: [PATCH] style: format code using clang-format

---
 src/dialogs/actiondialog.cpp           |  8 ++++++--
 src/dialogs/scriptrepositorydialog.cpp |  6 ++----
 src/dialogs/scriptrepositorydialog.h   |  2 +-
 src/dialogs/settingsdialog.cpp         | 20 +++++++++++++++-----
 src/dialogs/tododialog.cpp             |  4 +++-
 src/helpers/toolbarcontainer.cpp       |  3 +--
 src/mainwindow.cpp                     | 23 +++++++++++++----------
 src/services/scriptingservice.cpp      |  4 +++-
 src/utils/gui.cpp                      |  8 ++++++--
 src/utils/urlhandler.cpp               |  7 ++++---
 src/widgets/logwidget.cpp              |  4 +++-
 src/widgets/passwordlineedit.cpp       |  9 +++++----
 12 files changed, 62 insertions(+), 36 deletions(-)

diff --git a/src/dialogs/actiondialog.cpp b/src/dialogs/actiondialog.cpp
index 78c6a13ca0..4de6f62d35 100644
--- a/src/dialogs/actiondialog.cpp
+++ b/src/dialogs/actiondialog.cpp
@@ -27,7 +27,9 @@ void ActionDialog::refreshUi() {
     QList<QMenu *> menuList =
         _menuBar->findChildren<QMenu *>(QString(), Qt::FindDirectChildrenOnly);
 
-    Q_FOREACH (QMenu *menu, menuList) { buildActionTreeForMenu(menu); }
+    Q_FOREACH (QMenu *menu, menuList) {
+        buildActionTreeForMenu(menu);
+    }
 
     ui->actionTreeWidget->setRootIsDecorated(false);
     ui->actionTreeWidget->expandAll();
@@ -62,7 +64,9 @@ void ActionDialog::buildActionTreeForMenu(QMenu *menu, QTreeWidgetItem *parentIt
     QList<QMenu *> menuList = menu->findChildren<QMenu *>(QString(), Qt::FindDirectChildrenOnly);
 
     // build the tree for that sub-menu
-    Q_FOREACH (QMenu *subMenu, menuList) { buildActionTreeForMenu(subMenu, menuItem); }
+    Q_FOREACH (QMenu *subMenu, menuList) {
+        buildActionTreeForMenu(subMenu, menuItem);
+    }
 
     // add all actions to the tree widget item
     Q_FOREACH (QAction *action, menu->actions()) {
diff --git a/src/dialogs/scriptrepositorydialog.cpp b/src/dialogs/scriptrepositorydialog.cpp
index c2b937a7df..177bc6caa5 100644
--- a/src/dialogs/scriptrepositorydialog.cpp
+++ b/src/dialogs/scriptrepositorydialog.cpp
@@ -130,7 +130,7 @@ void ScriptRepositoryDialog::addScriptTreeWidgetItem(const ScriptInfoJson &scrip
     ui->scriptTreeWidget->resizeColumnToContents(0);
 }
 
-void ScriptRepositoryDialog::searchForUpdatesForScripts(const QList<Script>& scripts) {
+void ScriptRepositoryDialog::searchForUpdatesForScripts(const QList<Script> &scripts) {
     ui->searchScriptEdit->hide();
     setWindowTitle(tr("Script updates"));
     ui->overviewLabel->setText(tr("All scripts are up-to-date."));
@@ -176,9 +176,7 @@ void ScriptRepositoryDialog::searchForUpdatesForScripts(const QList<Script>& scr
 /**
  * Searches for script updates
  */
-void ScriptRepositoryDialog::searchForUpdates() {
-    searchForUpdatesForScripts(Script::fetchAll());
-}
+void ScriptRepositoryDialog::searchForUpdates() { searchForUpdatesForScripts(Script::fetchAll()); }
 
 void ScriptRepositoryDialog::parseScriptRepositoryMetaData(const QByteArray &arr) {
     QJsonDocument doc = QJsonDocument::fromJson(arr);
diff --git a/src/dialogs/scriptrepositorydialog.h b/src/dialogs/scriptrepositorydialog.h
index e24eb6f22c..f7c97e5832 100644
--- a/src/dialogs/scriptrepositorydialog.h
+++ b/src/dialogs/scriptrepositorydialog.h
@@ -23,7 +23,7 @@ class ScriptRepositoryDialog : public MasterDialog {
     Script getLastInstalledScript();
 
     void searchForUpdates();
-    void searchForUpdatesForScripts(const QList<Script>& scripts);
+    void searchForUpdatesForScripts(const QList<Script> &scripts);
 
    signals:
     void updateFound();
diff --git a/src/dialogs/settingsdialog.cpp b/src/dialogs/settingsdialog.cpp
index 68d61b9a45..def41a51c1 100644
--- a/src/dialogs/settingsdialog.cpp
+++ b/src/dialogs/settingsdialog.cpp
@@ -1375,7 +1375,9 @@ void SettingsDialog::loadInterfaceStyleComboBox() const {
     ui->interfaceStyleComboBox->clear();
     ui->interfaceStyleComboBox->addItem(tr("Automatic (needs restart)"));
 
-    Q_FOREACH (QString style, QStyleFactory::keys()) { ui->interfaceStyleComboBox->addItem(style); }
+    Q_FOREACH (QString style, QStyleFactory::keys()) {
+        ui->interfaceStyleComboBox->addItem(style);
+    }
 
     QSettings settings;
     QString interfaceStyle = settings.value(QStringLiteral("interfaceStyle")).toString();
@@ -2960,7 +2962,9 @@ void SettingsDialog::reloadCurrentScriptPage() {
             // scriptSettingsFrame
             QList<ScriptSettingWidget *> widgets =
                 ui->scriptSettingsFrame->findChildren<ScriptSettingWidget *>();
-            Q_FOREACH (ScriptSettingWidget *widget, widgets) { delete widget; }
+            Q_FOREACH (ScriptSettingWidget *widget, widgets) {
+                delete widget;
+            }
 
             foreach (QVariant variable, variables) {
                 QMap<QString, QVariant> varMap = variable.toMap();
@@ -3247,7 +3251,9 @@ void SettingsDialog::on_shortcutSearchLineEdit_textChanged(const QString &arg1)
         }
     } else {
         // show all items otherwise
-        Q_FOREACH (QTreeWidgetItem *item, allItems) { item->setHidden(false); }
+        Q_FOREACH (QTreeWidgetItem *item, allItems) {
+            item->setHidden(false);
+        }
     }
 }
 
@@ -3576,7 +3582,9 @@ void SettingsDialog::on_searchLineEdit_textChanged(const QString &arg1) {
         }
     } else {
         // show all items otherwise
-        Q_FOREACH (QTreeWidgetItem *item, allItems) { item->setHidden(false); }
+        Q_FOREACH (QTreeWidgetItem *item, allItems) {
+            item->setHidden(false);
+        }
     }
 }
 
@@ -3835,7 +3843,9 @@ void SettingsDialog::on_exportSettingsButton_clicked() {
             QSettings settings;
 
             const QStringList keys = settings.allKeys();
-            Q_FOREACH (QString key, keys) { exportSettings.setValue(key, settings.value(key)); }
+            Q_FOREACH (QString key, keys) {
+                exportSettings.setValue(key, settings.value(key));
+            }
         }
     }
 }
diff --git a/src/dialogs/tododialog.cpp b/src/dialogs/tododialog.cpp
index 5e5337892f..5805f15801 100644
--- a/src/dialogs/tododialog.cpp
+++ b/src/dialogs/tododialog.cpp
@@ -661,7 +661,9 @@ void TodoDialog::on_newItemEdit_textChanged() {
         }
     } else {
         // show all items otherwise
-        Q_FOREACH (QTreeWidgetItem *item, allItems) { item->setHidden(false); }
+        Q_FOREACH (QTreeWidgetItem *item, allItems) {
+            item->setHidden(false);
+        }
     }
 
     // let's highlight the text from the search line edit
diff --git a/src/helpers/toolbarcontainer.cpp b/src/helpers/toolbarcontainer.cpp
index 74647433a5..2aa16d59bb 100644
--- a/src/helpers/toolbarcontainer.cpp
+++ b/src/helpers/toolbarcontainer.cpp
@@ -12,8 +12,7 @@
 
 ToolbarContainer::ToolbarContainer(QToolBar *toolbar)
     : name(toolbar->objectName()), title(toolbar->windowTitle()) {
-    foreach (QAction *a, toolbar->actions())
-        actions.push_back(a->objectName());
+    foreach (QAction *a, toolbar->actions()) actions.push_back(a->objectName());
 }
 
 QToolBar *ToolbarContainer::create(QMainWindow *w) const {
diff --git a/src/mainwindow.cpp b/src/mainwindow.cpp
index 9a986f0bd0..46283bc761 100644
--- a/src/mainwindow.cpp
+++ b/src/mainwindow.cpp
@@ -48,7 +48,6 @@
 #include <widgets/notetreewidgetitem.h>
 
 #include <QAbstractEventDispatcher>
-#include <QtConcurrent>
 #include <QActionGroup>
 #include <QClipboard>
 #include <QColorDialog>
@@ -91,6 +90,7 @@
 #include <QTreeWidgetItem>
 #include <QUuid>
 #include <QWidgetAction>
+#include <QtConcurrent>
 #include <libraries/qttoolbareditor/src/toolbar_editor.hpp>
 #include <memory>
 #include <utility>
@@ -508,8 +508,10 @@ void MainWindow::initTreeWidgets() {
     connect(ui->noteSubFolderTreeWidget, &NoteSubFolderTree::currentSubFolderChanged, this,
             &MainWindow::onCurrentSubFolderChanged);
     if (NoteFolder::isCurrentNoteTreeEnabled()) {
-        connect(ui->noteTreeWidget, &QTreeWidget::itemExpanded, ui->noteSubFolderTreeWidget, &NoteSubFolderTree::onItemExpanded);
-        connect(ui->noteTreeWidget, &QTreeWidget::itemCollapsed, ui->noteSubFolderTreeWidget, &NoteSubFolderTree::onItemExpanded);
+        connect(ui->noteTreeWidget, &QTreeWidget::itemExpanded, ui->noteSubFolderTreeWidget,
+                &NoteSubFolderTree::onItemExpanded);
+        connect(ui->noteTreeWidget, &QTreeWidget::itemCollapsed, ui->noteSubFolderTreeWidget,
+                &NoteSubFolderTree::onItemExpanded);
     }
 }
 
@@ -640,7 +642,9 @@ void MainWindow::triggerStartupMenuAction() {
  */
 void MainWindow::initGlobalKeyboardShortcuts() {
     // deleting old global shortcut assignments
-    foreach (QHotkey *hotKey, _globalShortcuts) { delete hotKey; }
+    foreach (QHotkey *hotKey, _globalShortcuts) {
+        delete hotKey;
+    }
 
     _globalShortcuts.clear();
     QSettings settings;
@@ -6788,8 +6792,7 @@ void MainWindow::storeNoteBookmark(int slot) {
     NoteHistoryItem item = NoteHistoryItem(&currentNote, ui->noteTextEdit);
     noteBookmarks[slot] = item;
 
-    QSettings().setValue(QStringLiteral("NoteBookmark%1").arg(slot),
-                         QVariant::fromValue(item));
+    QSettings().setValue(QStringLiteral("NoteBookmark%1").arg(slot), QVariant::fromValue(item));
 
     showStatusBarMessage(tr("Bookmarked note position at slot %1").arg(QString::number(slot)),
                          3000);
@@ -10966,7 +10969,7 @@ void MainWindow::automaticScriptUpdateCheck() {
     // We need to do that in a slot, because you can't use a timer in a separate thread
     QObject::connect(dialog, &ScriptRepositoryDialog::noUpdateFound, this, [this, dialog]() {
         showStatusBarMessage(tr("No script updates were found"), 3000);
-        delete(dialog);
+        delete (dialog);
     });
 
     // Show a dialog once if a script update was found
@@ -10978,7 +10981,7 @@ void MainWindow::automaticScriptUpdateCheck() {
 
         _scriptUpdateFound = true;
         showStatusBarMessage(tr("A script update was found!"), 4000);
-        delete(dialog);
+        delete (dialog);
 
         if (Utils::Gui::question(this, tr("Script updates"),
                                  tr("Updates to your scripts were found in the script "
@@ -10989,8 +10992,8 @@ void MainWindow::automaticScriptUpdateCheck() {
     });
 
     // Search for script updates in the background after the "updateFound" signal was connected
-    // We must not do a `delete (dialog)` in the new thread, that was causing a crash on some systems
-    // See https://github.com/pbek/QOwnNotes/issues/2937
+    // We must not do a `delete (dialog)` in the new thread, that was causing a crash on some
+    // systems, see https://github.com/pbek/QOwnNotes/issues/2937
     QFuture<void> future = QtConcurrent::run([this, dialog, scripts]() {
         qDebug() << "start searchForUpdates";
         dialog->searchForUpdatesForScripts(scripts);
diff --git a/src/services/scriptingservice.cpp b/src/services/scriptingservice.cpp
index 87c9de0073..15dddd1163 100644
--- a/src/services/scriptingservice.cpp
+++ b/src/services/scriptingservice.cpp
@@ -1950,7 +1950,9 @@ QList<int> ScriptingService::selectedNotesIds() const {
     MainWindow *mainWindow = MainWindow::instance();
 
     if (mainWindow != nullptr) {
-        Q_FOREACH (Note note, mainWindow->selectedNotes()) { selectedNotesIds << note.getId(); }
+        Q_FOREACH (Note note, mainWindow->selectedNotes()) {
+            selectedNotesIds << note.getId();
+        }
     }
 #endif
 
diff --git a/src/utils/gui.cpp b/src/utils/gui.cpp
index 4900f840e5..879088b153 100644
--- a/src/utils/gui.cpp
+++ b/src/utils/gui.cpp
@@ -123,7 +123,9 @@ void Utils::Gui::searchForTextInTreeWidget(QTreeWidget *treeWidget, const QStrin
         }
     } else {
         // show all items otherwise
-        Q_FOREACH (QTreeWidgetItem *item, allItems) { item->setHidden(false); }
+        Q_FOREACH (QTreeWidgetItem *item, allItems) {
+            item->setHidden(false);
+        }
     }
 }
 
@@ -150,7 +152,9 @@ void Utils::Gui::searchForTextInListWidget(QListWidget *listWidget, const QStrin
         }
     } else {
         // show all items otherwise
-        Q_FOREACH (QListWidgetItem *item, allItems) { item->setHidden(false); }
+        Q_FOREACH (QListWidgetItem *item, allItems) {
+            item->setHidden(false);
+        }
     }
 }
 
diff --git a/src/utils/urlhandler.cpp b/src/utils/urlhandler.cpp
index 21dd88a0eb..ff8f706b99 100644
--- a/src/utils/urlhandler.cpp
+++ b/src/utils/urlhandler.cpp
@@ -114,9 +114,10 @@ void UrlHandler::handleNoteUrl(QString urlString, const QString &fragment) {
         if (!fragment.isEmpty()) {
             // Search with a regular expression for the fragment to make sure
             // we are searching for the full heading
-            auto searchTerm = QStringLiteral("## ") + QRegularExpression::escape(fragment) +
-                              QStringLiteral("$");
-            mw->activeNoteTextEdit()->doSearch(searchTerm, QPlainTextEditSearchWidget::RegularExpressionMode);
+            auto searchTerm =
+                QStringLiteral("## ") + QRegularExpression::escape(fragment) + QStringLiteral("$");
+            mw->activeNoteTextEdit()->doSearch(searchTerm,
+                                               QPlainTextEditSearchWidget::RegularExpressionMode);
             mw->activeNoteTextEdit()->searchWidget()->deactivate();
         }
     } else {
diff --git a/src/widgets/logwidget.cpp b/src/widgets/logwidget.cpp
index 62b8dc4c89..d90d66735f 100644
--- a/src/widgets/logwidget.cpp
+++ b/src/widgets/logwidget.cpp
@@ -141,7 +141,9 @@ void LogWidget::log(LogWidget::LogType logType, const QString &text) {
          text.contains(QLatin1String("[Botan Error]  Invalid CBC padding")) ||
          text.contains(QLatin1String("Invalid version or not a cyphertext")) ||
          text.contains(QLatin1String("QTextCursor::setPosition: Position")) ||
-         text.contains(QLatin1String("QFont::setPointSizeF: Point size <= 0")) ||   // we don't even use that method directly
+         text.contains(
+             QLatin1String("QFont::setPointSizeF: Point size <= 0")) ||    // we don't even use that
+                                                                           // method directly
          text.contains(QLatin1String(
              "scroll event from unregistered device")) ||    // when YubiKey is plugged in or out
          text.contains(QLatin1String("QFileSystemWatcher::removePaths: list is empty")))) {
diff --git a/src/widgets/passwordlineedit.cpp b/src/widgets/passwordlineedit.cpp
index 137a66491f..0ad828e751 100644
--- a/src/widgets/passwordlineedit.cpp
+++ b/src/widgets/passwordlineedit.cpp
@@ -12,15 +12,16 @@
  */
 
 #include "passwordlineedit.h"
+
 #include <QAction>
 
 PasswordLineEdit::PasswordLineEdit(QWidget *parent) : QLineEdit(parent) {
     setEchoMode(QLineEdit::Password);
 
-    static const auto visibleIcon = QIcon::fromTheme(QStringLiteral("view-visible"),
-                                                     QIcon(":/icons/breeze-qownnotes/16x16/view-visible.svg"));
-    static const auto hiddenIcon = QIcon::fromTheme(QStringLiteral("view-hidden"),
-                                                     QIcon(":/icons/breeze-qownnotes/16x16/view-hidden.svg"));
+    static const auto visibleIcon = QIcon::fromTheme(
+        QStringLiteral("view-visible"), QIcon(":/icons/breeze-qownnotes/16x16/view-visible.svg"));
+    static const auto hiddenIcon = QIcon::fromTheme(
+        QStringLiteral("view-hidden"), QIcon(":/icons/breeze-qownnotes/16x16/view-hidden.svg"));
     static const auto visibleText = tr("Show password");
     static const auto hiddenText = tr("Hide password");