[ruby-core:35551] [Ruby 1.8 - Bug #4498][Open] [Patch] REXML Pretty formater does use specified 'width' to wrap lines

REXML::Formatters::Pretty has 'width' attribute used to wrap lines.
This is not used when the wrap method is invoked.

The pretty formatter within the REXML library is used to nicely indent
xml files for viewing. I use this feature to present XML files to a
user for editing. The width attribute is available to set a maximum
line width, and force text wrapping for nodes with long text values.

There are two modes for the pretty formatter - @compact =3D { true |
false }. When compact is on, @width is used for text wrapping under
the condition that ...

"If compact and all children are text, and if the formatted output is
less than the specified width, then try to print everything on one
line"

But in the case when @compact =3D false, I believe that the node text
should still be wrapped based on user specification. In the current
state, max width is hard coded to 80 chars.

# lib/rexml/formatters/pretty.rb
def write_text( node, output )
=A0 s =3D node.to_s()
=A0 s.gsub!(/\s/,' ')
=A0 s.squeeze!(" ")
=A0 s =3D wrap(s, 80-@level)     ##  <-- HERE: is hard coded, value should
depend on @width and @level (e.g. @width - @level)
=A0 s =3D indent_text(s, @level, " ", true)
=A0 output << (' '*@level + s)
end

I've attached a patch for this issue. I've raised bug #4497 for the
same issue in 1.8.7 branch

-Michael Frasca