This revision was automatically updated to reflect the committed changes.
Closed by commit rL364799: __threading_support: Remove (void) in favor of ().
(authored by brucem, committed by ).
Herald added a project: LLVM.
Herald added a subscriber: llvm-commits.
Repository:
rL LLVM
CHANGES SINCE
mclow.lists accepted this revision.
mclow.lists added a comment.
This revision is now accepted and ready to land.
This looks fine to me.
Repository:
rCXX libc++
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D43226/new/
https://reviews.llvm.org/D43226
___
brucem added a comment.
Herald added subscribers: libcxx-commits, ldionne.
Can we revive this review? I'd still like to land it ...
Repository:
rCXX libc++
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D43226/new/
https://reviews.llvm.org/D43226
brucem added a comment.
This is `modernize-redundant-void-arg`:
https://clang.llvm.org/extra/clang-tidy/checks/modernize-redundant-void-arg.html
Repository:
rCXX libc++
https://reviews.llvm.org/D43226
___
cfe-commits mailing list
cfe-commits@lis
mclow.lists added a comment.
Herald added a subscriber: christof.
Which clang-tidy module generates this warning, and what is the actual warning?
Repository:
rCXX libc++
https://reviews.llvm.org/D43226
___
cfe-commits mailing list
cfe-commits@lis
brucem created this revision.
brucem added reviewers: mclow.lists, EricWF.
This fixes a clang-tidy warning when building something that uses
this file.
Repository:
rCXX libc++
https://reviews.llvm.org/D43226
Files:
include/__threading_support
Index: include/__threading_support
==