Browse Source

address a few TODOs

Jan Tattermusch 5 years ago
parent
commit
c06812b0b2
1 changed files with 2 additions and 4 deletions
  1. 2 4
      csharp/src/Google.Protobuf.Test/FieldCodecTest.cs

+ 2 - 4
csharp/src/Google.Protobuf.Test/FieldCodecTest.cs

@@ -124,11 +124,10 @@ namespace Google.Protobuf
             {
                 var stream = new MemoryStream();
                 var codedOutput = new CodedOutputStream(stream);
-
-                // TODO: simplify
                 WriteContext.Initialize(codedOutput, out WriteContext ctx);
                 try
                 {
+                    // only write the value using the codec
                     codec.ValueWriter(ref ctx, sampleValue);
                 }
                 finally
@@ -186,11 +185,10 @@ namespace Google.Protobuf
                 if (codec.DefaultValue != null) // This part isn't appropriate for message types.
                 {
                     codedOutput = new CodedOutputStream(stream);
-
-                    // TODO: simplify
                     WriteContext.Initialize(codedOutput, out WriteContext ctx);
                     try
                     {
+                        // only write the value using the codec
                         codec.ValueWriter(ref ctx, codec.DefaultValue);
                     }
                     finally