]> git.saurik.com Git - wxWidgets.git/commitdiff
Don't give error message for non-existent paths in wxFileSystemWatcher.
authorVadim Zeitlin <vadim@wxwidgets.org>
Sat, 2 Mar 2013 12:11:40 +0000 (12:11 +0000)
committerVadim Zeitlin <vadim@wxwidgets.org>
Sat, 2 Mar 2013 12:11:40 +0000 (12:11 +0000)
The file we're trying to watch might have been just deleted, this is an
unavoidable (and not only in theory but also in practice when watching busy
directories) race condition so don't give an error message if this does happen
but simply return false from Add().

Closes #15051.

git-svn-id: https://svn.wxwidgets.org/svn/wx/wxWidgets/trunk@73591 c3d73ce0-8a6f-49c7-b76d-6d57e0e08775

src/common/fswatchercmn.cpp

index ce36aa2783d5e7267afd0a11e77a70798102ea7d..51d9d578f8687052ea57d931772ae3116c691586 100644 (file)
@@ -100,7 +100,10 @@ bool wxFileSystemWatcherBase::Add(const wxFileName& path, int events)
     }
     else
     {
-        wxLogError(_("Can't monitor non-existent path \"%s\" for changes."),
+        // Don't overreact to being passed a non-existent item. It may have
+        // only just been deleted, in which case doing nothing is correct
+        wxLogTrace(wxTRACE_FSWATCHER,
+                   "Can't monitor non-existent path \"%s\" for changes.",
                    path.GetFullPath());
         return false;
     }