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
fix Issue 15517: std.experimental.logger: using 'sharedLog' to change to file logging for default logger does not work #5371
Conversation
|
std/experimental/logger/core.d
Outdated
@@ -3102,3 +3102,39 @@ private void trustedStore(T)(ref shared T dst, ref T src) @trusted | |||
tl.logf("%s", sts); | |||
assert(tl.msg == SystemToStringMsg); | |||
} | |||
|
|||
// Issue 15517 | |||
unittest |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please mark this as either system or safe.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
See also CircleCi or style
:
Running DScanner
../dscanner/dsc --config .dscanner.ini --styleCheck $(find etc std -type f -name '*.d' | grep -vE 'std/traits.d|std/typecons.d') -I.
std/experimental/logger/core.d(3107:1)[warn]: A unittest should be annotated with at least @safe or @system
make: *** [style] Error 1
Please improve the commit message. |
std/experimental/logger/core.d
Outdated
|
||
foreach (t; ts) | ||
{ | ||
sharedLog.log(t); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Wait, isn't this exactly what the bug report was talking about? This works, but just doing
log(t);
doesn't
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
my bad, and fixed
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@burner This has now reverted
@burner Please rebase |
@JackStouffer rebased |
std/experimental/logger/core.d
Outdated
|
||
foreach (t; ts) | ||
{ | ||
sharedLog.log(t); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@burner This has now reverted
fixed |
was already fixed but had no fixture
@JackStouffer win32 works finally, I didn't allow me to open a file twice. |
thanks |
was already fixed but had no fixture