Skip to content

Commit 7073a76

Browse files
committed
Merge pull request #261 from quake/master
[performance] set default KeyFormatter to nil
2 parents 116fbb6 + d86454c commit 7073a76

File tree

1 file changed

+3
-3
lines changed

1 file changed

+3
-3
lines changed

lib/jbuilder.rb

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -5,13 +5,13 @@
55
require 'multi_json'
66

77
class Jbuilder
8-
@@key_formatter = KeyFormatter.new
8+
@@key_formatter = nil
99
@@ignore_nil = false
1010

1111
def initialize(options = {})
1212
@attributes = {}
1313

14-
@key_formatter = options.fetch(:key_formatter){ @@key_formatter.clone }
14+
@key_formatter = options.fetch(:key_formatter){ @@key_formatter ? @@key_formatter.clone : nil}
1515
@ignore_nil = options.fetch(:ignore_nil, @@ignore_nil)
1616

1717
yield self if ::Kernel.block_given?
@@ -275,7 +275,7 @@ def _merge_values(current_value, updates)
275275
end
276276

277277
def _key(key)
278-
@key_formatter.format(key)
278+
@key_formatter ? @key_formatter.format(key) : key.to_s
279279
end
280280

281281
def _set_value(key, value)

0 commit comments

Comments
 (0)