Skip to content

Commit 8a15b62

Browse files
committed
nit: compute noRequestFilters once
1 parent 32f313a commit 8a15b62

File tree

1 file changed

+3
-4
lines changed

1 file changed

+3
-4
lines changed

client/src/main/java/org/asynchttpclient/DefaultAsyncHttpClient.java

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -42,6 +42,7 @@ public class DefaultAsyncHttpClient implements AsyncHttpClient {
4242

4343
private final static Logger LOGGER = LoggerFactory.getLogger(DefaultAsyncHttpClient.class);
4444
private final AsyncHttpClientConfig config;
45+
private final boolean noRequestFilters;
4546
private final AtomicBoolean closed = new AtomicBoolean(false);
4647
private final ChannelManager channelManager;
4748
private final NettyRequestSender requestSender;
@@ -78,7 +79,7 @@ public DefaultAsyncHttpClient() {
7879
public DefaultAsyncHttpClient(AsyncHttpClientConfig config) {
7980

8081
this.config = config;
81-
82+
this.noRequestFilters = config.getRequestFilters().isEmpty();
8283
allowStopNettyTimer = config.getNettyTimer() == null;
8384
nettyTimer = allowStopNettyTimer ? newNettyTimer() : config.getNettyTimer();
8485

@@ -179,10 +180,8 @@ public BoundRequestBuilder prepareRequest(RequestBuilder requestBuilder) {
179180

180181
@Override
181182
public <T> ListenableFuture<T> executeRequest(Request request, AsyncHandler<T> handler) {
182-
183-
if (config.getRequestFilters().isEmpty()) {
183+
if (noRequestFilters) {
184184
return execute(request, handler);
185-
186185
} else {
187186
FilterContext<T> fc = new FilterContext.FilterContextBuilder<T>().asyncHandler(handler).request(request).build();
188187
try {

0 commit comments

Comments
 (0)