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

Remove class map from FieldMethodizer

    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

        1. VELOCITY-593.patch
          1 kB
          Benjamin Bentmann

          Activity

            People

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

              Dates

              • Created:
                Updated:
                Resolved: