Eliminate some extra reader copies (#131)

serdedotnet · Apr 20, 2023 · 5380dd3 · 5380dd3
1 parent 2a8290a
commit 5380dd3
Showing 1 changed file with 10 additions and 19 deletions.
diff --git a/src/serde/json/JsonDeserializer.cs b/src/serde/json/JsonDeserializer.cs
@@ -35,9 +35,9 @@ private void SaveState(in Utf8JsonReader reader)
             _state.Reader = reader;
         }
 
-        private Utf8JsonReader GetReader()
+        private ref Utf8JsonReader GetReader()
         {
-            return _state.Reader;
+            return ref _state.Reader;
         }
 
         public T DeserializeAny<T, V>(V v) where V : IDeserializeVisitor<T>
@@ -76,24 +76,22 @@ private Utf8JsonReader GetReader()
 
         public T DeserializeBool<T, V>(V v) where V : IDeserializeVisitor<T>
         {
-            var reader = GetReader();
+            ref var reader = ref GetReader();
             reader.ReadOrThrow();
             bool b = reader.GetBoolean();
-            SaveState(reader);
             return v.VisitBool(b);
         }
 
         public T DeserializeDictionary<T, V>(V v) where V : IDeserializeVisitor<T>
         {
-            var reader = GetReader();
+            ref var reader = ref GetReader();
             reader.ReadOrThrow();
 
             if (reader.TokenType != JsonTokenType.StartObject)
             {
                 throw new InvalidDeserializeValueException("Expected object start");
             }
 
-            SaveState(reader);
             var map = new DeDictionary(this);
             return v.VisitDictionary(ref map);
         }
@@ -103,33 +101,30 @@ private Utf8JsonReader GetReader()
 
         public T DeserializeDouble<T, V>(V v) where V : IDeserializeVisitor<T>
         {
-            var reader = GetReader();
+            ref var reader = ref GetReader();
             reader.ReadOrThrow();
             var d = reader.GetDouble();
-            SaveState(reader);
             return v.VisitDouble(d);
         }
 
         public T DeserializeDecimal<T, V>(V v) where V : IDeserializeVisitor<T>
         {
-            var reader = GetReader();
+            ref var reader = ref GetReader();
             reader.ReadOrThrow();
             var d = reader.GetDecimal();
-            SaveState(reader);
             return v.VisitDecimal(d);
         }
 
         public T DeserializeEnumerable<T, V>(V v) where V : IDeserializeVisitor<T>
         {
-            var reader = GetReader();
+            ref var reader = ref GetReader();
             reader.ReadOrThrow();
 
             if (reader.TokenType != JsonTokenType.StartArray)
             {
                 throw new InvalidDeserializeValueException("Expected array start");
             }
 
-            SaveState(reader);
             var enumerable = new DeEnumerable(this);
             return v.VisitEnumerable(ref enumerable);
         }
@@ -232,27 +227,24 @@ public DeDictionary(JsonDeserializer de)
 
         public T DeserializeI64<T, V>(V v) where V : IDeserializeVisitor<T>
         {
-            var reader = GetReader();
+            ref var reader = ref GetReader();
             reader.ReadOrThrow();
             var i64 = reader.GetInt64();
-            SaveState(reader);
             return v.VisitI64(i64);
         }
 
         public T DeserializeString<T, V>(V v) where V : IDeserializeVisitor<T>
         {
-            var reader = GetReader();
+            ref var reader = ref GetReader();
             reader.ReadOrThrow();
             if (reader.HasValueSequence || reader.ValueIsEscaped)
             {
                 var s = reader.GetString()!;
-                SaveState(reader);
                 return v.VisitString(s);
             }
             else
             {
                 var result = v.VisitUtf8Span(reader.ValueSpan);
-                SaveState(reader);
                 return result;
             }
         }
@@ -277,10 +269,9 @@ public DeDictionary(JsonDeserializer de)
 
         public T DeserializeU64<T, V>(V v) where V : IDeserializeVisitor<T>
         {
-            var reader = GetReader();
+            ref var reader = ref GetReader();
             reader.ReadOrThrow();
             var u64 = reader.GetUInt64();
-            SaveState(reader);
             return v.VisitU64(u64);
         }