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 21:18:04 GMT


ASF GitHub Bot commented on LANG-1317:

Github user PascalSchumacher commented on a diff in the pull request:
    --- Diff: src/main/java/org/apache/commons/lang3/ ---
    @@ -457,6 +457,44 @@ public static String getAbbreviatedName(final String className, final
int len) {
    +     * <p>Gets a combination of {@link #getAllSuperclasses}(Class)} and
    +     * {@link #getAllInterfaces}(Class)}, one from superclasses, one
    +     * from interfaces, and so on in a breadth first way.</p>
    +     *
    +     * @param cls  the class to look up, may be {@code null}
    +     * @return the {@code List} of superclasses in order going up from this one
    --- End diff --
    This should also mention Interfaces.
    Can you add a since tag to this method?

> 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