Fix 3 code smells

bakgerman · Jun 1, 2022 · 4899155 · 4899155
1 parent da60d63
commit 4899155
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 5 deletions.
diff --git a/src/Shared/LayoutRenderers/AspNetLayoutMultiValueRendererBase.cs b/src/Shared/LayoutRenderers/AspNetLayoutMultiValueRendererBase.cs
@@ -52,7 +52,7 @@ public bool SingleAsArray
         /// </summary>
         public bool ValuesOnly { get; set; }
 
-#region Pairs
+        #region Pairs
         /// <summary>
         /// Serialize multiple key/value pairs
         /// </summary>
@@ -170,7 +170,7 @@ private void SerializePairsFlat(IEnumerable<KeyValuePair<string, string>> pairs,
 
 
         #endregion
-
+        
         #region Singles
 
         /// <summary>
@@ -215,7 +215,7 @@ private void SerializeSinglesJson(IEnumerable<string> values, StringBuilder buil
             }
         }
 
-        private void SerializeSingleJson(StringBuilder builder, string value)
+        private static void SerializeSingleJson(StringBuilder builder, string value)
         {
             // Quoted value
             AppendQuoted(builder, value);

diff --git a/src/Shared/LayoutRenderers/AspNetRequestLocalPortLayoutRenderer.cs b/src/Shared/LayoutRenderers/AspNetRequestLocalPortLayoutRenderer.cs
@@ -31,7 +31,7 @@ protected override void DoAppend(StringBuilder builder, LogEventInfo logEvent)
                 return;
             }
 
-            builder.Append(connection.LocalPort.ToString());
+            builder.Append(connection.LocalPort);
 #else
             var request = httpContext.TryGetRequest();
             if (request == null)

diff --git a/src/Shared/LayoutRenderers/AspNetRequestRemotePortLayoutRenderer.cs b/src/Shared/LayoutRenderers/AspNetRequestRemotePortLayoutRenderer.cs
@@ -32,7 +32,7 @@ protected override void DoAppend(StringBuilder builder, LogEventInfo logEvent)
                 return;
             }
 
-            builder.Append(connection.RemotePort.ToString());
+            builder.Append(connection.RemotePort);
 #else
             var request = httpContext.TryGetRequest();
             if (request == null)