aboutsummaryrefslogtreecommitdiffstats
path: root/activesupport/lib/active_support/json/decoding.rb
diff options
context:
space:
mode:
authorRick Olson <technoweenie@gmail.com>2007-12-28 18:37:17 +0000
committerRick Olson <technoweenie@gmail.com>2007-12-28 18:37:17 +0000
commit9e2536393e9a41f53bf543d0ea39a16b96e9bc17 (patch)
tree0935676c8fb7017f77ee3fbbbd50b8d70fcff56b /activesupport/lib/active_support/json/decoding.rb
parent0e3a54a3b92c0f9a7bf95f8cb11ba2db89f6eecc (diff)
downloadrails-9e2536393e9a41f53bf543d0ea39a16b96e9bc17.tar.gz
rails-9e2536393e9a41f53bf543d0ea39a16b96e9bc17.tar.bz2
rails-9e2536393e9a41f53bf543d0ea39a16b96e9bc17.zip
remove multiple enumerations from ActiveSupport::JSON#convert_json_to_yaml when dealing with date/time values. [rick]
git-svn-id: http://svn-commit.rubyonrails.org/rails/trunk@8505 5ecf4fe2-1ee6-0310-87b1-e25e094e27de
Diffstat (limited to 'activesupport/lib/active_support/json/decoding.rb')
-rw-r--r--activesupport/lib/active_support/json/decoding.rb13
1 files changed, 8 insertions, 5 deletions
diff --git a/activesupport/lib/active_support/json/decoding.rb b/activesupport/lib/active_support/json/decoding.rb
index f24aa9e57c..c58001f49f 100644
--- a/activesupport/lib/active_support/json/decoding.rb
+++ b/activesupport/lib/active_support/json/decoding.rb
@@ -45,11 +45,14 @@ module ActiveSupport
if marks.empty?
json.gsub(/\\\//, '/')
else
- # FIXME: multiple slow enumerations
- output = ([0] + marks.map(&:succ)).
- zip(marks + [json.length]).
- map { |left, right| json[left..right] }.
- join(" ")
+ left_pos = [-1].push(*marks)
+ right_pos = marks << json.length
+ output = []
+ left_pos.each_with_index do |left, i|
+ output << json[left.succ..right_pos[i]]
+ end
+ output = output * " "
+
times.each { |i| output[i-1] = ' ' }
output.gsub!(/\\\//, '/')
output