| T O P I C R E V I E W |
| FriendlyRabbit |
Posted - Nov 16 2023 : 03:59:58 AM
There are three clang-tidy checks which are just aliases:
- modernize-use-override
- cppcoreguidelines-explicit-virtual-functions
- hicpp-use-override
According to https://clang.llvm.org/extra/clang-tidy/checks/list.html, modernize-use-override is the actual check while the others are aliases. But modernize-use-override is missing in Visual Assist. It must have slipped through. Please add it.
Just for reference, I add a link to a discussion about treatment of aliases here: https://forums.wholetomato.com/forum/topic.asp?TOPIC_ID=20534. |
| 3 L A T E S T R E P L I E S (Newest First) |
| feline |
Posted - Nov 16 2023 : 10:42:47 AM
I ended up thinking the same thing, it would make more sense to add the unevaluated checkers to the list, rather than only showing the unevaluated ones. I have put in a feature request for this:
case=163852 |
| FriendlyRabbit |
Posted - Nov 16 2023 : 08:41:18 AM
You are right. I was a bit confused because regardless of the checkbox, enabled unevaluated checks are always shown. So it is in fact not missing.
Do you want this checkbox to show everything if it is enabled? Right now it is more like toggling between two views. |
| feline |
Posted - Nov 16 2023 : 07:59:20 AM
"modernize-use-override" is a supported check, and it is listed here:
https://docs.wholetomato.com/default.asp?W761
I think what is happening is that you have turned On "Show Unevaluated Checkers" in the Code Inspection settings. It turns out while this shows the unevaluated checkers, it ONLY shows them, so it is hiding "modernize-use-override" even though this one is also in the list, its just an evaluated checker. |
