[
https://issues.apache.org/jira/browse/HTTPCORE-536?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16569964#comment-16569964
]
ASF GitHub Bot commented on HTTPCORE-536:
-----------------------------------------
Github user ok2c commented on a diff in the pull request:
https://github.com/apache/httpcomponents-core/pull/72#discussion_r207833056
--- Diff:
httpcore5/src/main/java/org/apache/hc/core5/http/impl/bootstrap/AsyncRequester.java
---
@@ -27,12 +27,6 @@
package org.apache.hc.core5.http.impl.bootstrap;
-import java.io.IOException;
--- End diff --
@clevertension Can we avoid moving imports around? It is not a big problem
but it makes the change-set unnecessarily noisy.
> duplicate connect method could be extract
> -----------------------------------------
>
> Key: HTTPCORE-536
> URL: https://issues.apache.org/jira/browse/HTTPCORE-536
> Project: HttpComponents HttpCore
> Issue Type: Improvement
> Components: HttpCore NIO
> Reporter: silver9886
> Priority: Major
>
> DefaultConnectingIOReactor and DefaultListeningIOReactor connect method are
> same.
> create abstractDefaultIOReactor class,put the connect code to the abstract
> class.
> AsyncServer and AsyncRequester connect method are same.
> create abstractAsyncServerandRequester class,put the connect code to the
> abstract class.
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
