changes made

Signed-off-by: Vinayakjeet Singh Karki <139736674+vinayakjeet@users.noreply.github.com>

Author: vinayakjeet

.github/workflows/codacy.yml

@@ -53,13 +53,17 @@ jobs:
           # This will handover control about PR rejection to the GitHub side
           max-allowed-issues: 2147483647
 
-      - name: Filter out MISRA
+      # Filter out MISRA and potentially reduce the number of runs in the SARIF output
+      - name: Filter and Reduce SARIF
         run: |
           pip install globber
-          python3 scripts/filter_sarif.py --input results.sarif --output filtered.sarif --split-lines -- "-**/*.*:cppcheck_misra*"
+          python3 scripts/filter_and_reduce_sarif.py --input results.sarif --output filtered_and_reduced.sarif
+        env:
+          PYTHONUNBUFFERED: 1
 
-      # Upload the SARIF file generated in the previous step
+
+       # Upload the SARIF file generated in the previous step
       - name: Upload SARIF results file
         uses: github/codeql-action/upload-sarif@v3
         with:
-          sarif_file: filtered.sarif
+          sarif_file: filtered_and_reduced.sarif

avogadro/mainwindow.cpp

@@ -1980,41 +1980,45 @@ void MainWindow::buildMenu()
   m_fileToolBar->addAction(action);
   connect(action, SIGNAL(triggered()), SLOT(saveFileAs()));
   // Initialize autosave feature
-m_autosaveInterval = 5; // Autosave interval in minutes
-m_autosaveTimer = new QTimer(this);
-connect(m_autosaveTimer, &QTimer::timeout, this, &MainWindow::autosaveDocument);
-m_autosaveTimer->start(m_autosaveInterval * 60000); // Convert minutes to milliseconds
-
-void MainWindow::autosaveDocument()
-{
+  m_autosaveInterval = 5; // Autosave interval in minutes
+  m_autosaveTimer = new QTimer(this);
+  connect(m_autosaveTimer, &QTimer::timeout, this,
+          &MainWindow::autosaveDocument);
+  m_autosaveTimer->start(m_autosaveInterval *
+                         60000); // Convert minutes to milliseconds
+
+  void MainWindow::autosaveDocument()
+  {
     if (!m_molecule || !m_moleculeDirty) {
-        return; // No molecule loaded or no changes made since the last save.
+      return; // No molecule loaded or no changes made since the last save.
     }
 
-    QString autosaveDirPath = QStandardPaths::writableLocation(QStandardPaths::AppDataLocation) + "/autosave";
+    QString autosaveDirPath =
+      QStandardPaths::writableLocation(QStandardPaths::AppDataLocation) +
+      "/autosave";
     QDir autosaveDir(autosaveDirPath);
     if (!autosaveDir.exists()) {
-        autosaveDir.mkpath(".");
+      autosaveDir.mkpath(".");
     }
 
     // Construct autosave file name
     QString autosaveFilename;
     if (m_molecule->hasData("fileName")) {
-        QFileInfo fileInfo(m_molecule->data("fileName").toString().c_str());
-        autosaveFilename = fileInfo.baseName() + "_autosave.cjson";
+      QFileInfo fileInfo(m_molecule->data("fileName").toString().c_str());
+      autosaveFilename = fileInfo.baseName() + "_autosave.cjson";
     } else {
-        autosaveFilename = "unsaved_autosave.cjson";
+      autosaveFilename = "unsaved_autosave.cjson";
     }
     QString autosaveFilePath = autosaveDirPath + "/" + autosaveFilename;
 
     // Use CJSON format for autosaving
     Io::CjsonFormat writer;
     if (!writer.writeFile(autosaveFilePath, *m_molecule)) {
-        qWarning() << "Failed to autosave the document to" << autosaveFilePath;
+      qWarning() << "Failed to autosave the document to" << autosaveFilePath;
     } else {
-        qDebug() << "Document autosaved to" << autosaveFilePath;
+      qDebug() << "Document autosaved to" << autosaveFilePath;
     }
-}
+  }
 
   // Export action for menu
   QStringList exportPath = path;

avogadro/mainwindow.h

@@ -69,7 +69,7 @@ class MainWindow : public QMainWindow
 
 public slots:
   void setMolecule(Avogadro::QtGui::Molecule* molecule);
-  void autosaveDocument(); //line to declare the autosave slot
+  void autosaveDocument(); // line to declare the autosave slot
   /**
    * Update internal state to reflect that the molecule has been modified.
    */
@@ -391,16 +391,15 @@ private slots:
   void setProjectionPerspective();
 
 private:
-
   QtGui::Molecule* m_molecule;
   QtGui::RWMolecule* m_rwMolecule;
   QtGui::MoleculeModel* m_moleculeModel;
   QtGui::LayerModel* m_layerModel;
   QtGui::ScenePlugin* m_activeScenePlugin;
   bool m_queuedFilesStarted;
   QStringList m_queuedFiles;
-  QTimer* m_autosaveTimer;  // for the autosave timer
-  int m_autosaveInterval;   // for autosave interval in minutes
+  QTimer* m_autosaveTimer; // for the autosave timer
+  int m_autosaveInterval;  // for autosave interval in minutes
   QStringList m_recentFiles;
   QList<QAction*> m_actionRecentFiles;