vim/YouCompleteMe
1
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

Only joining threads if they are created

Browse Source
This commit is contained in:
Strahinja Val Markovic 2013-02-16 12:07:20 -08:00
parent 30d5a3b8a5
commit 2ff85a5a60
2 changed files with 8 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
cpp/ycm/ClangCompleter/ClangCompleter.cpp
View File

@ -101,9 +101,11 @@ ClangCompleter::~ClangCompleter() {
sorting_threads_.interrupt_all(); sorting_threads_.interrupt_all();
sorting_threads_.join_all(); sorting_threads_.join_all();
if ( clang_thread_ ) {
clang_thread_->interrupt(); clang_thread_->interrupt();
clang_thread_->join(); clang_thread_->join();
} }
}
// We need this mostly so that we can not use it in tests. Apparently the // We need this mostly so that we can not use it in tests. Apparently the

2
cpp/ycm/IdentifierCompleter.cpp
View File

@ -103,9 +103,11 @@ IdentifierCompleter::~IdentifierCompleter() {
query_threads_.interrupt_all(); query_threads_.interrupt_all();
query_threads_.join_all(); query_threads_.join_all();
if ( buffer_identifiers_thread_ ) {
buffer_identifiers_thread_->interrupt(); buffer_identifiers_thread_->interrupt();
buffer_identifiers_thread_->join(); buffer_identifiers_thread_->join();
} }
}
// We need this mostly so that we can not use it in tests. Apparently the // We need this mostly so that we can not use it in tests. Apparently the