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

Commit 1783842

Browse files
committed
Naming
1 parent ea529ef commit 1783842

File tree

3 files changed

+7
-3
lines changed

3 files changed

+7
-3
lines changed

src/Microsoft.Extensions.Logging.EventSource/EventSourceLogger.cs

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -43,7 +43,9 @@ public LogLevel Level
4343
{
4444
get
4545
{
46-
_eventSource.VerifyLevel();
46+
// need to check if the filter spec and internal event source level has changed
47+
// and update the loggers level if it has
48+
_eventSource.ApplyFilterSpec();
4749
return _level;
4850
}
4951
set

src/Microsoft.Extensions.Logging.EventSource/EventSourceLoggerProvider.cs

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -37,7 +37,9 @@ public EventSourceLoggerProvider(LoggingEventSource eventSource, EventSourceLogg
3737
/// </summary>
3838
public ILogger CreateLogger(string categoryName)
3939
{
40-
_eventSource.VerifyLevel();
40+
// need to check if the filter spec and internal event source level has changed
41+
// and update the _defaultLevel if it has
42+
_eventSource.ApplyFilterSpec();
4143
var newLogger = _loggers = new EventSourceLogger(categoryName, _factoryID, _eventSource, _loggers);
4244
newLogger.Level = ParseLevelSpecs(_filterSpec, _defaultLevel, newLogger.CategoryName);
4345
return newLogger;

src/Microsoft.Extensions.Logging.EventSource/LoggingEventSource.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -236,7 +236,7 @@ private void SetFilterSpec(string filterSpec)
236236
}
237237

238238
[NonEvent]
239-
internal void VerifyLevel()
239+
internal void ApplyFilterSpec()
240240
{
241241
lock (_lockObj)
242242
{

0 commit comments

Comments
 (0)