castor
  1. castor
  2. CASTOR-3104

Extract 'molder' from TypeInfo, LockEngine, OID, and move it into ClassMolderRegistry

    Details

    • Type: Improvement Improvement
    • Status: Resolved Resolved
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 1.3.3rc1
    • Component/s: JDO
    • Labels:
      None
    • Number of attachments :
      1

      Description

      TypeInfo, LockEngine, OID all have some functions with 'molder', these make the code difficult to understand. This issue will extract 'molder' from TypeInfo, LockEngine, OID, and move it into ClassMolderRegistry.

        Activity

        Hide
        Wensheng Dou added a comment -

        What have been done in patch-C3104-20110605.txt?
        (1) Extract 'molder' in the TypeInfo and LockEngine to a new class ClassMolderRegistry
        (2) Simplify the TypeInfo
        (3) Modify some comments in the TypeInfo and LockEngine

        Show
        Wensheng Dou added a comment - What have been done in patch-C3104-20110605.txt? (1) Extract 'molder' in the TypeInfo and LockEngine to a new class ClassMolderRegistry (2) Simplify the TypeInfo (3) Modify some comments in the TypeInfo and LockEngine
        Hide
        Wensheng Dou added a comment -

        What should be done in next patch?
        (1) Extract 'molder' in OID and simplify the class OID
        (2) Maybe, the TypeInfo should rename now.

        Show
        Wensheng Dou added a comment - What should be done in next patch? (1) Extract 'molder' in OID and simplify the class OID (2) Maybe, the TypeInfo should rename now.
        Hide
        Ralf Joachim added a comment -

        I'll keep this open for more refactoring subtasks.

        Show
        Ralf Joachim added a comment - I'll keep this open for more refactoring subtasks.

          People

          • Assignee:
            Wensheng Dou
            Reporter:
            Wensheng Dou
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: