search

boocs / ue4-intellisense-fixes

Automatically fixes VSCode/Unreal Engine Intellisense Config bugs on startup
MIT License
162 stars 9 forks source link

should we add limitSymbolsToIncludedHeaders to the cpp_properties.json? #2

Closed miltoncandelero closed 3 years ago

miltoncandelero commented 3 years ago

Hello, I tried to see what you do with that variable and I can't make head or tails of the source of the plugin 😅

Do you know if it would be better to limit this?

An example of what I say we could add: image

boocs commented 3 years ago

No, you don't need to!

This extension already checks to see if you need this setting set.

By default limitSymbolsToIncludedHeaders is set to true. So if you don't have this set to false somewhere in the 3 user/folder/workspace configs then the extension won't bother adding it.

limitSymbolsToIncludedHeaders is also why the extension sets the browse.path in the UE4 workspace to [ ]. The setting won't be as efficient otherwise.