Labyricorn/liquid
1
0
Fork 0
mirror of https://github.com/Shopify/liquid.git synced 2026-09-12 23:40:45 -07:00
Code Issues Packages Projects Releases Wiki Activity

avoid allocating seen={} hash in Utils.to_s/inspect when not needed

Browse Source
This commit is contained in:
Tobi Lutke
2026-03-11 09:02:42 -04:00
parent c4186a16fc
commit 3799d4c488
1 changed files with 6 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files
lib/liquid/utils.rb
+6 -6
View File
@@ -96,7 +96,7 @@ module Liquid
obj
end
def self.to_s(obj, seen = {})
def self.to_s(obj, seen = nil)
case obj
when BigDecimal
obj.to_s("F")
@@ -105,30 +105,30 @@ module Liquid
# custom implementation. Otherwise we use Liquid's default
# implementation.
if obj.class.instance_method(:to_s) == HASH_TO_S_METHOD
hash_inspect(obj, seen)
hash_inspect(obj, seen || {})
else
obj.to_s
end
when Array
array_inspect(obj, seen)
array_inspect(obj, seen || {})
else
obj.to_s
end
end
def self.inspect(obj, seen = {})
def self.inspect(obj, seen = nil)
case obj
when Hash
# If the custom hash implementation overrides `#inspect`, use their
# custom implementation. Otherwise we use Liquid's default
# implementation.
if obj.class.instance_method(:inspect) == HASH_INSPECT_METHOD
hash_inspect(obj, seen)
hash_inspect(obj, seen || {})
else
obj.inspect
end
when Array
array_inspect(obj, seen)
array_inspect(obj, seen || {})
else
obj.inspect
end