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

Jump to bottom

Multiple loggers - not logging what they should #1294

Closed
tchaloupka opened this issue Oct 15, 2015 · 1 comment
Closed

Multiple loggers - not logging what they should #1294

tchaloupka opened this issue Oct 15, 2015 · 1 comment

Comments

@tchaloupka
Copy link
Contributor

Is this break intended?
https://github.com/rejectedsoftware/vibe.d/blob/master/source/vibe/core/log.d#L127

It causes that when you are using multiple loggers (for example one for console output and one for file output and one for syslog) it does not log the line to other loggers even though they has valid min log level to also log it.
s-ludwig added a commit that referenced this issue Oct 15, 2015
@s-ludwig
Remove spurious "break;" statement. Fixes #1294.
f27d356 
This was a relict of the old code that didn't use a range based Logger interface.
@s-ludwig
Copy link
Member

It was a relict that I missed in the old code that didn't yet use a range interface for writing log messages.

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
@s-ludwig @tchaloupka