Fix double flushing json/yaml builders (#10716)

crystal-lang · May 21, 2021 · 4af98d4 · 4af98d4
1 parent 60d227c
commit 4af98d4
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 4 deletions.
diff --git a/src/json/builder.cr b/src/json/builder.cr
@@ -53,7 +53,7 @@ class JSON::Builder
     when DocumentEndState
       # okay
     end
-    @io.flush
+    flush
   end
 
   def document
@@ -415,6 +415,5 @@ module JSON
     builder.document do
       yield builder
     end
-    io.flush
   end
 end
diff --git a/src/yaml/builder.cr b/src/yaml/builder.cr
@@ -53,7 +53,6 @@ class YAML::Builder
   def self.build(io : IO, & : self ->) : Nil
     builder = new(io)
     yield builder ensure builder.close
-    io.flush
   end
 
   # Starts a YAML stream.
@@ -64,7 +63,7 @@ class YAML::Builder
   # Ends a YAML stream.
   def end_stream
     emit stream_end
-    @io.flush
+    flush
   end
 
   # Starts a YAML stream, invokes the block, and ends it.