-
-
Notifications
You must be signed in to change notification settings - Fork 1.1k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Added EEPROM support for char*
, char const*
and String
.
#496
base: master
Are you sure you want to change the base?
Conversation
Memory usage change @ fcb276c
Click for full report table
Click for full report CSV
|
The spell check fails on a piece of code that was not touched in this patch. |
Should I perhaps rebase this patch and try again to pass the spell check test? |
@jfjlaros you are welcome to do so. We wouldn't be strict about requiring all the continuous integration checks to pass before merging in a case like this where the failure was unrelated to the proposal, but it is useful to get it into a fully passing state simply to save the reviewer the effort of having to evaluate whether the failure was relevant. |
Memory usage change @ 8e6d333
Click for full report table
Click for full report CSV
|
It seems that a number of people expect the EEPROM
put
andget
functions to work with C strings andString
objects [1, 2]. This patch implements that functionality.Example usage:
I hope you find this patch useful.