-
Notifications
You must be signed in to change notification settings - Fork 363
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
Refactoring: cleanup CxxSquidSensor - remove CxxVCppBuildLogParser #1325
Comments
I checked the behavior and the log file parser is called once per module (per VC++ project) which is correct.
The parser will analyze each line of the log file which has a performance impact for large multi module log files and may also create wrong settings because the parser does not handle the module settings separately e.g. it consumes all include paths or preprocessor definitions. sample log file parser results
|
Would it not be better to run this only once, and create a map of settings per file. This is the same as the clang compilations settings. So perhaps a refactoring to support build settings per file that can be used for all import methods. |
Remove
CxxVCppBuildLogParser
call fromCxxSquidSensor
:cxxConf.setCompilationPropertiesWithBuildLog
increateConfiguration
.This is slowing down the plugin because
CxxSquidSensor
is called for each source file.CxxVCppBuildLogParser
should be called only once and save parsed setting at a central place e.g.CxxConfiguration
.The text was updated successfully, but these errors were encountered: