Excerpts from Israel Herraiz's message of Tue Nov 04 01:08:41 +0100 2008:
> Hi,
Hi,
> this patch wrap text attachaments, as well as messages. For me is
> quite annoying to have to scroll to see the text attachments. Perhaps
> it is useful for others too.
I have no opinion about this change.
However there is already a WRAP_LEN constant in this file, please provide
a patch that reuse the same constant.
Best regards,
> ---
> lib/sup/message-chunks.rb | 3 +++
> 1 files changed, 3 insertions(+), 0 deletions(-)
>
> diff --git a/lib/sup/message-chunks.rb b/lib/sup/message-chunks.rb
> index cc895f3..99bd12b 100644
> --- a/lib/sup/message-chunks.rb
> +++ b/lib/sup/message-chunks.rb
> @@ -65,6 +65,8 @@ Return value:
> EOS
> #' stupid ruby-mode
>
> + WRAP_LEN = 80 # wrap text attachments at this width
> +
> ## raw_content is the post-MIME-decode content. this is used for
> ## saving the attachment to disk.
> attr_reader :content_type, :filename, :lines, :raw_content
> @@ -95,6 +97,7 @@ EOS
> @lines = nil
> if text
> @lines = text.gsub("\r\n", "\n").gsub(/\t/, " ").gsub(/\r/,
> "").split("\n")
> + @lines = lines.map {|l| l.chomp.wrap WRAP_LEN}.flatten
> @quotable = true
> end
> end
--
Nicolas Pouillard aka Ertai
_______________________________________________
sup-talk mailing list
[email protected]
http://rubyforge.org/mailman/listinfo/sup-talk
