Merge pull request #252 from jlahtinen/master

Remove usage of Rack::Utils.bytesize

Author: headius

src/main/ruby/jruby/rack/error_app.rb

@@ -70,15 +70,14 @@ def serve(code, path, env)
         headers = { 'Last-Modified' => last_modified }
         DEFAULT_HEADERS.each { |field, content| headers[field] = content }
         ext = File.extname(path)
+        size = File.size?(path)
         mime = ::Rack::Mime.mime_type(ext, DEFAULT_MIME) if defined?(::Rack::Mime)
         mime = 'text/html' if ! mime && ( ext == '.html' || ext == '.htm' )
         headers['Content-Type'] = mime if mime
 
-        body = env['REQUEST_METHOD'] == 'HEAD' ? [] : FileBody.new(path, size = File.size?(path))
+        body = env['REQUEST_METHOD'] == 'HEAD' ? [] : FileBody.new(path, size)
         response = [ code, headers, body ]
 
-        size ||= ::Rack::Utils.bytesize(File.read(path)) if defined?(::Rack::Utils.bytesize)
-
         response[1]['Content-Length'] = size.to_s if size
         response
       end
@@ -169,4 +168,4 @@ def http_accept?(env, mime)
 
     end
   end
-end
+end

src/main/ruby/jruby/rack/error_app/show_status.rb

@@ -27,7 +27,7 @@ def call(env)
         detail = detail = env["rack.showstatus.detail"] || message
 
         body = @template.result(binding)
-        size = ::Rack::Utils.bytesize(body)
+        size = body.bytesize
         [status, headers.merge('Content-Type' => "text/html", 'Content-Length' => size.to_s), [body]]
       else
         [status, headers, body]