[PATCH] D84924: [clang-tidy][WIP] Added command line option `fix-notes`

njames93 updated this revision to Diff 313080. njames93 marked 2 inline comments as done. njames93 added a comment. Use enum for handling fix behaviour. Fix tests using `--fix-notes` instead of `-fix-notes`. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION

[PATCH] D84924: [clang-tidy][WIP] Added command line option `fix-notes`

aaron.ballman added a comment. In D84924#2421116 , @njames93 wrote: > Fix documentation. > > Ping?? Ah, sorry, this fell to the bottom of my review list because the title still says [WIP] and so I thought there was more work being done on it. You may

[PATCH] D84924: [clang-tidy][WIP] Added command line option `fix-notes`

njames93 updated this revision to Diff 308153. njames93 added a comment. Fix documentation. Ping?? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D84924/new/ https://reviews.llvm.org/D84924 Files: clang-tools-extra/clang-tidy/ClangTidy.cpp

[PATCH] D84924: [clang-tidy][WIP] Added command line option `fix-notes`

njames93 updated this revision to Diff 304145. njames93 marked 2 inline comments as done. njames93 added a comment. Made `-fix-notes` imply `-fix` Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D84924/new/ https://reviews.llvm.org/D84924 Files:

[PATCH] D84924: [clang-tidy][WIP] Added command line option `fix-notes`

njames93 marked 2 inline comments as done. njames93 added inline comments. Comment at: clang-tools-extra/clang-tidy/ClangTidy.cpp:109 SourceMgr(Diags, Files), Context(Context), ApplyFixes(ApplyFixes), -TotalFixes(0), AppliedFixes(0), WarningsAsErrors(0) { +

[PATCH] D84924: [clang-tidy][WIP] Added command line option `fix-notes`

Eugene.Zelenko added inline comments. Comment at: clang-tools-extra/clang-tidy/ClangTidy.cpp:109 SourceMgr(Diags, Files), Context(Context), ApplyFixes(ApplyFixes), -TotalFixes(0), AppliedFixes(0), WarningsAsErrors(0) { +ApplyAnyFix(ApplyAnyFix),

[PATCH] D84924: [clang-tidy][WIP] Added command line option `fix-notes`

njames93 added a comment. This is very much a work in progress Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D84924/new/ https://reviews.llvm.org/D84924 ___ cfe-commits mailing list