replace first and last characters check

3 files changed, 3 insertions, 3 deletions

diff --git a/lib/rack/media_type.rb b/lib/rack/media_type.rb
index 7e6cd3a8..f9397453 100644
--- a/lib/rack/media_type.rb
+++ b/lib/rack/media_type.rb
@@ -31,7 +31,7 @@ module Rack
       private
 
         def strip_doublequotes(str)
-          (str[0] == ?" && str[-1] == ?") ? str[1..-2] : str
+          (str.start_with?('"') && str.end_with?('"')) ? str[1..-2] : str
         end
     end
   end
diff --git a/lib/rack/multipart/parser.rb b/lib/rack/multipart/parser.rb
index 74a7ee67..f661da10 100644
--- a/lib/rack/multipart/parser.rb
+++ b/lib/rack/multipart/parser.rb
@@ -345,7 +345,7 @@ module Rack
               k,v = param.split('=', 2)
               k.strip!
               v.strip!
-              v = v[1..-2] if v[0] == '"' && v[-1] == '"'
+              v = v[1..-2] if v.start_with?('"') && v.end_with?('"')
               encoding = Encoding.find v if k == CHARSET
             end
           end
diff --git a/lib/rack/request.rb b/lib/rack/request.rb
index 5bf3eb17..e2d2bb56 100644
--- a/lib/rack/request.rb
+++ b/lib/rack/request.rb
@@ -337,7 +337,7 @@ module Rack
 
             # Fix for Safari Ajax postings that always append \0
             # form_vars.sub!(/\0\z/, '') # performance replacement:
-            form_vars.slice!(-1) if form_vars[-1] == ?\0
+            form_vars.slice!(-1) if form_vars.end_with?("\0")
 
             set_header RACK_REQUEST_FORM_VARS, form_vars
             set_header RACK_REQUEST_FORM_HASH, parse_query(form_vars, '&')