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

    • Type: Improvement
    • Status: Resolved
    • Priority: Trivial
    • Resolution: Fixed
    • Affects Version/s: 1.5
    • Fix Version/s: 1.6
    • Component/s: Engine
    • Labels:
      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

            People

            • Assignee:
              Unassigned
              Reporter:
              bentmann Benjamin Bentmann

              Dates

              • Created:
                Updated:
                Resolved:

                Issue deployment