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 Thu, 20 Apr 2017 14:11:04 GMT


ASF GitHub Bot commented on LANG-1317:

Github user PascalSchumacher commented on the issue:
    @yasserzamani Sorry for the delay, other open source projects and vacations interfered.
    I'm not sure if the priority parameter of `getAllSuperclassesAndInterfaces` is necessary?
Are you aware of any use-cases where it is necessary? Maybe we should keep things simple and
remove it?
    Otherwise the pull request is fine. :=) :+1: 

> 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