Include in FASTLED_DEBUG_COUNT_FRAME_RETRIES block?

Class AVRSoftwareSPIOutput< DATA_PIN, CLOCK_PIN, SPI_SPEED >

Replace the select pin definition with a set of pins, to allow using mux hardware for routing in the future.

Member AVRSoftwareSPIOutput< DATA_PIN, CLOCK_PIN, SPI_SPEED >::select ()

Move select responsibility out of the SPI classes entirely, make it up to the caller to remember to lock/select the line?

Research whether this really means 'hi' or 'lo'.

Member AVRSoftwareSPIOutput< DATA_PIN, CLOCK_PIN, SPI_SPEED >::writeBytes (FASTLED_REGISTER uint8_t *data, int len)

Need to type this better so that explicit casts into the call aren't required.

Member CFastLED::countFPS (int nFrames=25)

Make this a rolling counter

Class CPixelView< PIXEL_TYPE >::pixelset_iterator_base< T >

Make this a fully specified/proper iterator

Class DATA_NOP

Convinience macro for building these

Member fill_raw_2dnoise8 (uint8_t *pData, int width, int height, uint8_t octaves, q44 freq44, fract8 amplitude, int skip, uint16_t x, int scalex, uint16_t y, int scaley, uint16_t time)

Why isn't this declared in the header (noise.h)?

Module HSV2RGB

Provide the "maximum brightness no matter what" variation.

Member HSV_SECTION_3

I believe this is mis-named, and should be HSV_SECTION_4

Member HSV_SECTION_6

Unused. Remove?

Member IEEE754binary32_t::exp_

Doc: what is this for?

Member IEEE754binary32_t::mant16

Doc: what is this for?

Member IEEE754binary32_t::mant7

Doc: what is this for?

Member IEEE754binary32_t::mant_hi16_exp_lo1

Doc: what is this for?

Member IEEE754binary32_t::mant_lo8

Doc: what is this for?

Member IEEE754binary32_t::sb_

Doc: what is this for?

Member IEEE754binary32_t::sb_exphi7

Doc: what is this for?

Member MASK_SKIP_BITS

Unused. Remove?

Member MAX_DATA_RATE

Unused. Remove?

Member nblendPaletteTowardPalette (CRGBPalette16 &currentPalette, CRGBPalette16 &targetPalette, uint8_t maxChanges=24)

Shouldn't the targetPalette be const?

Member nmax

Remove?

Member nmin

Remove?

Member NO_PIN

Unused, remove?

Member noise_max

Remove?

Member noise_min

Remove?

Member nscale8_raw (CRGB *leds, uint16_t num_leds, uint8_t scale)

Remove this or add a declaration? This is not listed in the colorutils.h header.

Member NUM_CONTROLLERS

Remove?

Module PaletteClasses

For documentation purposes it would be nice to reorder these definitions by type and in ascending number of entries.

Member RECOMMENDED_VIRTUAL_BITS

Can these macros be replaced with constants scoped to PixelController::init_binary_dithering()?