Yet more styling fixes

Put the return type and the function name on the same line in the .cpp file too.
While neither astyle, nor cpplint warn about this, this is how Kaleidoscope core
does it, so lets do the same.

Signed-off-by: Gergely Nagy <algernon@madhouse-project.org>

src/Kaleidoscope/Colormap.cpp

@@ -32,15 +32,13 @@ uint8_t ColormapEffect::max_layers_;
 ColormapEffect::ColormapEffect(void) {
 }
 
-void
+void ColormapEffect::begin(void) {
-ColormapEffect::begin(void) {
     LEDMode::begin();
 
     USE_PLUGINS(&::EEPROMSettings, &::LEDPaletteTheme);
 }
 
-void
+void ColormapEffect::max_layers(uint8_t max_) {
-ColormapEffect::max_layers(uint8_t max_) {
     if (map_base_ != 0)
         return;
 
@@ -48,8 +46,7 @@ ColormapEffect::max_layers(uint8_t max_) {
     map_base_ = ::LEDPaletteTheme.reserveThemes(max_layers_);
 }
 
-void
+void ColormapEffect::update(void) {
-ColormapEffect::update(void) {
     for (uint8_t l = 0; l < 32; l++) {
         if (!Layer.isOn(l))
             continue;
@@ -58,8 +55,7 @@ ColormapEffect::update(void) {
     }
 }
 
-bool
+bool ColormapEffect::focusHook(const char *command) {
-ColormapEffect::focusHook(const char *command) {
     return ::LEDPaletteTheme.themeFocusHandler(command, PSTR("colormap.map"),
             map_base_, max_layers_);
 }