Remove redundant Debug.Assert; Minor refactor

pfpack · Dec 25, 2024 · d15f3ca · d15f3ca
1 parent 58eab52
commit d15f3ca
Showing 1 changed file with 4 additions and 13 deletions.
diff --git a/src/core-unit/Unit/UnitJsonConverter/UnitJsonConverter.cs b/src/core-unit/Unit/UnitJsonConverter/UnitJsonConverter.cs
@@ -1,5 +1,4 @@
-using System.Diagnostics;
-using System.Text.Json;
+using System.Text.Json;
 using System.Text.Json.Serialization;
 
 namespace System;
@@ -8,17 +7,11 @@ internal sealed class UnitJsonConverter : JsonConverter<Unit>
 {
     public override Unit Read(ref Utf8JsonReader reader, Type typeToConvert, JsonSerializerOptions options)
     {
-        var tokenType = reader.TokenType;
-        Debug.Assert(tokenType is not JsonTokenType.None);
-
-        switch (tokenType)
+        switch (reader.TokenType)
         {
             case JsonTokenType.StartObject:
             case JsonTokenType.StartArray:
                 reader.Skip();
-                Debug.Assert(
-                    tokenType is JsonTokenType.StartObject && reader.TokenType is JsonTokenType.EndObject ||
-                    tokenType is JsonTokenType.StartArray && reader.TokenType is JsonTokenType.EndArray);
                 return default;
 
             case JsonTokenType.String:
@@ -28,15 +21,13 @@ tokenType is JsonTokenType.StartObject && reader.TokenType is JsonTokenType.EndO
             case JsonTokenType.Null:
                 return default;
 
-            default:
-                throw new JsonException($"An unexpected JSON token type ({tokenType}).");
+            case var unexpected:
+                throw new JsonException($"An unexpected JSON token type ({unexpected}).");
         };
     }
 
     public override void Write(Utf8JsonWriter writer, Unit value, JsonSerializerOptions options)
     {
-        Debug.Assert(writer is not null);
-
         writer.WriteStartObject();
         writer.WriteEndObject();
     }