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

Multiple loggers - not logging what they should #1294

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

Comments

Projects
None yet
2 participants
@tchaloupka
Contributor

tchaloupka commented Oct 15, 2015

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

Remove spurious "break;" statement. Fixes #1294.
This was a relict of the old code that didn't use a range based Logger interface.
@s-ludwig

This comment has been minimized.

Show comment
Hide comment
@s-ludwig

s-ludwig Oct 15, 2015

Member

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

Member

s-ludwig commented Oct 15, 2015

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

@s-ludwig s-ludwig closed this in 2fffd94 Oct 15, 2015

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment