Skip to content
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

Handle additional include paths #55

Open
firewave opened this issue Nov 21, 2020 · 2 comments
Open

Handle additional include paths #55

firewave opened this issue Nov 21, 2020 · 2 comments

Comments

@firewave
Copy link
Collaborator

Currently no additional includes paths are being passed to the analysis. This may lead to false positives or findings not being reported at all.

We should also indicate that a non-system include wasn't found in the editor (albeit optional).

These need to be made available to the plugin and it seems they are not available yet.

@firewave
Copy link
Collaborator Author

To get the project configurations it is necessary to use CLion-specific functionality. I haven't figured out yet how to develop with them. There also were some recent changes outlined in https://blog.jetbrains.com/clion/2020/12/migration-guide-for-plugins-2020-3/

This probably makes the plugin CLion/AppCode-only but to be honest without the proper data for the file Cppcheck will never give you all the actual results.

@firewave
Copy link
Collaborator Author

If we have access to the build path (I know we can access the workspace path) we could leverage a potentially existing compile_commands.json to get the configuration. Maybe this could be done by leveraging the internal macros mentioned in #40.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant