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

I'd like to add support for Log levels #4

Open
MatiasMFM2001 opened this issue Feb 15, 2022 · 1 comment
Open

I'd like to add support for Log levels #4

MatiasMFM2001 opened this issue Feb 15, 2022 · 1 comment

Comments

@MatiasMFM2001
Copy link
Contributor

Hi again!

I'm aware of what README.md says. But I was wondering some ways to implement that feature, without adding much CPU/Flash/RAM usage (I can give you details if you like the idea)
By default, level code will not be included, because user must #define ENABLE_LOG4ARDUINO_LEVELS first

About YAGNI, I understand this would add some bloat to the library, even if user does not use ENABLE_LOG4ARDUINO_LEVELS, because there are now more use-cases we have to test with every future update
And if I really need levels functionality, there are more logging libraries available I can use

What do you think?
@jandelgado
Copy link
Owner

So what is your idea?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@jandelgado @MatiasMFM2001