email_reply_parser 0.5.0 → 0.5.1

data/email_reply_parser.gemspec

@@ -13,8 +13,8 @@ Gem::Specification.new do |s|
   ## If your rubyforge_project name is different, then edit it and comment out
   ## the sub! line in the Rakefile
   s.name              = 'email_reply_parser'
-  s.version           = '0.5.0'
-  s.date              = '2012-03-01'
+  s.version           = '0.5.1'
+  s.date              = '2012-05-01'
   s.rubyforge_project = 'email_reply_parser'
 
   ## Make sure your summary is short. The description may be as long

data/lib/email_reply_parser.rb

@@ -30,7 +30,7 @@ require 'strscan'
 #
 # [mail]: https://github.com/mikel/mail
 class EmailReplyParser
-  VERSION = "0.5.0"
+  VERSION = "0.5.1"
 
   # Public: Splits an email body into a list of Fragments.
   #
@@ -76,9 +76,12 @@ class EmailReplyParser
     #
     # Returns this same Email instance.
     def read(text)
+      # in 1.9 we want to operate on the raw bytes
+      text = text.dup.force_encoding('binary') if text.respond_to?(:force_encoding)
+
       # Check for multi-line reply headers. Some clients break up
       # the "On DATE, NAME <EMAIL> wrote:" line into multiple lines.
-      if text =~ /^(On(.+)wrote:)$/m
+      if text =~ /^(On(.+)wrote:)$/nm
         # Remove all new lines from the reply header.
         text.gsub! $1, $1.gsub("\n", " ")
       end
@@ -98,7 +101,7 @@ class EmailReplyParser
 
       # Use the StringScanner to pull out each line of the email content.
       @scanner = StringScanner.new(text)
-      while line = @scanner.scan_until(/\n/)
+      while line = @scanner.scan_until(/\n/n)
         scan_line(line)
       end
 
@@ -121,8 +124,8 @@ class EmailReplyParser
 
   private
     EMPTY = "".freeze
-    SIG_REGEX = /(--|__|\w-$)|(^(\w+\s*){1,3} #{"Sent from my".reverse}$)/
-    
+    SIG_REGEX = /(--|__|\w-$)|(^(\w+\s*){1,3} #{"Sent from my".reverse}$)/n
+
     ### Line-by-Line Parsing
 
     # Scans the given line of text and figures out which fragment it belongs
@@ -137,7 +140,7 @@ class EmailReplyParser
 
       # We're looking for leading `>`'s to see if this line is part of a
       # quoted Fragment.
-      is_quoted = !!(line =~ /(>+)$/)
+      is_quoted = !!(line =~ /(>+)$/n)
 
       # Mark the current Fragment as a signature if the current line is empty
       # and the Fragment starts with a common signature indicator.
@@ -170,7 +173,7 @@ class EmailReplyParser
     #
     # Returns true if the line is a valid header, or false.
     def quote_header?(line)
-      line =~ /^:etorw.*nO$/
+      line =~ /^:etorw.*nO$/n
     end
 
     # Builds the fragment string and reverses it, after all lines have been

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: email_reply_parser
 version: !ruby/object:Gem::Version
-  version: 0.5.0
+  version: 0.5.1
   prerelease: 
 platform: ruby
 authors:
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2012-03-01 00:00:00.000000000 Z
+date: 2012-05-01 00:00:00.000000000 Z
 dependencies: []
 description: Long description. Maybe copied from the README.
 email: technoweenie@gmail.com
@@ -59,9 +59,10 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubyforge_project: email_reply_parser
-rubygems_version: 1.8.11
+rubygems_version: 1.8.10
 signing_key: 
 specification_version: 2
 summary: Short description used in Gem listings.
 test_files:
 - test/email_reply_parser_test.rb
+has_rdoc: