Update .pp files

damianh · Jan 6, 2019 · 2a8ecfb · 2a8ecfb
1 parent 8c06287
commit 2a8ecfb
Show file tree

Hide file tree

Showing 8 changed files with 35 additions and 21 deletions.
diff --git a/src/LibLog/LogExtensions.cs.pp b/src/LibLog/LogExtensions.cs.pp
@@ -508,7 +508,7 @@
         // ReSharper disable once UnusedParameter.Local
         private static void GuardAgainstNullLogger(ILog logger)
         {
-            if (logger == null) throw new ArgumentNullException(nameof(logger));
+            if (logger == null) throw new ArgumentNullException("logger");
         }
 
         private static void LogFormat(this ILog logger, LogLevel logLevel, string message, params object[] args)
@@ -530,8 +530,7 @@
         // Allow passing callsite-logger-type to LogProviderBase using messageFunc
         internal static Func<string> WrapLogSafeInternal(LoggerExecutionWrapper logger, Func<string> messageFunc)
         {
-            string WrappedMessageFunc()
-            {
+            var WrappedMessageFunc = new Func<string>(() => {
                 try
                 {
                     return messageFunc();
@@ -543,18 +542,18 @@
                 }
 
                 return null;
-            }
+            });
 
             return WrappedMessageFunc;
         }
 
         // Allow passing callsite-logger-type to LogProviderBase using messageFunc
         private static Func<string> WrapLogInternal(Func<string> messageFunc)
         {
-            string WrappedMessageFunc()
+            var WrappedMessageFunc = new Func<string>(() =>
             {
                 return messageFunc();
-            }
+            });
 
             return WrappedMessageFunc;
         }

diff --git a/src/LibLog/LogProvider.cs.pp b/src/LibLog/LogProvider.cs.pp
@@ -114,7 +114,10 @@
             }
         }
 
-        internal static ILogProvider CurrentLogProvider => s_currentLogProvider;
+        internal static ILogProvider CurrentLogProvider
+        { 
+            get { return s_currentLogProvider; } 
+        }
 
         /// <summary>
         /// Gets a logger for the specified type.
@@ -126,7 +129,10 @@
 #else
         internal
 #endif
-        static ILog For<T>() => GetLogger(typeof(T));
+        static ILog For<T>() 
+        {
+            return GetLogger(typeof(T));
+        }
 
         /// <summary>
         /// Gets a logger for the current class.
@@ -298,7 +304,9 @@
             internal static readonly NoOpLogger Instance = new NoOpLogger();
 
             public bool Log(LogLevel logLevel, Func<string> messageFunc, Exception exception, params object[] formatParameters)
-                => false;
+            { 
+                return false; 
+            }
         }
 #endif
     }

diff --git a/src/LibLog/LogProviders/DisposableAction.cs.pp b/src/LibLog/LogProviders/DisposableAction.cs.pp
@@ -18,7 +18,7 @@
 
         public void Dispose()
         {
-            _onDispose?.Invoke();
+            if(_onDispose != null) _onDispose.Invoke();
         }
     }
 }
diff --git a/src/LibLog/LogProviders/Log4NetLogProvider.cs.pp b/src/LibLog/LogProviders/Log4NetLogProvider.cs.pp
@@ -372,7 +372,7 @@
                     case LogLevel.Fatal:
                         return s_levelFatal;
                     default:
-                        throw new ArgumentOutOfRangeException(nameof(logLevel), logLevel, null);
+                        throw new ArgumentOutOfRangeException("logLevel", logLevel, null);
                 }
             }
         }

diff --git a/src/LibLog/LogProviders/LogMessageFormatter.cs.pp b/src/LibLog/LogProviders/LogMessageFormatter.cs.pp
@@ -35,6 +35,7 @@
             return () =>
             {
                 var targetMessage = messageBuilder();
+                IEnumerable<string> _;
                 return FormatStructuredMessage(targetMessage, formatParameters, out _);
             };
         }
@@ -61,7 +62,8 @@
             {
                 var arg = match.Groups["arg"].Value;
 
-                if (!int.TryParse(arg, out _))
+				int result;
+                if (!int.TryParse(arg, out result))
                 {
                     processedArguments = processedArguments ?? new List<string>(formatParameters.Length);
                     var argumentIndex = processedArguments.IndexOf(arg);

diff --git a/src/LibLog/LogProviders/LogProviderBase.cs.pp b/src/LibLog/LogProviders/LogProviderBase.cs.pp
@@ -74,7 +74,7 @@
         /// <returns>A provider-specific method to open a nested diagnostics context.</returns>
         protected virtual OpenNdc GetOpenNdcMethod()
         {
-            return _ => NoopDisposableInstance;
+            return (_) => NoopDisposableInstance;
         }
 
         /// <summary>

diff --git a/src/LibLog/LogProviders/NLogLogProvider.cs.pp b/src/LibLog/LogProviders/NLogLogProvider.cs.pp
@@ -2,7 +2,8 @@
 namespace $rootnamespace$.Logging.LogProviders
 {
     using System;
-    using System.Diagnostics.CodeAnalysis;
+	using System.Collections.Generic;
+	using System.Diagnostics.CodeAnalysis;
     using System.Linq;
     using System.Linq.Expressions;
 
@@ -21,7 +22,12 @@
             _getLoggerByNameDelegate = GetGetLoggerMethodCall();
         }
 
-        public static bool ProviderIsAvailableOverride { get; set; } = true;
+        static NLogLogProvider()
+        {
+            ProviderIsAvailableOverride = true;
+        }
+
+        public static bool ProviderIsAvailableOverride { get; set; }
 
         public override Logger GetLogger(string name)
         {
@@ -284,6 +290,7 @@
                         var formatMessage = messageFunc();
                         if (!s_structuredLoggingEnabled)
                         {
+							IEnumerable<string> _;
                             formatMessage =
                                 LogMessageFormatter.FormatStructuredMessage(formatMessage,
                                     formatParameters,

diff --git a/src/LibLog/LoggerExecutionWrapper.cs.pp b/src/LibLog/LoggerExecutionWrapper.cs.pp
@@ -50,7 +50,7 @@
                     formatParameters);
             }
 
-            string WrappedMessageFunc()
+            var WrappedMessageFunc = new Func<string>(() => 
             {
                 try
                 {
@@ -62,22 +62,20 @@
                 }
 
                 return null;
-            }
+            });
 
             // Callsite HACK - Need to ensure proper callsite stack without inlining, so calling the logger within a virtual interface method
             return _callsiteLogger.Log(WrappedLogger, logLevel, WrappedMessageFunc, exception, formatParameters);
         }
 
         private interface ICallSiteExtension
         {
-            bool Log(Logger logger, LogLevel logLevel, Func<string> messageFunc, Exception exception,
-                object[] formatParameters);
+            bool Log(Logger logger, LogLevel logLevel, Func<string> messageFunc, Exception exception, object[] formatParameters);
         }
 
         private class CallSiteExtension : ICallSiteExtension
         {
-            bool ICallSiteExtension.Log(Logger logger, LogLevel logLevel, Func<string> messageFunc, Exception exception,
-                object[] formatParameters)
+            bool ICallSiteExtension.Log(Logger logger, LogLevel logLevel, Func<string> messageFunc, Exception exception, object[] formatParameters)
             {
                 return logger(logLevel, messageFunc, exception, formatParameters);
             }