This revision was not accepted when it landed; it landed in state "Needs Review". This revision was automatically updated to reflect the committed changes. Closed by commit rL367968: [NFC][DirectoryWatchedTests] Unlocks mutexes before signaling condition variable (authored by zer0, committed by ). Herald added a project: LLVM. Herald added a subscriber: llvm-commits.
Changed prior to commit: https://reviews.llvm.org/D65708?vs=213220&id=213510#toc Repository: rL LLVM CHANGES SINCE LAST ACTION https://reviews.llvm.org/D65708/new/ https://reviews.llvm.org/D65708 Files: cfe/trunk/unittests/DirectoryWatcher/DirectoryWatcherTest.cpp Index: cfe/trunk/unittests/DirectoryWatcher/DirectoryWatcherTest.cpp =================================================================== --- cfe/trunk/unittests/DirectoryWatcher/DirectoryWatcherTest.cpp +++ cfe/trunk/unittests/DirectoryWatcher/DirectoryWatcherTest.cpp @@ -132,8 +132,10 @@ } else { ExpectedInitial.erase(It); } - if (result()) + if (result()) { + L.unlock(); ResultIsReady.notify_one(); + } } void consumeNonInitial(DirectoryWatcher::Event E) { @@ -151,8 +153,10 @@ } else { ExpectedNonInitial.erase(It); } - if (result()) + if (result()) { + L.unlock(); ResultIsReady.notify_one(); + } } // This method is used by DirectoryWatcher.
Index: cfe/trunk/unittests/DirectoryWatcher/DirectoryWatcherTest.cpp =================================================================== --- cfe/trunk/unittests/DirectoryWatcher/DirectoryWatcherTest.cpp +++ cfe/trunk/unittests/DirectoryWatcher/DirectoryWatcherTest.cpp @@ -132,8 +132,10 @@ } else { ExpectedInitial.erase(It); } - if (result()) + if (result()) { + L.unlock(); ResultIsReady.notify_one(); + } } void consumeNonInitial(DirectoryWatcher::Event E) { @@ -151,8 +153,10 @@ } else { ExpectedNonInitial.erase(It); } - if (result()) + if (result()) { + L.unlock(); ResultIsReady.notify_one(); + } } // This method is used by DirectoryWatcher.
_______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits