Subject: [sup-devel] [PATCH] fix missing parentheses warning

From: Rich Lane <[email-masked]>
Date: Mon, 25 Jan 2010 16:52:23 -0500

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

diff --git a/lib/sup/message.rb b/lib/sup/message.rb
index 3e55de5..a85cc0d 100644
--- a/lib/sup/message.rb
+++ b/lib/sup/message.rb
@@ -450,9 +450,9 @@ private
         subj = decode_header_field(payload.header.subject) || DEFAULT_SUBJECT
         subj = Message.normalize_subj(subj.gsub(/\s+/, " ").gsub(/\s+$/, ""))
         msgdate = payload.header.date
-        from_person = from ? Person.from_address(decode_header_field from) : nil
-        to_people = to ? Person.from_address_list(decode_header_field to) : nil
-        cc_people = cc ? Person.from_address_list(decode_header_field cc) : nil
+        from_person = from ? Person.from_address(decode_header_field(from)) : nil
+        to_people = to ? Person.from_address_list(decode_header_field(to)) : nil
+        cc_people = cc ? Person.from_address_list(decode_header_field(cc)) : nil
         [Chunk::EnclosedMessage.new(from_person, to_people, cc_people, msgdate, subj)] + message_to_chunks(payload, encrypted)
       else
         debug "no body for message/rfc822 enclosure; skipping"
-- 
1.5.6.5

_______________________________________________
Sup-devel mailing list
[email-masked]
http://rubyforge.org/mailman/listinfo/sup-devel

---

From: William Morgan <[email-masked]>
Date: Tue, 26 Jan 2010 08:20:58 -0500

Applied directly to master. Thanks!

What do people think of turning warnings off entirely for bin/sup?
Should just be a matter of setting $VERBOSE = nil. It is a curses app,
after all.
-- 
William <[email-masked]>
_______________________________________________
Sup-devel mailing list
[email-masked]
http://rubyforge.org/mailman/listinfo/sup-devel