Log directories skipped by watcher

Author: TheAssassin

src/daemon/main.cpp

@@ -62,6 +62,11 @@ void initialSearchForAppImages(const QDirSet& dirsToSearch, Worker& worker) {
     std::cout << "Searching for existing AppImages" << std::endl;
 
     for (const auto& dir : dirsToSearch) {
+        if (!dir.exists()) {
+            std::cout << "Directory " << dir.path().toStdString() << " does not exist, skipping" << std::endl;
+            continue;
+        }
+
         std::cout << "Searching directory: " << dir.absolutePath().toStdString() << std::endl;
 
         for (QDirIterator it(dir); it.hasNext();) {

src/fswatcher/filesystemwatcher.cpp

@@ -288,13 +288,11 @@ bool FileSystemWatcher::updateWatchedDirectories(QDirSet watchedDirectories) {
     // so when they'll be created, we'll notice
     {
         // erase-remove doesn't work with sets apparently (see https://stackoverflow.com/a/26833313)
-        // therefore we use a simple custom algorithm
-        auto it = watchedDirectories.begin();
-        while (it != watchedDirectories.end()) {
+        // therefore we use a simple linear search to remove non-existing directories
+        for (auto it = watchedDirectories.begin(); it != watchedDirectories.end(); ++it) {
             if (!it->exists()) {
+                std::cout << "Directory " << it->path().toStdString() << " does not exist, skipping" << std::endl;
                 it = watchedDirectories.erase(it);
-            } else {
-                ++it;
             }
         }
     }