machinekit-hal icon indicating copy to clipboard operation
machinekit-hal copied to clipboard

Coding style and static analysis

Open ArcEye opened this issue 7 years ago • 1 comments

Issue by machinekoder Tue Dec 2 10:10:02 2014 Originally opened as https://github.com/machinekit/machinekit/issues/391


We should enforce a specific coding style for every language in the project. This can e.g. be done by using astyle. My code editor (Kate) always shows mixed indentations (tabs and spaces) everywhere. This makes the code unreadable on some platforms and is possible error source.

Furthermore every piece of code that is commit should be checked with suitable static code analysis tool. For C and C++ cppcheck and clang might be suitable. For Python pep8 is the way to go. This ensures that common error will not make its way into the project.

ArcEye avatar Aug 03 '18 15:08 ArcEye

See https://github.com/machinekit/machinekit-cnc/issues/38 for full text, this is a copy header only.

ArcEye avatar Aug 03 '18 15:08 ArcEye