search

Cevelop / Issues

The official issue tracker for Cevelop.
9 stars 1 forks source link

clang-tidy integration #76

Open PeterSommerlad opened 6 years ago

PeterSommerlad commented 6 years ago

Expected Behavior

It would be great if we could ship/include clang-tidy, so that it can be run automatically with the settings the compiler would be run. I am not sure about clang-tidy fixes, but showing the results in the IDE and adding the NOLINT comments as a quick-fix could be handy.

It might be worth to first do an analysis what checkers clang-tidy provides that are duplicates from ours and if we can provide an interactive ui for setting its plentyful options (must be generic because of clang-tidy frequent changes and updates)

Actual Behavior

integration can only achieved manually so far. Does the llvm eclipse cdt plugin provide something?

Cevelop Version, Operating System and Compiler

current

PeterSommerlad commented 6 years ago

https://github.com/Ericsson/CodeCheckerEclipsePlugin seems to be very limited and since 1 year quiet attempt to do so

PeterSommerlad commented 6 years ago

Another candidate for integration is https://github.com/wangzw/CppStyle that uses clang-format under the hood.

tcorbat commented 6 years ago

Integration of such a tool would be great. My concern lies with the bunding of external tools.

I've checked the compatibility of clang-tidy and windows. If clang is installed, it works properly. However, clang does not ship with a standard library. The visual studio build tools can be used as fallback for it. But I doubt we may ship them with Cevelop. Maybe in a plug-in the configured toolchain's standard library can be accessed.

I'm currently not convinced by its performance. Maybe I need to configure the tool properly.