Remove AbstractResponse#to_i, which is too unexpected, considering:

[godfat]

require 'webmock'
require 'rest-client'
WebMock::API.stub_request(:get, 'http://host/').to_return(:body => '123')

RestClient.get('http://host/')        # "123"
RestClient.get('http://host/').to_s   # "123"
RestClient.get('http://host/').to_str # "123"
RestClient.get('http://host/').class  # String

RestClient.get('http://host/').to_i         # 200
RestClient.get('http://host/').to_s.to_i    # 200
RestClient.get('http://host/').to_str.to_i  # 200
String(RestClient.get('http://host/')).to_i # 200

RestClient.get('http://host/').dup.to_i         # 123
String.new(RestClient.get('http://host/')).to_i # 123
"#{RestClient.get('http://host/')}".to_i        # 123

RestClient.get('http://host/').clone.to_i       # 200

[ayamomiji]

+1 for this, i just wasted too much time on this...

[L2G]

(For reference only: This is one of many patches included in #106.)

[dinatih]

+1 please

[ab]

Ugh, this behavior is painful. I'm wary of removing it outright though, since it's been around for so long, and people might be relying on it.