FilteringTargetWrapper - Remove delegate allocation

NLog · May 27, 2020 · 2879d7d · 2879d7d
1 parent 024896f
commit 2879d7d
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/src/NLog/Targets/Wrappers/FilteringTargetWrapper.cs b/src/NLog/Targets/Wrappers/FilteringTargetWrapper.cs
@@ -138,7 +138,7 @@ protected override void Write(AsyncLogEventInfo logEvent)
         /// <inheritdoc/>
         protected override void Write(IList<AsyncLogEventInfo> logEvents)
         {
-            var filterLogEvents = logEvents.Filter(Filter, ShouldLogEvent);
+            var filterLogEvents = logEvents.Filter(Filter, (logEvent, filter) => ShouldLogEvent(logEvent, filter));
             WrappedTarget.WriteAsyncLogEvents(filterLogEvents);
         }
 

diff --git a/src/NLog/Targets/Wrappers/PostFilteringTargetWrapper.cs b/src/NLog/Targets/Wrappers/PostFilteringTargetWrapper.cs
@@ -165,7 +165,7 @@ protected override void Write(IList<AsyncLogEventInfo> logEvents)
             else
             {
                 InternalLogger.Trace("PostFilteringWrapper(Name={0}): Filter to apply: {1}", Name, resultFilter);
-                var resultBuffer = logEvents.Filter(resultFilter, ApplyFilter);
+                var resultBuffer = logEvents.Filter(resultFilter, (logEvent, filter) => ApplyFilter(logEvent, filter));
                 InternalLogger.Trace("PostFilteringWrapper(Name={0}): After filtering: {1} events.", Name, resultBuffer.Count);
                 if (resultBuffer.Count > 0)
                 {