Uploaded image for project: 'Velocity'
  1. Velocity
  2. VELOCITY-593

Remove class map from FieldMethodizer

Attach filesAttach ScreenshotVotersWatch issueWatchersCreate sub-taskLinkCloneUpdate Comment AuthorReplace String in CommentUpdate Comment VisibilityDelete Comments
    XMLWordPrintableJSON

Details

    • Improvement
    • Status: Resolved
    • Trivial
    • Resolution: Fixed
    • 1.5
    • 1.6
    • Engine
    • None

    Description

      According to the API docs of Field.get():

      If the underlying field is a static field, the obj argument is ignored; it may be null.

      So there is no need to waste memory on keeping a class map (besides, if one needs to know the class defining a field, there is Field.getDeclaringClass()).

      Attachments

        Activity

          This comment will be Viewable by All Users Viewable by All Users
          Cancel

          People

            Unassigned Unassigned
            bentmann Benjamin Bentmann
            Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:

              Slack

                Issue deployment