You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hc.apache.org by ol...@apache.org on 2012/07/05 10:56:58 UTC
svn commit: r1357500 - in /httpcomponents/httpasyncclient/trunk:
httpasyncclient/src/test/java/org/apache/http/nio/client/methods/TestZeroCopy.java
pom.xml
Author: olegk
Date: Thu Jul 5 08:56:58 2012
New Revision: 1357500
URL: http://svn.apache.org/viewvc?rev=1357500&view=rev
Log:
Upgraded HttpCore to version 4.2.1; upgraded HttpClient to version 4.2.1; downgraded Commons IO to version 2.2
Modified:
httpcomponents/httpasyncclient/trunk/httpasyncclient/src/test/java/org/apache/http/nio/client/methods/TestZeroCopy.java
httpcomponents/httpasyncclient/trunk/pom.xml
Modified: httpcomponents/httpasyncclient/trunk/httpasyncclient/src/test/java/org/apache/http/nio/client/methods/TestZeroCopy.java
URL: http://svn.apache.org/viewvc/httpcomponents/httpasyncclient/trunk/httpasyncclient/src/test/java/org/apache/http/nio/client/methods/TestZeroCopy.java?rev=1357500&r1=1357499&r2=1357500&view=diff
==============================================================================
--- httpcomponents/httpasyncclient/trunk/httpasyncclient/src/test/java/org/apache/http/nio/client/methods/TestZeroCopy.java (original)
+++ httpcomponents/httpasyncclient/trunk/httpasyncclient/src/test/java/org/apache/http/nio/client/methods/TestZeroCopy.java Thu Jul 5 08:56:58 2012
@@ -39,6 +39,7 @@ import org.apache.commons.io.FileUtils;
import org.apache.commons.io.IOUtils;
import org.apache.commons.io.LineIterator;
import org.apache.commons.io.output.FileWriterWithEncoding;
+import org.apache.http.Consts;
import org.apache.http.HttpAsyncTestBase;
import org.apache.http.HttpEntity;
import org.apache.http.HttpEntityEnclosingRequest;
@@ -231,7 +232,11 @@ public class TestZeroCopy extends HttpAs
InputStream instream = requestEntity.getContent();
try {
ContentType contentType = ContentType.getOrDefault(requestEntity);
- LineIterator it = IOUtils.lineIterator(instream, contentType.getCharset());
+ Charset charset = contentType.getCharset();
+ if (charset == null) {
+ charset = Consts.ISO_8859_1;
+ }
+ LineIterator it = IOUtils.lineIterator(instream, charset.name());
int count = 0;
while (it.hasNext()) {
String line = it.next();
Modified: httpcomponents/httpasyncclient/trunk/pom.xml
URL: http://svn.apache.org/viewvc/httpcomponents/httpasyncclient/trunk/pom.xml?rev=1357500&r1=1357499&r2=1357500&view=diff
==============================================================================
--- httpcomponents/httpasyncclient/trunk/pom.xml (original)
+++ httpcomponents/httpasyncclient/trunk/pom.xml Thu Jul 5 08:56:58 2012
@@ -28,7 +28,7 @@
<parent>
<artifactId>project</artifactId>
<groupId>org.apache.httpcomponents</groupId>
- <version>5</version>
+ <version>6</version>
<relativePath>../project/pom.xml</relativePath>
</parent>
<modelVersion>4.0.0</modelVersion>
@@ -71,10 +71,10 @@
<maven.compile.target>1.5</maven.compile.target>
<maven.compile.optimize>true</maven.compile.optimize>
<maven.compile.deprecation>true</maven.compile.deprecation>
- <httpcore.version>4.2</httpcore.version>
- <httpclient.version>4.2-beta1</httpclient.version>
+ <httpcore.version>4.2.1</httpcore.version>
+ <httpclient.version>4.2.1</httpclient.version>
<commons-logging.version>1.1.1</commons-logging.version>
- <commons-io.version>2.3</commons-io.version>
+ <commons-io.version>2.2</commons-io.version>
<junit.version>4.8.1</junit.version>
<mockito.version>1.8.5</mockito.version>
</properties>
Re: svn commit: r1357500
Posted by sebb <se...@gmail.com>.
On 8 July 2012 21:32, Oleg Kalnichevski <ol...@apache.org> wrote:
> On Fri, 2012-07-06 at 18:40 +0100, sebb wrote:
>> On 5 July 2012 09:56, <ol...@apache.org> wrote:
>> > Author: olegk
>> > Date: Thu Jul 5 08:56:58 2012
>> > New Revision: 1357500
>> >
>> > URL: http://svn.apache.org/viewvc?rev=1357500&view=rev
>> > Log:
>> > Upgraded HttpCore to version 4.2.1; upgraded HttpClient to version 4.2.1; downgraded Commons IO to version 2.2
>>
>> Presumably IO 2.2 was chosen because it only requires Java 1.5?
>
> Yes, it was. Given Commons IO is used in unit tests only I do not think
> it makes a big difference.
Agreed.
> Oleg
>
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscribe@hc.apache.org
> For additional commands, e-mail: dev-help@hc.apache.org
>
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@hc.apache.org
For additional commands, e-mail: dev-help@hc.apache.org
Re: svn commit: r1357500
Posted by Oleg Kalnichevski <ol...@apache.org>.
On Fri, 2012-07-06 at 18:40 +0100, sebb wrote:
> On 5 July 2012 09:56, <ol...@apache.org> wrote:
> > Author: olegk
> > Date: Thu Jul 5 08:56:58 2012
> > New Revision: 1357500
> >
> > URL: http://svn.apache.org/viewvc?rev=1357500&view=rev
> > Log:
> > Upgraded HttpCore to version 4.2.1; upgraded HttpClient to version 4.2.1; downgraded Commons IO to version 2.2
>
> Presumably IO 2.2 was chosen because it only requires Java 1.5?
Yes, it was. Given Commons IO is used in unit tests only I do not think
it makes a big difference.
Oleg
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@hc.apache.org
For additional commands, e-mail: dev-help@hc.apache.org
Re: svn commit: r1357500 - in /httpcomponents/httpasyncclient/trunk:
httpasyncclient/src/test/java/org/apache/http/nio/client/methods/TestZeroCopy.java
pom.xml
Posted by sebb <se...@gmail.com>.
On 5 July 2012 09:56, <ol...@apache.org> wrote:
> Author: olegk
> Date: Thu Jul 5 08:56:58 2012
> New Revision: 1357500
>
> URL: http://svn.apache.org/viewvc?rev=1357500&view=rev
> Log:
> Upgraded HttpCore to version 4.2.1; upgraded HttpClient to version 4.2.1; downgraded Commons IO to version 2.2
Presumably IO 2.2 was chosen because it only requires Java 1.5?
> Modified:
> httpcomponents/httpasyncclient/trunk/httpasyncclient/src/test/java/org/apache/http/nio/client/methods/TestZeroCopy.java
> httpcomponents/httpasyncclient/trunk/pom.xml
>
> Modified: httpcomponents/httpasyncclient/trunk/httpasyncclient/src/test/java/org/apache/http/nio/client/methods/TestZeroCopy.java
> URL: http://svn.apache.org/viewvc/httpcomponents/httpasyncclient/trunk/httpasyncclient/src/test/java/org/apache/http/nio/client/methods/TestZeroCopy.java?rev=1357500&r1=1357499&r2=1357500&view=diff
> ==============================================================================
> --- httpcomponents/httpasyncclient/trunk/httpasyncclient/src/test/java/org/apache/http/nio/client/methods/TestZeroCopy.java (original)
> +++ httpcomponents/httpasyncclient/trunk/httpasyncclient/src/test/java/org/apache/http/nio/client/methods/TestZeroCopy.java Thu Jul 5 08:56:58 2012
> @@ -39,6 +39,7 @@ import org.apache.commons.io.FileUtils;
> import org.apache.commons.io.IOUtils;
> import org.apache.commons.io.LineIterator;
> import org.apache.commons.io.output.FileWriterWithEncoding;
> +import org.apache.http.Consts;
> import org.apache.http.HttpAsyncTestBase;
> import org.apache.http.HttpEntity;
> import org.apache.http.HttpEntityEnclosingRequest;
> @@ -231,7 +232,11 @@ public class TestZeroCopy extends HttpAs
> InputStream instream = requestEntity.getContent();
> try {
> ContentType contentType = ContentType.getOrDefault(requestEntity);
> - LineIterator it = IOUtils.lineIterator(instream, contentType.getCharset());
> + Charset charset = contentType.getCharset();
> + if (charset == null) {
> + charset = Consts.ISO_8859_1;
> + }
> + LineIterator it = IOUtils.lineIterator(instream, charset.name());
> int count = 0;
> while (it.hasNext()) {
> String line = it.next();
>
> Modified: httpcomponents/httpasyncclient/trunk/pom.xml
> URL: http://svn.apache.org/viewvc/httpcomponents/httpasyncclient/trunk/pom.xml?rev=1357500&r1=1357499&r2=1357500&view=diff
> ==============================================================================
> --- httpcomponents/httpasyncclient/trunk/pom.xml (original)
> +++ httpcomponents/httpasyncclient/trunk/pom.xml Thu Jul 5 08:56:58 2012
> @@ -28,7 +28,7 @@
> <parent>
> <artifactId>project</artifactId>
> <groupId>org.apache.httpcomponents</groupId>
> - <version>5</version>
> + <version>6</version>
> <relativePath>../project/pom.xml</relativePath>
> </parent>
> <modelVersion>4.0.0</modelVersion>
> @@ -71,10 +71,10 @@
> <maven.compile.target>1.5</maven.compile.target>
> <maven.compile.optimize>true</maven.compile.optimize>
> <maven.compile.deprecation>true</maven.compile.deprecation>
> - <httpcore.version>4.2</httpcore.version>
> - <httpclient.version>4.2-beta1</httpclient.version>
> + <httpcore.version>4.2.1</httpcore.version>
> + <httpclient.version>4.2.1</httpclient.version>
> <commons-logging.version>1.1.1</commons-logging.version>
> - <commons-io.version>2.3</commons-io.version>
> + <commons-io.version>2.2</commons-io.version>
> <junit.version>4.8.1</junit.version>
> <mockito.version>1.8.5</mockito.version>
> </properties>
>
>
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@hc.apache.org
For additional commands, e-mail: dev-help@hc.apache.org