do not convert body encoding if it's not a text

lib/patron/response.rb

@@ -38,11 +38,14 @@ def initialize(url, status, redirect_count, header_data, body, default_charset =
 
       @charset        = determine_charset(header_data, body) || default_charset
 
-      [url, header_data, body].each do |attr|
+      [url, header_data].each do |attr|
         convert_to_default_encoding!(attr)
       end
 
       parse_headers(header_data)
+      if @headers["Content-Type"] && @headers["Content-Type"][0, 5] == "text/"
+        convert_to_default_encoding!(@body)
+      end
     end
 
     attr_reader :url, :status, :status_line, :redirect_count, :body, :headers, :charset

script/test_server

@@ -109,6 +109,13 @@ class RepetitiveHeaderServlet < HTTPServlet::AbstractServlet
   end
 end
 
+class PictureServlet < HTTPServlet::AbstractServlet
+  def do_GET(req, res)
+    res['Content-Type'] = "image/png"
+    res.body = File.read("./pic.png")
+  end
+end
+
 class WrongContentLengthServlet < HTTPServlet::AbstractServlet
   def do_GET(req, res)
     res.keep_alive = false
@@ -122,6 +129,7 @@ server.mount("/test", TestServlet)
 server.mount("/testpost", TestPostBodyServlet)
 server.mount("/timeout", TimeoutServlet)
 server.mount("/redirect", RedirectServlet)
+server.mount("/picture", PictureServlet)
 server.mount("/setcookie", SetCookieServlet)
 server.mount("/repetitiveheader", RepetitiveHeaderServlet)
 server.mount("/wrongcontentlength", WrongContentLengthServlet)

spec/response_spec.rb

@@ -43,6 +43,12 @@
     response.headers['Set-Cookie'].should == ["a=1","b=2"]
   end
 
+  it "should works with non-text files" do
+    response = @session.get("/picture")
+    response.headers['Content-Type'].should == 'image/png'
+    response.body.encoding.should == Encoding::ASCII_8BIT
+  end
+
   it "should not allow a default charset to be nil" do
     Encoding.stub(:default_internal).and_return("UTF-8")
     expect {