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

[Wolfgang Nádasi-Donner]

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

 

[James Britt]

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