Lots of people work together on Smoothie's code.


Here is a list of the useful ressources you want to read if you want to contribute to the project : 
2017/01/25 14:19 · arthur

In order to make everybody's life easier, and keep the codebase clean and manageable, here are a few rules you should follow when coding Smoothie stuff :

Do not include other headers in a header file unless absolutely necessary to define something in that header

Un-necessary includes clutter the code, and makes compile time longer.

Bad example ( in a .h file ) : 

code type="Cpp"

  1. include “Kernel.h”
  2. include “PanelScreen.h”
  3. include “Button.h”
  4. include <array>
  5. include <vector>

Instead do : 
code type="Cpp"

  1. include <array>
  2. include <vector>

class Kernel;
class PanelScreen;
class Button;

However, you still need to include whatever is necessary in your .cpp file

Use class pointers in classes and use Class xxx in header to forward define it

Same thing : no unnecessary includes in .h files : 

code type="Cpp" class Robot;
class Stepper;
class Planner;

class Kernel {

      Robot* robot;
      Stepper* stepper;
      Planner* planner;


Include headers only in cpp files, and only ones that are needed to fully define a class

Never include something in the header file ( unless absolutely necessary ). Include it in the .cpp file, and only if it is really required.

Do not put code in a method in a header unless absolutely necessary

the only time this is needed is for time critical code that must be inlined

Do not use <kbd>#ifdef</kbd> in the code

This is rarely needed and is only currently used by the makefile and main.cpp to eliminate named modules

Follow C== practices

  • class fields should be private, use getters and setters where necessary
  • methods not needed to be called externally should be private
  • try not to use multiple inheritance
  • make destructors virtual
  • use std::STL where appropriate
  • use C++11 where appropriate

Code formatting

Must compile cleanly

Your contributed code must compile cleanly with no warnings.

Test your code!!

This is really important. If you make changes to existing code make sure it is thoroughly tested before issuing a pull request.
If it is a new feature it should be tested as well as possible, or request testing by pointing to your branch in your repo.

Keep pull requests small

Only one feature or bug fix or refactor should go into a specific pull request, it is very hard to scan a huge pull request and to test it, so keep the
changes small and testable. Please follow git flow practices where possible.
branches should be called feature/feature-name or fix/fix-name, and only contain changed relevant to that feature or fix.
If other changes need to be made that are not directly related to this change but this change relies on them then it should be noted in the pull request that it depends on pull request #xxxx