Change Proposal for "String#word_wrap" (Ruby on Rails - ActionView::Helpers::TextHelper)

ruby-talk
1

Hi!

Hope this is the right place (I'm not a Ruby on Rails user).

The Method ist defined as (http://api.rubyonrails.org/classes/ActionView/Helpers/TextHelper.html#M000616):

def word_wrap(text, line_width = 80)
   text.gsub(/\n/, "\n\n").gsub(/(.{1,#{line_width}})(\s+|$)/, "\\1\n").strip
end

The sense of the part 'gsub(/\n/, "\n\n")' is not visible for me, because newlines will be removed in the next step by '\s+'.

I made some tests (Ruby, not Ruby on Rails) and couldn't find any difference to the following definition:

def word_wrap(text, line_width = 80)
   text.gsub(/(.{1,#{line_width}})(\s+|$)/, "\\1\n").strip
end

Wolfgang Nádasi-Donner

2

Wolfgang Nádasi-Donner wrote:

Hi!

Hope this is the right place (I'm not a Ruby on Rails user).

Rails topics are better addressed on the Rails mailing list.

List info may be found at http://www.rubyonrails.org

···

--
James Britt

"I never dispute another person's delusions, just their facts."
   - Len Bullard