[ 
https://issues.apache.org/jira/browse/HTTPCORE-536?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16569994#comment-16569994
 ] 

ASF GitHub Bot commented on HTTPCORE-536:
-----------------------------------------

Github user clevertension commented on a diff in the pull request:

    https://github.com/apache/httpcomponents-core/pull/72#discussion_r207838048
  
    --- Diff: 
httpcore5/src/main/java/org/apache/hc/core5/reactor/DefaultConnectingIOReactor.java
 ---
    @@ -119,28 +115,13 @@ public IOReactorStatus getStatus() {
         }
     
         @Override
    -    public List<ExceptionEvent> getExceptionLog() {
    -        return auditLog.isEmpty() ? 
Collections.<ExceptionEvent>emptyList() : new ArrayList<>(auditLog);
    +    public IOWorkers.Selector getWorkerSelector() {
    --- End diff --
    
    change it to **protected** or **friendly**?


> 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]

Reply via email to