LED_BUILTIN must be macro not const int

[bperrybap]

The symbol LED_BUILTIN is supposed to be a macro not a const int
This was discussed at length on the Arduino developers mailing list years ago and final conclusion is that it must be a macro.
The reason is that some boards have a builtin led and some don't.
The only way for code, particularly library code that runs a many different platforms/core/boards, to "know" if the board that the code is being compiled for has a built in led is to use a macro for LED_BUILTIN so that the code can do compile time conditionals and enable / disable any led code appropriately when being built for various boards.

If you go look at the variants that arduino.cc supplies and even the esp8266 variants, you will see that LED_BUILTIN is a macro rather than a const int

variants that use const int instead of a macro will "trick" code into thinking that there is no built in led.

[CelliesProjects]

I have been bitten by this issue too. Even inside the esp32 eco system it causes confusion.

[bperrybap]

If it is desired to leave it as a const int, an easy thing to do is to add this define:
#define LED_BUILTIN LED_BUILTIN
That will give the best of both worlds.
A macro as needed but that gets replace with the const int value.

And for all those that are unfamiliar with the cpp macro parsing rules defined by the standard, it is not a circular definition.
cpp will replace the LED_BUILTIN in the source with LED_BUILTIN, essentially leaving it alone.
The compiler will replace LED_BUILTIN with the const int value during compilation.

--- bill

[stale]

[STALE_SET] This issue has been automatically marked as stale because it has not had recent activity. It will be closed in 14 days if no further activity occurs. Thank you for your contributions.

[bperrybap]

bump.
This is still an issue. Why should it automatically be closed?

[stale]

[STALE_CLR] This issue has been removed from the stale queue. Please ensure activity to keep it openin the future.

[CelliesProjects]

bump.

[stale]

[STALE_SET] This issue has been automatically marked as stale because it has not had recent activity. It will be closed in 14 days if no further activity occurs. Thank you for your contributions.

[bperrybap]

Still an issue.
The fix is simple.
In all the variant pins_arduino.h files change the LED_BUILTIN lines to include a #define LED_BUILTIN

example:

static const uint8_t LED_BUILTIN = 5;
#define BUILTIN_LED  LED_BUILTIN // backward compatibility

would become:

static const uint8_t LED_BUILTIN = 5;
#define BUILTIN_LED  LED_BUILTIN // backward compatibility
#define LED_BUILTIN LED_BUILTIN // create macro to indicate presence of built in LED

[stale]

[STALE_CLR] This issue has been removed from the stale queue. Please ensure activity to keep it openin the future.

[atanisoft]

@bperrybap submit a PR containing the fix.

[bperrybap]

I would consider it if the project owners would comment and agree to the fix.
This involves changing every single variant file. Not worth the work, if the project owners will simply ignore it.