[all-commits] [llvm/llvm-project] d08563: [clangd] Make Notification a little safer.
Sam McCall via All-commits
all-commits at lists.llvm.org
Sat Jan 25 06:32:31 PST 2020
Branch: refs/heads/master
Home: https://github.com/llvm/llvm-project
Commit: d08563486e06df3ddb4d7c1018d1e1e762690ee8
https://github.com/llvm/llvm-project/commit/d08563486e06df3ddb4d7c1018d1e1e762690ee8
Author: Sam McCall <sam.mccall at gmail.com>
Date: 2020-01-25 (Sat, 25 Jan 2020)
Changed paths:
M clang-tools-extra/clangd/Threading.cpp
Log Message:
-----------
[clangd] Make Notification a little safer.
I just fixed a test involving a similar Notification class: 18e6a65bae93a
The pattern (notify() on one thread, wait() and then destroy the Notification
on the other) seems innocuous enough. I'm not sure we actually use it in clangd,
but better safe than sorry.
More information about the All-commits
mailing list