Skip to content

PR for issue #1555: change BUILTIN_LED to LED_BUILTIN #1556

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
wants to merge 25 commits into from
Closed

PR for issue #1555: change BUILTIN_LED to LED_BUILTIN #1556

wants to merge 25 commits into from

Conversation

tuxedo0801
Copy link
Contributor

changed all occurrences of BUILTIN_LED to arduino like LED_BUILTIN.

I hope I missed nothing ...

@tuxedo0801 tuxedo0801 changed the title PR for issue #1555 PR for issue #1555: change BUILTIN_LED to LED_BUILTIN Feb 2, 2016
@igrr
Copy link
Member

igrr commented Feb 2, 2016

I'd vote for keeping BUILTIN_LED defined in pins_arduino.h, alongside with LED_BUILTIN.

@tuxedo0801
Copy link
Contributor Author

Having both would be OK for me. I will update the PR accordingly.

@tuxedo0801
Copy link
Contributor Author

now both variants are included. ESP style and Arduino style

LED_BUILTIN
BUILTIN_LED

igrr pushed a commit that referenced this pull request Feb 4, 2016
keep arduino incompatible BUILTIN_LED (+22 squashed commits)
[e124f9c] keep arduino incompatible BUILTIN_LED
[18dab66] keep arduino incompatible BUILTIN_LED
[0095de6] keep arduino incompatible BUILTIN_LED
[cca2714] keep arduino incompatible BUILTIN_LED
[ff62bfb] keep arduino incompatible BUILTIN_LED
[fd6f7c0] keep arduino incompatible BUILTIN_LED
[220b02a] keep arduino incompatible BUILTIN_LED
[b871b1b] keep arduino incompatible BUILTIN_LED
[1d97cac] keep arduino incompatible BUILTIN_LED
[ebc2667] changed BUILTIN_LED to LED_BUILTIN
[7008a27] changed BUILTIN_LED to LED_BUILTIN
[cf3f6da] changed BUILTIN_LED to LED_BUILTIN
[5e11c43] changed BUILTIN_LED to LED_BUILTIN
[8645bd5] changed BUILTIN_LED to LED_BUILTIN
[faae742] changed BUILTIN_LED to LED_BUILTIN
[df2659b] changed BUILTIN_LED to LED_BUILTIN
[1eb74d5] changed BUILTIN_LED to LED_BUILTIN
[fb9e977] changed BUILTIN_LED to LED_BUILTIN
[b735737] changed BUILTIN_LED to LED_BUILTIN
[d0557dd] changed BUILTIN_LED to LED_BUILTIN
[a25e4aa] changed BUILTIN_LED to LED_BUILTIN
[7865ace] changed BUILTIN_LED to LED_BUILTIN
@igrr
Copy link
Member

igrr commented Feb 4, 2016

Squashed and merged in 573a0fb, thanks.

@igrr igrr closed this Feb 4, 2016
narongrat added a commit to narongrat/Arduino that referenced this pull request Feb 8, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants