Code: [Select] x = CRGB::Gold; You have declared x to be an int, that means it only has 16 bits. This is way fewer bits than a CRGB variable type which is 32 bits long. So x should be a long int.You code goes on to set all LEDs to gold and then 10mS later it proceeds to set them all to black. You you think this will be at all noticeable?You then do the same thing in a most turgid way, if you learned how to use arrays all that could be reduced to just 3 extra lines instead of the dozens you have.No big deal, just write a function to set all the LEDs to black and call that function whenever you need it.
x = CRGB::Gold;
but I'm curious how you would suggest I reduce my code.