[jboss-remoting-issues] [JBoss JIRA] Resolved: (JBREM-1258) EndpointImpl$ConnectionProviderContextImpl.accept() creates a redundant ConnectionHandler

[Ron Sigal (JIRA)]

     [ https://jira.jboss.org/browse/JBREM-1258?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Ron Sigal resolved JBREM-1258.
------------------------------

    Resolution: Done


Changed accept() to

        public void accept(final ConnectionHandlerFactory connectionHandlerFactory) {
            new ConnectionImpl(EndpointImpl.this, connectionHandlerFactory, this, "client");
        }


> EndpointImpl$ConnectionProviderContextImpl.accept() creates a redundant ConnectionHandler
> -----------------------------------------------------------------------------------------
>
>                 Key: JBREM-1258
>                 URL: https://jira.jboss.org/browse/JBREM-1258
>             Project: JBoss Remoting
>          Issue Type: Bug
>      Security Level: Public(Everyone can see) 
>    Affects Versions: 3.1.0.Beta2
>            Reporter: Ron Sigal
>            Assignee: Ron Sigal
>             Fix For: 3.1.0.Beta3
>
>
> org.jboss.remoting3.EndpointImpl$ConnectionProviderContextImpl.accept():
>        public void accept(final ConnectionHandlerFactory connectionHandlerFactory) {
>             connectionHandlerFactory.createInstance(new LocalConnectionContext(connectionProviderContext, new ConnectionImpl(EndpointImpl.this, connectionHandlerFactory, this, "client")));
>         }
> creates a ConnectionHandler, but the call to new ConnectionImpl() also creates a ConnectionHandler.  

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: https://jira.jboss.org/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira