fix: review comments

Author: DeepDiver1975

src/common/vfs.cpp

@@ -228,8 +228,6 @@ Vfs::Mode OCC::VfsPluginManager::bestAvailableVfsMode() const
 {
     if (isVfsPluginAvailable(Vfs::WindowsCfApi)) {
         return Vfs::WindowsCfApi;
-    } else if (isVfsPluginAvailable(Vfs::Off)) {
-        return Vfs::Off;
     }
 
     return Vfs::Off;

src/csync/csync_exclude.cpp

@@ -393,32 +393,33 @@ CSYNC_EXCLUDE_TYPE ExcludedFiles::traversalPatternMatch(QStringView path, ItemTy
         bnameStr = path.mid(lastSlash + 1);
     }
 
-    QRegularExpressionMatch m;
+    QRegularExpressionMatch regularExpressionMatch;
     if (filetype == ItemTypeDirectory) {
-        m = _bnameTraversalRegexDir.matchView(bnameStr);
+        regularExpressionMatch = _bnameTraversalRegexDir.matchView(bnameStr);
     } else {
-        m = _bnameTraversalRegexFile.matchView(bnameStr);
+        regularExpressionMatch = _bnameTraversalRegexFile.matchView(bnameStr);
     }
-    if (!m.hasMatch())
+    if (!regularExpressionMatch.hasMatch())
         return CSYNC_NOT_EXCLUDED;
-    if (m.capturedStart(QStringLiteral("exclude")) != -1) {
+    if (regularExpressionMatch.capturedStart(QStringLiteral("exclude")) != -1) {
         return CSYNC_FILE_EXCLUDE_LIST;
-    } else if (m.capturedStart(QStringLiteral("excluderemove")) != -1) {
+    } else if (regularExpressionMatch.capturedStart(QStringLiteral("excluderemove")) != -1) {
         return CSYNC_FILE_EXCLUDE_AND_REMOVE;
     }
 
     // third capture: full path matching is triggered
     QStringView pathStr = path;
 
     if (filetype == ItemTypeDirectory) {
-        m = _fullTraversalRegexDir.matchView(pathStr);
+        regularExpressionMatch = _fullTraversalRegexDir.matchView(pathStr);
     } else {
-        m = _fullTraversalRegexFile.matchView(pathStr);
+        regularExpressionMatch = _fullTraversalRegexFile.matchView(pathStr);
     }
-    if (m.hasMatch()) {
-        if (m.capturedStart(QStringLiteral("exclude")) != -1) {
+    if (regularExpressionMatch.hasMatch()) {
+        if (regularExpressionMatch.capturedStart(QStringLiteral("exclude")) != -1) {
             return CSYNC_FILE_EXCLUDE_LIST;
-        } else if (m.capturedStart(QStringLiteral("excluderemove")) != -1) {
+        }
+        if (regularExpressionMatch.capturedStart(QStringLiteral("excluderemove")) != -1) {
             return CSYNC_FILE_EXCLUDE_AND_REMOVE;
         }
     }

src/gui/folder.cpp

@@ -540,7 +540,7 @@ void Folder::startVfs()
         connect(_vfs.get(), &Vfs::needSync, this, [this] {
             if (canSync()) {
                 // the vfs plugin detected that its metadata is out of sync and requests a new sync
-                // the request has a height priority as it is probably issued after a user request
+                // the request has a higher priority as it is probably issued after a user request
                 FolderMan::instance()->scheduler()->enqueueFolder(this, SyncScheduler::Priority::High);
             }
         });

src/gui/main.cpp

@@ -397,7 +397,7 @@ int main(int argc, char **argv)
         // load the resources
         const OCC::ResourcesLoader resource;
 
-        // Create a `Platform` instance so it can set up/teared down stuff for us, and do any
+        // Create a `Platform` instance so it can set up/tear down stuff for us, and do any
         // initialisation that needs to be done before creating a QApplication
         const auto platform = Platform::create();
 

src/gui/socketapi/socketapi.cpp

@@ -925,11 +925,6 @@ SocketApi::FileData SocketApi::FileData::get(const QString &localFile)
     return data;
 }
 
-QString SocketApi::FileData::folderRelativePathNoVfsSuffix() const
-{
-    return folderRelativePath;
-}
-
 bool OCC::SocketApi::FileData::isSyncFolder() const
 {
     return folderRelativePath.isEmpty();

src/gui/socketapi/socketapi.h

@@ -86,13 +86,10 @@ private Q_SLOTS:
         SyncJournalFileRecord journalRecord() const;
         FileData parentFolder() const;
 
-        // Relative path of the file locally, without any vfs suffix
-        QString folderRelativePathNoVfsSuffix() const;
-
         Folder *folder = nullptr;
-        // Absolute path of the file locally. (May be a virtual file)
+        // Absolute path of the file locally. (Maybe a virtual file)
         QString localPath;
-        // Relative path of the file locally, as in the DB. (May be a virtual file)
+        // Relative path of the file locally, as in the DB. (Maybe a virtual file)
         QString folderRelativePath;
         // Path of the file on the server (In case of virtual file, it points to the actual file)
         QString serverRelativePath;