-
Notifications
You must be signed in to change notification settings - Fork 804
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
Console/Terminal Log standardization #2135
Comments
or we could use a macro; perhaps something like this
and replace the 'Log.trace' with 'LOG_TRACE' and we would get the filename |
Since we are only building for ESP devices now I would suggest replacing the log library with the ESP_LOG macros and then we could use log tag filtering. |
I'm sorry, I didn't understand the next sentence....
so, OMG will be only ESP device? |
I thought we only had builds for ESP devices, or am I missing something? |
1.8.0 supports only ESP devices |
Is your feature request related to a problem? Please describe.
It seems to me that if we have 2 or 3 modules enabled (like RF + BT + IR) it is very difficult to understand the logs on the console because each module writes something when requested without providing context (probably all assuming only their own module is active). Sometimes you log multi-lines not with a simple call to the LOG function but with a call to multiple LOGs... this makes it a nightmare to understand the flow.
Describe the solution you'd like
Having a clear console register in which registers are always contextualised and encapsulated.
Describe alternatives you've considered
Create a soft rules like:
A - The log need to have a prefix the module involved (Example "BT .... " or "RF ..." etc )
B - The log need to be encapsulated:
As example:
should be
The text was updated successfully, but these errors were encountered: