search

Wumpf / IncludeToolbox

Visual Studio extension to format, prune, and inspect include directives.
https://marketplace.visualstudio.com/items?itemName=Wumpf.IncludeToolbox
MIT License
50 stars 22 forks source link

Includes with angled brackets are not being trialed for removal #52

Closed kreuzerkrieg closed 6 years ago

kreuzerkrieg commented 6 years ago

Looks like you are not even trying to remove it. the has been included (and not needed) however according to the tool it is not on the trial list

Could not remove #include: 'tinyxml2.h' Could not remove #include: 'P...r.h' Removed 0 #include directives from 'E...r.cpp'

Wumpf commented 6 years ago

I think this worked fine for me. However, by default the first include is not checked for removal since it is expected to be the corresponding header. You can change this setting in: settings > Include Toolbox > Trial and Error Include Removal > Ignore First Include

Can you confirm that that was the issue?

kreuzerkrieg commented 6 years ago

correct, this is the issue. so, there is no issue :) except when the first include is a precompiled header and the second one is the corresponding one

Wumpf commented 6 years ago

For that there is (\/|\\|^)$(currentFilename)\.(h|hpp|hxx|inl|c|cpp|cxx)$ in the ignore list by default :)