String Manipulation and Comparison Operations

lib/http/content_type.rb

@@ -13,14 +13,14 @@ def parse(str)
 
       # :nodoc:
       def mime_type(str)
-        md = str.to_s.match MIME_TYPE_RE
-        md && md[1].to_s.strip.downcase
+        m = str.to_s[MIME_TYPE_RE, 1]
+        m && m.strip.downcase
       end
 
       # :nodoc:
       def charset(str)
-        md = str.to_s.match CHARSET_RE
-        md && md[1].to_s.strip.gsub(/^"|"$/, "")
+        m = str.to_s[CHARSET_RE, 1]
+        m && m.strip.delete('"')
       end
     end
   end

lib/http/redirector.rb

@@ -68,13 +68,13 @@ def perform(request, response)
     # Check if we reached max amount of redirect hops
     # @return [Boolean]
     def too_many_hops?
-      1 <= @max_hops && @max_hops < @visited.count
+      @max_hops > 0 && @visited.count > @max_hops
     end
 
     # Check if we got into an endless loop
     # @return [Boolean]
     def endless_loop?
-      2 <= @visited.count(@visited.last)
+      @visited.count(@visited.last) > 1
     end
 
     # Redirect policy for follow
@@ -90,7 +90,7 @@ def redirect_to(uri)
         verb = :get
       end
 
-      verb = :get if !SEE_OTHER_ALLOWED_VERBS.include?(verb) && 303 == code
+      verb = :get if !SEE_OTHER_ALLOWED_VERBS.include?(verb) && code == 303
 
       @request.redirect(uri, verb)
     end

lib/http/response/status.rb

@@ -43,7 +43,7 @@ def coerce(object)
         # @param [#to_s] str
         # @return [Symbol]
         def symbolize(str)
-          str.to_s.downcase.tr("-", " ").gsub(/[^a-z ]/, "").gsub(/\s+/, "_").to_sym
+          str.to_s.downcase.tr("- ", "_").delete("^a-z_").to_sym
         end
       end
 
@@ -132,7 +132,7 @@ def inspect
       SYMBOLS.each do |code, symbol|
         class_eval <<-RUBY, __FILE__, __LINE__
           def #{symbol}?      # def bad_request?
-            #{code} == code   #   400 == code
+            code == #{code}   #   code == 400
           end                 # end
         RUBY
       end