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

✨ Introduce log environment #679

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

✨ Introduce log environment #679

wants to merge 2 commits into from

Conversation

elbeno
Copy link
Contributor

@elbeno elbeno commented Jan 12, 2025

Problem:

  • Logging is parameterized on many axes. Most logging systems include the level.
    Some also include a module or source. Some include other parameters.
  • Logging machinery should be agnostic to the parameterizations of each logging
    backend. Adding extra parameters to logging macros is not scalable.

Solution:

  • Expand the current logging module to the idea of a logging environment. An
    environment allows compile-time scope-based arguments to be passed through the
    logging machinery; logging backends can extract whichever values they can use
    from the current environment.

Problem:
- `stdx` added a breaking change, viz. the ct_string format result always being
  returned from `ct_format`.

Solution:
- Update code to deal with that.
@elbeno elbeno force-pushed the log-env branch 2 times, most recently from 0d5762e to c887b00 Compare January 12, 2025 05:31
Problem:
- Logging is parameterized on many axes. Most logging systems include the level.
  Some also include a module or source. Some include other parameters.
- Logging machinery should be agnostic to the parameterizations of each logging
  backend. Adding extra parameters to logging macros is not scalable.

Solution:
- Expand the current logging module to the idea of a logging environment. An
  environment allows compile-time scope-based arguments to be passed through the
  logging machinery; logging backends can extract whichever values they can use
  from the current environment.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant