Whole Tomato Software Forums
Whole Tomato Software Forums
Main Site | Profile | Register | Active Topics | Members | Search | FAQ
User name:
Password:
Save Password
Forgot your password?

 All Forums
 Visual Assist
 Feature Requests
 Missing code inspection: modernize-use-override
 New Topic  Reply to Topic
 Printer Friendly
Author Previous Topic Topic Next Topic  

FriendlyRabbit
Junior Member

Germany
20 Posts

Posted - Nov 16 2023 :  03:59:58 AM  Show Profile  Reply with Quote
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.

feline
Whole Tomato Software

United Kingdom
18798 Posts

Posted - Nov 16 2023 :  07:59:20 AM  Show Profile  Reply with Quote
"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.

zen is the art of being at one with the two'ness
Go to Top of Page

FriendlyRabbit
Junior Member

Germany
20 Posts

Posted - Nov 16 2023 :  08:41:18 AM  Show Profile  Reply with Quote
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.
Go to Top of Page

feline
Whole Tomato Software

United Kingdom
18798 Posts

Posted - Nov 16 2023 :  10:42:47 AM  Show Profile  Reply with Quote
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

zen is the art of being at one with the two'ness
Go to Top of Page
  Previous Topic Topic Next Topic  
 New Topic  Reply to Topic
 Printer Friendly
Jump To:
© 2023 Whole Tomato Software, LLC Go To Top Of Page
Snitz Forums 2000