enginmanap / enginmanap/limonEngine

Proper logger needed

Open
#155 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
C++
Stars
712
Forks
70
PR merge metrics
No merged PRs in 30d

Description

We have a logger in LimonAPI/util/Logger.h, and interface wise it is good.

Problem is, it is not good implementation wise. Many missing features.

1) Per SubSystem log level management.
Using single flag for enabling debug logs is not going to be enough, as it will just flood the logs. We need per subsystem level configuration.
2) Sink selection.
Current logger just logs to screen through graphics backend. that means using it for troubleshooting rendering is not possible. It should be able to sink to terminal, file and screen, based on configuration. Also should allow any combination of these.
3) Runtime changes.
Configuration changes should reflect at runtime.

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.