Permalink
Browse files

Disprefer JSONGem decoder since it only decodes JSON objects

  • Loading branch information...
jeremy committed Mar 9, 2010
1 parent ea4f8ef commit a87683fb38d6cf66f39a7bd3faa6c969c63b1f46
Showing with 1 addition and 1 deletion.
  1. +1 −1 activesupport/lib/active_support/json/decoding.rb
@@ -7,7 +7,7 @@ module ActiveSupport
module JSON
# Listed in order of preference.
- DECODERS = %w(Yajl JSONGem Yaml)

This comment has been minimized.

Show comment
Hide comment
@DevNerd

DevNerd Jan 29, 2013

WHY?!?

This comment has been minimized.

Show comment
Hide comment
@jeremy

jeremy Jan 29, 2013

Member

Because each JSON gem release would override all objects' #to_json in new and incompatible ways. Painful to interoperate with, whereas Yajl integration is clean and trouble-free.

@jeremy

jeremy Jan 29, 2013

Member

Because each JSON gem release would override all objects' #to_json in new and incompatible ways. Painful to interoperate with, whereas Yajl integration is clean and trouble-free.

+ DECODERS = %w(Yajl Yaml)
class << self
attr_reader :parse_error

0 comments on commit a87683f

Please sign in to comment.