[PATCH] D36051: Move from a long list of checkers to tables

2020-01-03 Thread Alexander Kornienko via Phabricator via cfe-commits
alexfh added a comment. This list used to be automatically updated by the clang-tools-extra/clang-tidy/add_new_check.py script. I suppose, this is broken now? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D36051/new/ https://reviews.llvm.org/D3605

[PATCH] D36051: Move from a long list of checkers to tables

2019-12-31 Thread Sylvestre Ledru via Phabricator via cfe-commits
sylvestre.ledru added a comment. Thanks. Fixed in https://github.com/llvm/llvm-project/commit/e8c9110b56b516a22b41e95e347bc141814ab87c Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D36051/new/ https://reviews.llvm.org/D36051

[PATCH] D36051: Move from a long list of checkers to tables

2019-12-31 Thread Jonas Toth via Phabricator via cfe-commits
JonasToth added a comment. In D36051#1799785 , @sylvestre.ledru wrote: > @malcolm.parsons really? Oh, where is it implemented? > > https://github.com/llvm/llvm-project/blob/61504079515f76ca094bb836c4d53b41064220d6/clang-tools-extra/clang-tidy/modernize

[PATCH] D36051: Move from a long list of checkers to tables

2019-12-31 Thread Sylvestre Ledru via Phabricator via cfe-commits
sylvestre.ledru added a comment. @malcolm.parsons really? Oh, where is it implemented? https://github.com/llvm/llvm-project/blob/61504079515f76ca094bb836c4d53b41064220d6/clang-tools-extra/clang-tidy/modernize/MakeUniqueCheck.cpp I didn't see it here Repository: rG LLVM Github Monorepo CHANGE

[PATCH] D36051: Move from a long list of checkers to tables

2019-12-31 Thread Malcolm Parsons via Phabricator via cfe-commits
malcolm.parsons added inline comments. Comment at: clang-tools-extra/docs/clang-tidy/checks/list.rst:205 + `modernize-make-shared `_, "Yes", "low" + `modernize-make-unique `_, , "low" + `modernize-pass-by-value `_, "Yes", "low" modernize-make-unique offers

[PATCH] D36051: Move from a long list of checkers to tables

2019-12-23 Thread Sylvestre Ledru via Phabricator via cfe-commits
This revision was not accepted when it landed; it landed in state "Needs Review". This revision was automatically updated to reflect the committed changes. Closed by commit rGd2c9c9157b05: Move from a long list of checkers to tables (authored by sylvestre.ledru). Repository: rG LLVM Github Mon