ThingPulse / esp8266-weather-station

ESP8266 Weather Station library supporting OpenWeatherMap, Aeris and other sources
MIT License
1.06k stars 362 forks source link

Make allowedHours const #181

Closed ahnolds closed 4 years ago

ahnolds commented 4 years ago

Right now allowedHours/setAllowedHours take a plain unsigned char pointer. The allowedHours array is never modified, so it would be beneficial to have it be const so that a caller can pass in a const array without having to cast away the const.

marcelstoer commented 4 years ago

Thanks. I'm sure we have a ton of variables that could be made constants or use smaller data types than what they currently use.