Skip to content

Commit 26ae219

Browse files
author
Stephane Landelle
committed
Use StringBuilders instead of StringBuffers, close AsyncHttpClient#305
1 parent 60c7343 commit 26ae219

File tree

5 files changed

+5
-5
lines changed

5 files changed

+5
-5
lines changed

api/src/main/java/org/asynchttpclient/multipart/MultipartRequestEntity.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -142,7 +142,7 @@ public String getContentType() {
142142
if (contentType.contains("boundary="))
143143
return contentType;
144144
else {
145-
StringBuffer buffer = new StringBuffer(contentType);
145+
StringBuilder buffer = new StringBuilder(contentType);
146146
if (!contentType.endsWith(";"))
147147
buffer.append(";");
148148
buffer.append(" boundary=");

api/src/main/java/org/asynchttpclient/resumable/PropertiesBasedResumableProcessor.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -90,7 +90,7 @@ public void save(Map<String, Long> map) {
9090
}
9191

9292
private static String append(Map.Entry<String, Long> e) {
93-
return new StringBuffer(e.getKey()).append("=").append(e.getValue()).append("\n").toString();
93+
return new StringBuilder(e.getKey()).append("=").append(e.getValue()).append("\n").toString();
9494
}
9595

9696
/**

api/src/main/java/org/asynchttpclient/util/AsyncHttpProviderUtils.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -505,7 +505,7 @@ private static void add(StringBuilder sb, String name, int val) {
505505

506506
public static String constructUserAgent(Class<? extends AsyncHttpProvider> httpProvider,
507507
AsyncHttpClientConfig config) {
508-
return new StringBuffer(config.getUserAgent())
508+
return new StringBuilder(config.getUserAgent())
509509
.append(' ')
510510
.append('(')
511511
.append(httpProvider.getSimpleName())

api/src/test/java/org/asynchttpclient/async/BasicAuthTest.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -473,7 +473,7 @@ public AbstractHandler configureHandler() throws Exception {
473473
}
474474

475475
@Test(groups = { "standalone", "default_provider" }, enabled = false)
476-
public void StringBufferBodyConsumerTest() throws Throwable {
476+
public void StringBuilderBodyConsumerTest() throws Throwable {
477477

478478
SimpleAsyncHttpClient client = new SimpleAsyncHttpClient.Builder().setRealmPrincipal(user).setRealmPassword(admin).setUrl(getTargetUrl()).setHeader("Content-Type", "text/html").build();
479479
try {

api/src/test/java/org/asynchttpclient/async/SimpleAsyncHttpClientTest.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -57,7 +57,7 @@ public void inpuStreamBodyConsumerTest() throws Throwable {
5757
}
5858

5959
@Test(groups = { "standalone", "default_provider" })
60-
public void StringBufferBodyConsumerTest() throws Throwable {
60+
public void StringBuilderBodyConsumerTest() throws Throwable {
6161

6262
SimpleAsyncHttpClient client = new SimpleAsyncHttpClient.Builder().setIdleConnectionInPoolTimeoutInMs(100).setMaximumConnectionsTotal(50).setRequestTimeoutInMs(5 * 60 * 1000).setUrl(getTargetUrl()).setHeader("Content-Type", "text/html").build();
6363
try {

0 commit comments

Comments
 (0)