Skip to content
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.

Sign up for GitHub

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

Jump to bottom

log statement missing parameter #5778

Closed
davidlehrian opened this issue Oct 18, 2021 · 1 comment · Fixed by #5796
Closed

log statement missing parameter #5778

davidlehrian opened this issue Oct 18, 2021 · 1 comment · Fixed by #5796
Assignees
@SuGlider

Comments

@davidlehrian
Copy link
Contributor

davidlehrian commented Oct 18, 2021 •
edited
Loading

This is a super simple bug. On line 136 of EEPROM.cpp there is a log statement

log_e("Not enough memory for %d bytes in EEPROM");

Notice it is missing the parameter 'size'. It should be

log_e("Not enough memory for %d bytes in EEPROM",size);

@SuGlider SuGlider mentioned this issue Oct 23, 2021
Merged
@SuGlider
Copy link
Collaborator

SuGlider commented Oct 23, 2021 •
edited
Loading

@davidlehrian
Thanks for reporting.
This is fixed by #5796

@SuGlider SuGlider self-assigned this Oct 23, 2021
SuGlider added a commit that referenced this issue Oct 23, 2021
@SuGlider
log statement missing parameter #5778 (#5796)
a418058
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@SuGlider SuGlider

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

log statement missing parameter #5778
2 participants
@davidlehrian @SuGlider