String length is now UInt16

Steve Salas · Steve Salas · commit d1e811ef102d · 2018-03-27T07:52:33.000-04:00
diff --git a/dotnet-tracer/main/CodePulse.Client/Util/BinaryReaderExtensions.cs b/dotnet-tracer/main/CodePulse.Client/Util/BinaryReaderExtensions.cs
@@ -90,7 +90,7 @@ public static string ReadUtfBigEndian(this BinaryReader reader)
                 throw new ArgumentNullException(nameof(reader), "Expected non-null BinaryReader");
             }
 
-            var stringBytesLength = reader.ReadInt16BigEndian();
+            var stringBytesLength = reader.ReadUInt16BigEndian();
             var stringBytes = ReadBytesBigEndian(reader, stringBytesLength);
 
             Array.Reverse(stringBytes);

Original file line number	Diff line number	Diff line change
`@@ -90,7 +90,7 @@ public static string ReadUtfBigEndian(this BinaryReader reader)`
`90`	`90`	`throw new ArgumentNullException(nameof(reader), "Expected non-null BinaryReader");`
`91`	`91`	`}`
`92`	`92`
`93`		`- var stringBytesLength = reader.ReadInt16BigEndian();`
	`93`	`+ var stringBytesLength = reader.ReadUInt16BigEndian();`
`94`	`94`	`var stringBytes = ReadBytesBigEndian(reader, stringBytesLength);`
`95`	`95`
`96`	`96`	`Array.Reverse(stringBytes);`