From b876b35457121878345de26a1542d24cbb2d56a4 Mon Sep 17 00:00:00 2001
From: Jan Tattermusch <jtattermusch@google.com>
Date: Thu, 5 Sep 2019 13:48:31 +0200
Subject: [PATCH] address review comments

---
 src/csharp/Grpc.Core.Api/SerializationContext.cs              | 4 ++--
 .../Internal/DefaultSerializationContextTest.cs               | 3 ---
 2 files changed, 2 insertions(+), 5 deletions(-)

diff --git a/src/csharp/Grpc.Core.Api/SerializationContext.cs b/src/csharp/Grpc.Core.Api/SerializationContext.cs
index 76f1951b68e..59e14c12e3b 100644
--- a/src/csharp/Grpc.Core.Api/SerializationContext.cs
+++ b/src/csharp/Grpc.Core.Api/SerializationContext.cs
@@ -28,7 +28,7 @@ namespace Grpc.Core
     {
         /// <summary>
         /// Use the byte array as serialized form of current message and mark serialization process as complete.
-        /// <c>Complete()</c> can only be called once. By calling this method the caller gives up the ownership of the
+        /// <c>Complete(byte[])</c> can only be called once. By calling this method the caller gives up the ownership of the
         /// payload which must not be accessed afterwards.
         /// </summary>
         /// <param name="payload">the serialized form of current message</param>
@@ -47,7 +47,7 @@ namespace Grpc.Core
         }
 
         /// <summary>
-        /// Complete the payload written to the buffer writer. Complete() can only be called once.
+        /// Complete the payload written to the buffer writer. <c>Complete()</c> can only be called once.
         /// </summary>
         public virtual void Complete()
         {
diff --git a/src/csharp/Grpc.Core.Tests/Internal/DefaultSerializationContextTest.cs b/src/csharp/Grpc.Core.Tests/Internal/DefaultSerializationContextTest.cs
index fcb12ae6656..061230d8ca4 100644
--- a/src/csharp/Grpc.Core.Tests/Internal/DefaultSerializationContextTest.cs
+++ b/src/csharp/Grpc.Core.Tests/Internal/DefaultSerializationContextTest.cs
@@ -189,9 +189,6 @@ namespace Grpc.Core.Internal.Tests
             }
         }
 
-        // other ideas:
-        // AdjustTailSpace(0) if previous tail size is 0... (better for SliceBufferSafeHandle)
-
         private DefaultSerializationContext.UsageScope NewDefaultSerializationContextScope()
         {
             return new DefaultSerializationContext.UsageScope(new DefaultSerializationContext());