From mboxrd@z Thu Jan  1 00:00:00 1970
From: israel.herraiz@gmail.com (Israel Herraiz)
Date: Tue, 04 Nov 2008 15:07:21 +0100
Subject: [sup-talk] [PATCH] Wrap text attachments
In-Reply-To: <1225805250-sup-3958@ausone.local>
References: <1225757268-sup-9251@elly> <1225805250-sup-3958@ausone.local>
Message-ID: <1225807479-sup-1191@elly>

Excerpts from Nicolas's message on Nov  4, 2008 about  2 PM:
> However there is already a WRAP_LEN constant in this file, please provide
> a patch that reuse the same constant.

Find it below. I have moved the constant out of the Attachment and
Text classes, and it is now in the context of the Chunk module.

Cheers,
Israel


---
 lib/sup/message-chunks.rb |    4 +++-
 1 files changed, 3 insertions(+), 1 deletions(-)

diff --git a/lib/sup/message-chunks.rb b/lib/sup/message-chunks.rb
index cc895f3..5b5c314 100644
--- a/lib/sup/message-chunks.rb
+++ b/lib/sup/message-chunks.rb
@@ -41,6 +41,8 @@ end
 
 module Redwood
 module Chunk
+  WRAP_LEN = 80 # wrap messages and text attachments at this width
+
   class Attachment
     HookManager.register "mime-decode", <<EOS
 Executes when decoding a MIME attachment.
@@ -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
@@ -143,7 +146,6 @@ EOS
   end
 
   class Text
-    WRAP_LEN = 80 # wrap at this width
 
     attr_reader :lines
     def initialize lines
-- 
1.5.6.5