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.

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

Replace log level DEBUG with INFO #9770

Open
mixxxbot opened this issue Aug 23, 2022 · 0 comments
Open

Replace log level DEBUG with INFO #9770

mixxxbot opened this issue Aug 23, 2022 · 0 comments

Comments

@mixxxbot
Copy link
Collaborator

Reported by: uklotzde
Date: 2019-10-05T11:55:21Z
Status: Confirmed
Importance: Wishlist
Launchpad Issue: lp1846880


The log level DEBUG = qDebug() is not intended to be enabled in production and may cause performance issues. This log level should only be enabled temporarily upon request or during development.

Instead, the new log level INFO = qInfo() (available since Qt5) should be used for writing detailed log messages into the log file. The default log level for console outputs should be WARNING = qWarning() as before.

This is a long-running migration that could not be completed instantly. At least all new code should follow these rules. The logging rules need to be documented in the developer guide.

@mixxxbot mixxxbot transferred this issue from another repository Aug 24, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant