Whole Tomato Software Forums
Whole Tomato Software Forums
Main Site | Profile | Register | Active Topics | Members | Search | FAQ
 All Forums
 Visual Assist
 Feature Requests
 Missing code inspection: modernize-use-override

You must be registered to post a reply.
Click here to register.

Format: BoldItalicizeUnderlineStrikethrough Align leftCenterAlign right Insert horizontal ruleUpload and insert imageInsert hyperlinkInsert email addressInsert codeInsert quoted textInsert listInsert Emoji

Forum code is on.
Html is off.

Check to subscribe to this topic.

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:

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:


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.

© 2023 Whole Tomato Software, LLC Go To Top Of Page
Snitz Forums 2000