-
Notifications
You must be signed in to change notification settings - Fork 132
Check for cdt indexer options #825
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Comments
apparently index unused headers must be turned off as well. |
I added checks for these values directly reading the instancescope. |
This behavior breaks at least ESP-IDF projects. Changing IDE configuration, especially without a warning, especially from default settings, and especially if Sloeber installation failed because of transient errors is... time consuming for the unaware. It would be a welcome feature if the installer asked nicely to change this setting, and warned the user about consequences of not changing it. And only after making sure that all the prerequisites were successfully obtained. |
@climategadgets Not sure what you are talking about. As to
|
The installer immediately changes the indexer settings without giving any indication that it is going to do so. The side effect of that is Eclipse immediately loses track of references, all open files get red marks, and code cannot be navigated anymore. |
@jantje have same situation in time of editing file indexer marks all with red and sometimes recover sometimes not, so code completion works unreadable. |
@uzi18 First is always: rebuild the index. |
With more and more functions relying on a "good working" of the indexer it is a good idea to check whether the indexer settings are "as expected by sloeber".

My settings are:
note that index source files not included in the build must be turned off.
People using multiple configurations should use the active configuration to build the indexer.
I don't think other settings are really important.
Well indexer on is, should test what Sloeber does when the indexer is off.
The text was updated successfully, but these errors were encountered: