You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@velocity.apache.org by "Claude Brisson (JIRA)" <ji...@apache.org> on 2016/07/15 15:37:20 UTC

[jira] [Resolved] (VELOCITY-745) Modify ClassMap.java to use getMethods() rather than get getDeclaredMethods()

     [ https://issues.apache.org/jira/browse/VELOCITY-745?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Claude Brisson resolved VELOCITY-745.
-------------------------------------
    Resolution: Won't Fix

Since current implementation *works*, and since as noted by Nathan it may me more complex than it looks like to use getDeclaredMethods(), let's stay like this.

Of course, as usual, feel free to reopen if you want to chime in on this subject...

> Modify ClassMap.java to use getMethods() rather than get getDeclaredMethods()
> -----------------------------------------------------------------------------
>
>                 Key: VELOCITY-745
>                 URL: https://issues.apache.org/jira/browse/VELOCITY-745
>             Project: Velocity
>          Issue Type: Improvement
>          Components: Engine
>    Affects Versions: 1.6.2
>         Environment: All
>            Reporter: Steve O'Hara
>            Priority: Minor
>
> The code that recurses up the super classes to find all public methods using getDeclaredMethods() is now redundant and can be replaced with a single call to getMethods() - this intrinsically provides all inherited public methods.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@velocity.apache.org
For additional commands, e-mail: dev-help@velocity.apache.org