Uploaded image for project: 'JikesRVM'
  1. JikesRVM
  2. RVM-273

Remove VM_ prefix from all VM classes

    Details

    • Type: Improvement
    • Status: Closed
    • Priority: Low
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 3.0
    • Component/s: None
    • Labels:
      None

      Description

      Uniformly remove the VM_ prefix from all RVM classes.

      We'll manage primordial class inclusion primarily via package membership augmented with an inclusion/exclusion mechanism that could be done either by annotations or by a flie list.

      Needs to be fixed for 3.0 because file renaming can be disruptive for users.

        Gliffy Diagrams

          Attachments

            Activity

            dgrove David Grove created issue -
            Hide
            dgrove David Grove added a comment -

            When working on this; also rationalize the various include properties in build.xml.

            Get rid of include.all (make this the default behavior for the initial set of classes)

            This is then filtered to exclude opt/aos packages out if they are not included (base base configs).

            Show
            dgrove David Grove added a comment - When working on this; also rationalize the various include properties in build.xml. Get rid of include.all (make this the default behavior for the initial set of classes) This is then filtered to exclude opt/aos packages out if they are not included (base base configs).
            Hide
            dgrove David Grove added a comment -

            Have apparently working set of changes. Going to wait to commit it until the dust settles from the recent build breaks and attempted fixes so we know we have a reasonable baseline to detect new regression introduced by this change. Hope to commit tomorrow.

            Whoops: this comment was supposed to go to RVM-518. I have build system changes. It will be several days before I'm ready to commit the removal of VM_ "big change"

            Show
            dgrove David Grove added a comment - Have apparently working set of changes. Going to wait to commit it until the dust settles from the recent build breaks and attempted fixes so we know we have a reasonable baseline to detect new regression introduced by this change. Hope to commit tomorrow. Whoops: this comment was supposed to go to RVM-518 . I have build system changes. It will be several days before I'm ready to commit the removal of VM_ "big change"
            Hide
            dgrove David Grove added a comment -

            Renaming committed in r14507.

            Small cleanups of things that shouldn't have been renamed (mostly various forms of RVM_) followed in 14508-14513.

            Show
            dgrove David Grove added a comment - Renaming committed in r14507. Small cleanups of things that shouldn't have been renamed (mostly various forms of RVM_) followed in 14508-14513.
            Hide
            dgrove David Grove added a comment -

            reopening so I can modify fix target to 3.0

            Show
            dgrove David Grove added a comment - reopening so I can modify fix target to 3.0
            dgrove David Grove made changes -
            Field Original Value New Value
            Workflow jira [ 17399 ] X10 Workflow [ 18676 ]
            dgrove David Grove made changes -
            Priority Minor [ 7 ] Low [ 4 ]

              People

              • Assignee:
                dgrove David Grove
                Reporter:
                dgrove David Grove
              • Votes:
                0 Vote for this issue
                Watchers:
                0 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: