Merge pull request #713 from byroot/avoid-require

tenderlove · web-flow · commit 7da76a717fcd · 2025-02-06T17:08:47.000-05:00
Avoid calls to `require` in hotspots
diff --git a/lib/psych/nodes/node.rb b/lib/psych/nodes/node.rb
@@ -55,7 +55,8 @@ def to_ruby(symbolize_names: false, freeze: false, strict_integer: false)
       #
       # See also Psych::Visitors::Emitter
       def yaml io = nil, options = {}
-        require "stringio"
+        require "stringio" unless defined?(StringIO)
+
         real_io = io || StringIO.new(''.encode('utf-8'))
 
         Visitors::Emitter.new(real_io, options).accept self

Original file line number	Diff line number	Diff line change
`@@ -55,7 +55,8 @@ def to_ruby(symbolize_names: false, freeze: false, strict_integer: false)`
`55`	`55`	`#`
`56`	`56`	`# See also Psych::Visitors::Emitter`
`57`	`57`	`def yaml io = nil, options = {}`
`58`		`- require "stringio"`
	`58`	`+ require "stringio" unless defined?(StringIO)`
	`59`	`+`
`59`	`60`	`real_io = io \|\| StringIO.new(''.encode('utf-8'))`
`60`	`61`
`61`	`62`	`Visitors::Emitter.new(real_io, options).accept self`