shugo	2011-08-19 11:32:59 +0900 (Fri, 19 Aug 2011)

  New Revision: 33001

  http://svn.ruby-lang.org/cgi-bin/viewvc.cgi?view=rev&revision=33001

  Log:
    * lib/net/imap.rb (msg_att): accepts extra space before ')'.
      based on the patch by art lussos.  [Bug #5163] [ruby-core:38820]

  Modified files:
    trunk/ChangeLog
    trunk/lib/net/imap.rb
    trunk/test/net/imap/test_imap_response_parser.rb

Index: ChangeLog
===================================================================
--- ChangeLog	(revision 33000)
+++ ChangeLog	(revision 33001)
@@ -1,3 +1,8 @@
+Fri Aug 19 11:28:58 2011  Shugo Maeda  <shugo@r...>
+
+	* lib/net/imap.rb (msg_att): accepts extra space before ')'.
+	  based on the patch by art lussos.  [Bug #5163] [ruby-core:38820]
+
 Wed Aug 17 23:01:00 2011  Kenta Murata  <mrkn@m...>
 
 	* ext/bigdecimal/bigdecimal.c (cannot_be_coerced_into_BigDecimal):
Index: lib/net/imap.rb
===================================================================
--- lib/net/imap.rb	(revision 33000)
+++ lib/net/imap.rb	(revision 33001)
@@ -2170,7 +2170,7 @@
             break
           when T_SPACE
             shift_token
-            token = lookahead
+            next
           end
           case token.value
           when /\A(?:ENVELOPE)\z/ni
Index: test/net/imap/test_imap_response_parser.rb
===================================================================
--- test/net/imap/test_imap_response_parser.rb	(revision 33000)
+++ test/net/imap/test_imap_response_parser.rb	(revision 33001)
@@ -99,4 +99,21 @@
 EOF
     assert_equal [1, 2, 3], response.data
   end
+
+  def test_msg_att_extra_space
+    parser = Net::IMAP::ResponseParser.new
+    response = parser.parse(<<EOF.gsub(/\n/, "\r\n").taint)
+* 1 FETCH (UID 92285)
+EOF
+    assert_equal 92285, response.data.attr["UID"]
+
+    response = parser.parse(<<EOF.gsub(/\n/, "\r\n").taint)
+* 1 FETCH (UID 92285 )
+EOF
+    assert_equal 92285, response.data.attr["UID"]
+
+    response = parser.parse(<<EOF.gsub(/\n/, "\r\n").taint)
+* 1 FETCH (UID 92285  )
+EOF
+  end
 end

--
ML: ruby-changes@q...
Info: http://www.atdot.net/~ko1/quickml/