Chris--A / PGMWrap

A concept library for easy use of PROGMEM data.
18 stars 5 forks source link

Remove extra array initializer from arrays.ino #2

Closed per1234 closed 8 years ago

per1234 commented 8 years ago

Other suggestions:

Chris--A commented 8 years ago

Cheers I'll merge this in.

The subscript size was added (was empty originally) because I use a loop with it hard coded, rather than using sizeof to keep things simple. I wanted to make sure people could see where the number came from.

But you have pointed out my mistake, I had a slightly different example to begin with, and I added the extra values at the end later so I did forget to change int16_p to uint16_p. Cheers I'll fix this too. Also when I tidied up the sketch you can see I forgot to remove the duplicate '123' value.

I usually add the comma at the end of the values in enums and arrays by habit, but its fine removed.