[PATCH] D77983: clang-tidy doc: add a note for every checker with an autofix

2020-05-10 Thread Sylvestre Ledru via Phabricator via cfe-commits
sylvestre.ledru reopened this revision. sylvestre.ledru added a comment. has been reverted Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D77983/new/ https://reviews.llvm.org/D77983 ___ cfe-commits maili

[PATCH] D77983: clang-tidy doc: add a note for every checker with an autofix

2020-05-10 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 rGdc0f79ea5b99: clang-tidy doc: add a note for checkers with an autofix (authored by sylvestre.ledru). Repository: rG LLV

[PATCH] D77983: clang-tidy doc: add a note for every checker with an autofix

2020-05-10 Thread Sylvestre Ledru via Phabricator via cfe-commits
sylvestre.ledru added a comment. @alexfh ping Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D77983/new/ https://reviews.llvm.org/D77983 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://li

[PATCH] D77983: clang-tidy doc: add a note for every checker with an autofix

2020-04-18 Thread Sylvestre Ledru via Phabricator via cfe-commits
sylvestre.ledru added a comment. @alexfh ping Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D77983/new/ https://reviews.llvm.org/D77983 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://li

[PATCH] D77983: clang-tidy doc: add a note for every checker with an autofix

2020-04-13 Thread Sylvestre Ledru via Phabricator via cfe-commits
sylvestre.ledru updated this revision to Diff 257105. sylvestre.ledru added a comment. Doing it by hand. @alexfh please let me know if you would be ok with it (I will do it by hand) Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D77983/new/ https://r

[PATCH] D77983: clang-tidy doc: add a note for every checker with an autofix

2020-04-13 Thread Sylvestre Ledru via Phabricator via cfe-commits
sylvestre.ledru added a comment. Makes sense. I will give it a try :) thanks Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D77983/new/ https://reviews.llvm.org/D77983 ___ cfe-commits mailing list cfe-co

[PATCH] D77983: clang-tidy doc: add a note for every checker with an autofix

2020-04-13 Thread Alexander Kornienko via Phabricator via cfe-commits
alexfh added inline comments. Comment at: clang-tools-extra/docs/clang-tidy/checks/abseil-duration-conversion-cast.rst:29-35 Note: In the second example, the suggested fix could yield a different result, as the conversion to integer could truncate. In practice, this is very r

[PATCH] D77983: clang-tidy doc: add a note for every checker with an autofix

2020-04-13 Thread Sylvestre Ledru via Phabricator via cfe-commits
sylvestre.ledru updated this revision to Diff 256943. sylvestre.ledru added a comment. 2 spaces instead of 4 Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D77983/new/ https://reviews.llvm.org/D77983 Files: clang-tools-extra/docs/clang-tidy/checks

[PATCH] D77983: clang-tidy doc: add a note for every checker with an autofix

2020-04-12 Thread Eugene Zelenko via Phabricator via cfe-commits
Eugene.Zelenko added a comment. Herald added a subscriber: wuzish. It'll be reasonable to use two spaces indent. At least this is what mostly used for code blocks. Options are exceptions, but will be good idea to reformat them eventually. Repository: rG LLVM Github Monorepo CHANGES SINCE LA

[PATCH] D77983: clang-tidy doc: add a note for every checker with an autofix

2020-04-12 Thread Sylvestre Ledru via Phabricator via cfe-commits
sylvestre.ledru created this revision. sylvestre.ledru added reviewers: alexfh, aaron.ballman. Herald added subscribers: Charusso, arphaman, kbarton, nemanjai. Herald added a project: clang. Currently, when looking at a checker documentation, we have to go back to the whole list or look at the sou