Permalink
Browse files

Merge pull request #327 from kennyj/ambiguous_first_argument_put_pare…

…ntheses_or_even_spaces

Fix warning: ambiguous first argument; put parentheses or even spaces
  • Loading branch information...
2 parents 51e830e + 586bac1 commit 0e1b2a63f9447140132e2328457536b2d3ba2fd4 @jstirk jstirk committed Jan 14, 2012
Showing with 2 additions and 2 deletions.
  1. +1 −1 spec/mail/mime_messages_spec.rb
  2. +1 −1 spec/mail/part_spec.rb
@@ -468,7 +468,7 @@
m.parts.length.should eq 2
m.parts.first[:content_type].content_type.should eq 'image/png'
m.parts.last[:content_type].content_type.should eq 'text/plain'
- m.parts.last.to_s.should match /^First Line\r\n\r\nSecond Line\r\n\r\nThird Line/
+ m.parts.last.to_s.should match(/^First Line\r\n\r\nSecond Line\r\n\r\nThird Line/)
end
it "should not raise a warning if there is a charset defined and there are non ascii chars in the body" do
View
@@ -147,7 +147,7 @@
#Note: trailing \n\n is stripped off by Mail::Part initialization
part = Mail::Part.new(plain_text)
part[:content_type].content_type.should eq 'text/plain'
- part.to_s.should match /^First Line\r\n\r\nSecond Line\r\n\r\nThird Line/
+ part.to_s.should match(/^First Line\r\n\r\nSecond Line\r\n\r\nThird Line/)
end
end

0 comments on commit 0e1b2a6

Please sign in to comment.