Clearing rest of buffer in ws2812_push

[king2]

I see is that WS2812_THREE_SAMPLE will clear rest of buffer (fill it with zeroes), while WS2812_FOUR_SAMPLE leaves rest of buffer intact.

Is this a mistake or I cannot understand something important about how it works?

[cnlohr]

I don't really remember the rationale either way, it's unlike to cause serious adverse effects. I know that the 3-sample one can potentially have overlapping leds over the same word, so on that it's probably useful to clear out the next few, but on the four, it probably doesn't matter at all.

[king2]

It does matter just when you want to call ws_2812 with small buffer, not covering all LEDs you have. In this case with "3 sample" rest of LEDs will be set to zero, in case of '4 sample" rest of LEDs will be kept at previous state.

It is really not important, I just wanted to help to make code more clear and consistent :)

[cnlohr]

If you make a PR and test it, I'd merge, but otherwise, probably not worth the change.

[king2]

OK, I'll try to do it when I will return from Sysadmin's day celebration :)