change memory usage to bit fields#4
Conversation
char arrays for parsed cronexpr changed to uint8_t* bit fields added tests for memory allocation
|
Hi, the PR looks good, thanks! The only unclear thing is, why Besides that, can you please make it pass "compile and test" commands from here at least for some of the compilers? I really like to keep it "warning-free", currently cannot test the updated version with my app because of warnings. |
|
Forgot another bit, could you please rename |
compile commands in README.md edited
|
Macroname is fixed and use of the define extended. |
|
@Sanaloht Thanks for the update! Smoke tests are passing for me. Going to look deeper and merge it in a day or two. |
|
Merged and fixed and changed some minor things. Overall bitfields are very nice to have. Thanks again for PR! |
|
You're welcome and thanks to Niondir for helping! |
char arrays for parsed cronexpr changed to uint8_t* bit fields
added tests for memory allocation