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

Andrew Stitcher commented on QPID-4905:
---------------------------------------

the federationLink bool member of broker::ConnectionState is only ever set and 
never queried or otherwise used, so can be removed.
                
> C++ Broker/common code needs tidying up of obsolete/unused code
> ---------------------------------------------------------------
>
>                 Key: QPID-4905
>                 URL: https://issues.apache.org/jira/browse/QPID-4905
>             Project: Qpid
>          Issue Type: Improvement
>          Components: C++ Broker, C++ Client
>            Reporter: Andrew Stitcher
>            Assignee: Andrew Stitcher
>
> There is quite a lot of code in the Qpid C++ code base that is vestigial 
> (like your appendix). In the name of easier code maintenance we should remove 
> it.
> I will detail some of the things I have found as comments to this JIRA, if 
> they attract sufficient comment they can be broken out as subtasks to this 
> issue.
> Most of these issues are small and involve unused member functions or 
> unused/potentially unused interfaces or interfaces implemented unnecessarily.
> Much of this is still the fallout of removing the old cluster work which 
> added hook (and hence interfaces) in lots of code, now that old cluster is 
> gone the interfaces/members remain, but are unused anywhere.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@qpid.apache.org
For additional commands, e-mail: dev-h...@qpid.apache.org

Reply via email to