Make it optional, or explain it in comments in the config why 2 blades are created for the NPXL V3 connector instead of one. Maybe the config generator could generate something like this:
BladeConfig blades[] = {
{ 0,
// === Main Blade: ===
WS281XBladePtr<144, bladePin, Color8::GRB, PowerPINS<bladePowerPin2, bladePowerPin3> >(),
// === Main NPXL Connector "inner ring": ===
SubBladeWithStride(0, 15, 2, WS281XBladePtr<16, blade2Pin, Color8::GRB, PowerPINS<bladePowerPin2, bladePowerPin3> >()),
// === Main NPXL Connector "outer ring": ===
SubBladeWithStride(1, 15, 2, NULL),
CONFIGARRAY(presets) },
{ NO_BLADE,
// === Main Blade: ===
WS281XBladePtr<144, bladePin, Color8::GRB, PowerPINS<bladePowerPin2, bladePowerPin3> >(),
// === Main NPXL Connector "inner ring" ===
SubBladeWithStride(0, 15, 2, WS281XBladePtr<16, blade2Pin, Color8::GRB, PowerPINS<bladePowerPin2, bladePowerPin3> >()),
// === Main NPXL Connector "outer ring" ===
SubBladeWithStride(1, 15, 2, NULL),
CONFIGARRAY(no_blade_presets), "nb_save" }
,
};
Or if you want to keep it simple just one comment between the two NPXL subblades:
// two blades created for the NPXL connector, one for inner & the other for outer ring.
It would certainly have removed a lot of confusion on my part when I first started.
And I am pretty sure I am not the only one who could have benefited from a simple comment in the config generator.