diff options
author | jeg2 <jeg2@b2dd03c8-39d4-4d8f-98ff-823fe69b080e> | 2007-12-26 04:56:03 +0000 |
---|---|---|
committer | jeg2 <jeg2@b2dd03c8-39d4-4d8f-98ff-823fe69b080e> | 2007-12-26 04:56:03 +0000 |
commit | f672f6ec9fc15f49110cc623846f9571949e656a (patch) | |
tree | ac21837deb2b285209c90a50256ae8c2c3a260b0 | |
parent | a2f4666e76373f744354e30ce2326327f8eb00ec (diff) | |
download | ruby-f672f6ec9fc15f49110cc623846f9571949e656a.tar.gz ruby-f672f6ec9fc15f49110cc623846f9571949e656a.tar.xz ruby-f672f6ec9fc15f49110cc623846f9571949e656a.zip |
* lib/csv.rb: Cleaned up some code with Ruby 1.9 idioms.
git-svn-id: http://svn.ruby-lang.org/repos/ruby/trunk@14715 b2dd03c8-39d4-4d8f-98ff-823fe69b080e
-rw-r--r-- | ChangeLog | 4 | ||||
-rw-r--r-- | lib/csv.rb | 16 |
2 files changed, 10 insertions, 10 deletions
@@ -1,3 +1,7 @@ +Wed Dec 26 13:55:02 2007 James Edward Gray II <jeg2@ruby-lang.org> + + * lib/csv.rb: Cleaned up some code with Ruby 1.9 idioms. + Wed Dec 26 13:29:35 2007 Nobuyoshi Nakada <nobu@ruby-lang.org> * array.c (tmpbuf): use rb_str_tmp_new(). diff --git a/lib/csv.rb b/lib/csv.rb index b12142e50..f488141eb 100644 --- a/lib/csv.rb +++ b/lib/csv.rb @@ -187,12 +187,10 @@ class CSV @header_row = header_row # handle extra headers or fields - larger, smaller, transform = headers.size > fields.size ? - [headers, fields, :to_a] : - [fields, headers, :reverse] - @row = Array.new - larger.each_with_index do |e, i| - @row << [e, smaller[i]].send(transform) + @row = if headers.size > fields.size + headers.each_with_index.map { |header, i| [header, fields[i]] } + else + fields.each_with_index.map { |field, i| [headers[i], field] } end end @@ -1814,8 +1812,7 @@ class CSV # see if we are converting headers or fields converters = headers ? @header_converters : @converters - converted = Array.new - fields.each_with_index do |field, index| + fields.each_with_index.map do |field, index| # map_with_index converters.each do |converter| field = if converter.arity == 1 # straight field converter converter[field] @@ -1825,9 +1822,8 @@ class CSV end break unless field.is_a? String # short-curcuit pipeline for speed end - converted << field # final state of each field, converted or original + field # final state of each field, converted or original end - converted end # |