commons-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "ASF GitHub Bot (JIRA)" <>
Subject [jira] [Commented] (LANG-1317) Add findAnnotation and findMethodsWithAnnotation to MethodUtils
Date Sun, 26 Mar 2017 18:54:41 GMT


ASF GitHub Bot commented on LANG-1317:

Github user PascalSchumacher commented on the issue:
    You are right. The method signature I suggested omits the important super part. I agree
it should be something like `getMethodsWithAnnotation(Class cls, annotationCls, boolean searchSupers,
boolean ignoreAccess)`. (I'm not a fan of boolean parameters (especially multiple ones), so
if somebody has an idea for a good method name, that is welcome.)

> Add findAnnotation and findMethodsWithAnnotation to MethodUtils
> ---------------------------------------------------------------
>                 Key: LANG-1317
>                 URL:
>             Project: Commons Lang
>          Issue Type: Improvement
>          Components: lang.reflect.*
>            Reporter: Yasser Zamani
>              Labels: patch
> In order to fix WW-4744 , mainly, I am going to add two functionalities to MethodUtils:
findAnnotation and findMethodsWithAnnotation.
> findAnnotation will be an extension for Method.getAnnotation that also searches interfaces
and super classes while caching results with no memory leak.
> findMethodsWithAnnotation will be an extension for getMethodsWithAnnotation that also
supports non public methods, super class and interface methods, again, while caching results
as above.
> Generally, do you agree with these in a pull request? If so, I will be working on it

This message was sent by Atlassian JIRA

View raw message