[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