Skip to content
This repository was archived by the owner on Dec 13, 2018. It is now read-only.

Allow configuring the trace source logger without a TraceListener ins… #431

Closed
wants to merge 1 commit into from

Conversation

moozzyk
Copy link
Contributor

@moozzyk moozzyk commented May 20, 2016

…tance

Addresses parking lot/I-#405

@dnfclas
Copy link

dnfclas commented May 20, 2016

Hi @moozzyk, I'm your friendly neighborhood .NET Foundation Pull Request Bot (You can call me DNFBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!

The agreement was validated by .NET Foundation and real humans are currently evaluating your PR.

TTYL, DNFBOT;

@@ -115,7 +119,10 @@ public void Dispose()
{
_disposed = true;
_rootTraceListener.Flush();
_rootTraceListener.Dispose();
if (_rootTraceListener != null)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is so old school

_rootTraceListener?.Dispose();

💃

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

10 PRINT "I AM NOT OLDSCHOOL"
20 GOTO 10

@davidfowl
Copy link
Member

:shipit:

@cesarblum
Copy link
Contributor

👍

@moozzyk moozzyk closed this May 23, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants