Actions
Testing #647
open
Code review imlightsw to look for leaks
Start date:
18 Jul 2018
Due date:
% Done:
0%
Estimated time:
Severity:
04 - Low
Description
Many of the libraries being used should be safe, but they should be reviewed. All allocations should be static, as described in this StackOverflow discussion.
Also, all static strings (like path names for SPIFFS) should be wrapped in F() to push them to PROGMEM as discussed in this StackExchange thread.
Updated by Hammel over 6 years ago
And remove as many stack variables as possible to speed operation in exchange for program size. There are plenty of variables I could reuse (simple int's for example).
Updated by Hammel over 4 years ago
- Priority changed from Urgent to Immediate
- Severity changed from 01 - Critical to 04 - Low
Actions