Qpid
  1. Qpid
  2. QPID-4905

C++ Broker/common code needs tidying up of obsolete/unused code

    Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 0.23
    • Component/s: C++ Broker, C++ Client
    • Labels:
      None

      Description

      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.

        Activity

        Andrew Stitcher created issue -
        Andrew Stitcher made changes -
        Field Original Value New Value
        Assignee Andrew Stitcher [ astitcher ]
        Andrew Stitcher made changes -
        Description 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 shuold 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.
        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.
        Andrew Stitcher made changes -
        Description 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.
        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.
        Andrew Stitcher made changes -
        Status Open [ 1 ] Resolved [ 5 ]
        Fix Version/s 0.23 [ 12324273 ]
        Resolution Fixed [ 1 ]
        Justin Ross made changes -
        Status Resolved [ 5 ] Closed [ 6 ]

          People

          • Assignee:
            Andrew Stitcher
            Reporter:
            Andrew Stitcher
          • Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development