Do not close stream after deserializing text kv1

Fixes #104
ValveResourceFormat · Sep 15, 2024 · beb1a8f · beb1a8f
1 parent baff467
commit beb1a8f
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 1 deletion.
diff --git a/ValveKeyValue/ValveKeyValue.Test/Text/StreamsTestCase.cs b/ValveKeyValue/ValveKeyValue.Test/Text/StreamsTestCase.cs
@@ -93,4 +93,14 @@ public void CanHandleBlockingStreams(int maxReadAtOnce)
 
         Assert.That(data["test"].ToInt32(CultureInfo.InvariantCulture), Is.EqualTo(1337));
     }
+
+    [Test]
+    public void LeavesStreamOpenAfterDeserialize()
+    {
+        using var stream = TestDataHelper.OpenResource("Text.object_person.vdf");
+
+        KVSerializer.Create(KVSerializationFormat.KeyValues1Text).Deserialize(stream);
+
+        Assert.That(stream.CanRead, Is.True);
+    }
 }
diff --git a/ValveKeyValue/ValveKeyValue/KVSerializer.cs b/ValveKeyValue/ValveKeyValue/KVSerializer.cs
@@ -114,7 +114,7 @@ IVisitingReader MakeReader(Stream stream, IParsingVisitationListener listener, K
 
             return format switch
             {
-                KVSerializationFormat.KeyValues1Text => new KV1TextReader(new StreamReader(stream), listener, options),
+                KVSerializationFormat.KeyValues1Text => new KV1TextReader(new StreamReader(stream, null, true, -1, leaveOpen: true), listener, options),
                 KVSerializationFormat.KeyValues1Binary => new KV1BinaryReader(stream, listener, options.StringTable),
                 _ => throw new ArgumentOutOfRangeException(nameof(format), format, "Invalid serialization format."),
             };