clazy: fix warnings

1 files changed, 4 insertions, 12 deletions

diff --git a/src/mainwindow/mainwindow.cpp b/src/mainwindow/mainwindow.cpp
index 8a33e23..b3a7722 100644
--- a/src/mainwindow/mainwindow.cpp
+++ b/src/mainwindow/mainwindow.cpp
@@ -89,16 +89,6 @@ MainWindow::MainWindow(const std::unique_ptr<Configuration> &config, QWidget *pa
     }
     show();
 
-#ifdef QT_DEBUG
-    {
-        auto *debugMenu = ui->menubar->addMenu(tr("Debug"));
-
-        debugMenu->addAction(tr("Crash"), []() {
-            delete reinterpret_cast<QString*>(0xFEE1DEAD);
-        });
-    };
-#endif
-
     // connect smolbote menu
     {
         connect(ui->actionNewSubwindow, &QAction::triggered, this, [this, &config]() {
@@ -293,7 +283,8 @@ void MainWindow::addAction(ActionLocation where, QAction *action)
 void MainWindow::addDockWidget(Qt::DockWidgetArea area, QWidget *widget)
 {
     QDockWidget *lastDock = nullptr;
-    for(QDockWidget *dock : findChildren<QDockWidget *>()) {
+    const auto docks = findChildren<QDockWidget *>();
+    for(QDockWidget *dock : docks) {
         if(dockWidgetArea(dock) == area)
             lastDock = dock;
     }
@@ -334,7 +325,8 @@ void MainWindow::createTab(const QUrl &url)
 const QVector<SubWindow *> MainWindow::subWindows() const
 {
     QVector<SubWindow *> list;
-    for(auto *w : mdiArea->subWindowList()) {
+    const auto subwindows = mdiArea->subWindowList();
+    for(auto *w : subwindows) {
         auto *subwindow = qobject_cast<SubWindow *>(w);
         if(subwindow != nullptr)
             list.append(subwindow);