This revision was automatically updated to reflect the committed changes.
Closed by commit rL319785: [clang-tidy] adjust cppcoreguidelines-owning-memory
documentation (authored by JonasToth).
Repository:
rL LLVM
https://reviews.llvm.org/D40829
Files:
JonasToth updated this revision to Diff 125536.
JonasToth added a comment.
- [Fix] typo
Repository:
rCTE Clang Tools Extra
https://reviews.llvm.org/D40829
Files:
docs/clang-tidy/checks/cppcoreguidelines-owning-memory.rst
Index: docs/clang-tidy/checks/cppcoreguidelines-owning-memory.rst
JonasToth added a comment.
I agree.
Repository:
rCTE Clang Tools Extra
https://reviews.llvm.org/D40829
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
aaron.ballman accepted this revision.
aaron.ballman added a comment.
This revision is now accepted and ready to land.
This is reasonable, though if the list starts getting longer, we should
consider trying to fix the bug rather than enumerate them.
Comment at:
JonasToth created this revision.
Herald added subscribers: cfe-commits, kbarton, xazax.hun, nemanjai, klimek.
A user of the check opened a bugreport and reported that `std::exchange`
triggers a false positive. I adjusted the doc to include a list of known
(std) constructs that do trigger the