amccarth accepted this revision. amccarth added a comment. This revision is now accepted and ready to land.
LGTM. Thanks for extending this functionality to Windows! ================ Comment at: clang/lib/DirectoryWatcher/windows/DirectoryWatcher-windows.cpp:15 +#include "llvm/Support/Windows/WindowsSupport.h" #include <atomic> #include <condition_variable> ---------------- I don't see a reason to include `<atomic>` here. ================ Comment at: clang/lib/DirectoryWatcher/windows/DirectoryWatcher-windows.cpp:77 + void WatcherThreadProc(HANDLE DirectoryHandle); + void NotifierThreadProc(bool WaitForInitialSync); }; ---------------- I like the name change from HandlerThread to NotifierThread. Thanks! CHANGES SINCE LAST ACTION https://reviews.llvm.org/D88666/new/ https://reviews.llvm.org/D88666 _______________________________________________ cfe-commits mailing list [email protected] https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
