ESP8266 Community Forum

- Fri Feb 06, 2015 1:57 pm #9139 Hey thanks!

It would be nice if you would rename the lgpio_ws2812 function to ws2812_write, so that it is consistent with the function naming of the other modules.

Also, please change the indentation from tabs to two spaces -- this seems to be the coding style being used already.

- Fri Feb 06, 2015 4:24 pm #9146 Ok, I will do this. I imported the project to Eclipse so I was in the impression that the correct code style is already applied. I hope that someone can test this before my replacement modules arrive (15-31 days delivery time).

- Sat Feb 07, 2015 2:57 am #9162 I tested it -- seems to work fine

- Wed Feb 11, 2015 10:13 am #9476

If I get the ok from testers I will create a pull request.

Have you already made the pull request?