Derby
  1. Derby
  2. DERBY-5737

Remove GenericDescriptorList.elements and replace Enumerator usage with Iterator

    Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Trivial Trivial
    • Resolution: Fixed
    • Affects Version/s: 10.9.1.0
    • Fix Version/s: 10.9.1.0
    • Component/s: SQL
    • Labels:
      None

      Description

      I plan to refactor some of the code for dealing with lists of descriptors. As a first simple step I want to replace the use of Enumeration with Iterator. Later I'm planning to use composition instead of inheritance for the descriptor lists.

        Activity

        Kristian Waagan created issue -
        Hide
        Kristian Waagan added a comment -

        Attaching patch 1a, running regression tests.

        Show
        Kristian Waagan added a comment - Attaching patch 1a, running regression tests.
        Kristian Waagan made changes -
        Field Original Value New Value
        Attachment derby-5737-1a-iterator.diff [ 12525312 ]
        Hide
        Kristian Waagan added a comment -

        Regression tests passed.

        Show
        Kristian Waagan added a comment - Regression tests passed.
        Hide
        Knut Anders Hatlen added a comment -

        Patch 1a looks like a good improvement. +1

        Show
        Knut Anders Hatlen added a comment - Patch 1a looks like a good improvement. +1
        Hide
        Kristian Waagan added a comment -

        Thanks, Knut.
        Committed to trunk with revision 1333356.

        Show
        Kristian Waagan added a comment - Thanks, Knut. Committed to trunk with revision 1333356.
        Kristian Waagan made changes -
        Status Open [ 1 ] Closed [ 6 ]
        Issue & fix info Patch Available [ 10102 ]
        Fix Version/s 10.9.0.0 [ 12316344 ]
        Resolution Fixed [ 1 ]
        Gavin made changes -
        Workflow jira [ 12665159 ] Default workflow, editable Closed status [ 12796991 ]

          People

          • Assignee:
            Kristian Waagan
            Reporter:
            Kristian Waagan
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development