[PATCH] D36051: [clang-tidy] List the checkers with autofix

sylvestre.ledru updated this revision to Diff 234972. sylvestre.ledru added a comment. alias => aliases Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D36051/new/ https://reviews.llvm.org/D36051 Files:

[PATCH] D36051: [clang-tidy] List the checkers with autofix

Eugene.Zelenko added inline comments. Comment at: clang-tools-extra/docs/clang-tidy/checks/list.rst:291 + +.. csv-table:: Alias.. + :header: "Name", "Redirect", "Offers fixes", "Severity" Aliases? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST

[PATCH] D36051: [clang-tidy] List the checkers with autofix

sylvestre.ledru updated this revision to Diff 234918. sylvestre.ledru added a comment. Fix a rst warning + update the clang link "Clang Static Analyzer" Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D36051/new/ https://reviews.llvm.org/D36051

[PATCH] D36051: [clang-tidy] List the checkers with autofix

sylvestre.ledru added a comment. In D36051#1792909 , @whisperity wrote: > I am a little bit conflicted about the //Severity// column. While I know our > people put a great deal of effort into keeping this classification sane, what > was put into

[PATCH] D36051: [clang-tidy] List the checkers with autofix

sylvestre.ledru updated this revision to Diff 234911. sylvestre.ledru added a comment. Add a comment about codechecker Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D36051/new/ https://reviews.llvm.org/D36051 Files:

[PATCH] D36051: [clang-tidy] List the checkers with autofix

whisperity added a comment. I am a little bit conflicted about the //Severity// column. While I know our people put a great deal of effort into keeping this classification sane, what was put into CodeChecker is, at the end of the day, a pretty arbitrary classification. I think RSTs support

[PATCH] D36051: [clang-tidy] List the checkers with autofix

sylvestre.ledru updated this revision to Diff 234890. sylvestre.ledru added a comment. Remove artifacts Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D36051/new/ https://reviews.llvm.org/D36051 Files:

[PATCH] D36051: [clang-tidy] List the checkers with autofix

sylvestre.ledru added a comment. > But please, let's make that table heading nice. `Severity` is capitalised, > `name` is not, "has an autofix" sounds weird... Maybe "offers fixes" instead? Good point, thanks :) Done! Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION

[PATCH] D36051: [clang-tidy] List the checkers with autofix

sylvestre.ledru added a comment. In D36051#1792684 , @Eugene.Zelenko wrote: > May be also split table by checks groups? Maybe? I would like to see this patch land first and iterate then on it. :) This patch is already a significant improvement over the

[PATCH] D36051: [clang-tidy] List the checkers with autofix

Eugene.Zelenko added a comment. May be also split table by checks groups? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D36051/new/ https://reviews.llvm.org/D36051 ___ cfe-commits mailing list

[PATCH] D36051: [clang-tidy] List the checkers with autofix

whisperity added a comment. In D36051#1792432 , @sylvestre.ledru wrote: > F0342: Screenshot_2019-12-20 clang-tidy - Clang-Tidy Checks — Extra Clang > Tools 10 documentation.png > here is the result Thank you

[PATCH] D36051: [clang-tidy] List the checkers with autofix

sylvestre.ledru added a comment. F0342: Screenshot_2019-12-20 clang-tidy - Clang-Tidy Checks — Extra Clang Tools 10 documentation.png here is the result Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION

[PATCH] D36051: [clang-tidy] List the checkers with autofix

sylvestre.ledru updated this revision to Diff 234841. sylvestre.ledru edited the summary of this revision. sylvestre.ledru added a comment. Herald added a reviewer: jdoerfert. Herald added a project: clang. Move to the table idea... Two years after :) Repository: rG LLVM Github Monorepo

[PATCH] D36051: [clang-tidy] List the checkers with autofix

JonasToth added a comment. In my opinion, this list should be available to the executable for two reasons. 1. External tools, that want to integrate clang-tidy in their workflow should be able to get the information about fixits, since they could collidie with their own textual replacements 2.

[PATCH] D36051: [clang-tidy] List the checkers with autofix

alexfh added a comment. In https://reviews.llvm.org/D36051#825428, @xazax.hun wrote: > Maybe instead of a separate list, having this information like yes/no column > in a table in the original is more user-friendly. What the better format is would depend on for which purpose do we want to

[PATCH] D36051: [clang-tidy] List the checkers with autofix

Eugene.Zelenko added a comment. I think will be good idea to do this in -list-checks too. https://reviews.llvm.org/D36051 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[PATCH] D36051: [clang-tidy] List the checkers with autofix

sylvestre.ledru added a comment. Good ideas guys. I will try to update list.rst to a table with the extra information. https://reviews.llvm.org/D36051 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

[PATCH] D36051: [clang-tidy] List the checkers with autofix

xazax.hun added a comment. Maybe instead of a separate list, having this information like yes/no column in a table in the original is more user-friendly. https://reviews.llvm.org/D36051 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

[PATCH] D36051: [clang-tidy] List the checkers with autofix

JonasToth added a comment. would be a classification of checks in general interesting? i could think of "quality", some checks might be heuristic, some might be new and have some bugs, and some might catch everything correct and handle it perfectly. So checks could have 2 categories: ->

[PATCH] D36051: [clang-tidy] List the checkers with autofix

sylvestre.ledru created this revision. Herald added subscribers: xazax.hun, JDevlieghere. In the current list http://clang.llvm.org/extra/clang-tidy/checks/list.html it is hard to know which one has autofix or not. This isn't perfect because we will have to list but I haven't found a better