You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@commons.apache.org by "ASF GitHub Bot (JIRA)" <ji...@apache.org> on 2015/05/16 01:26:00 UTC
[jira] [Commented] (LANG-1115) Add support for varargs in
ConstructorUtils, MemberUtils, and MethodUtils
[ https://issues.apache.org/jira/browse/LANG-1115?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14546368#comment-14546368 ]
ASF GitHub Bot commented on LANG-1115:
--------------------------------------
GitHub user jimlloyd opened a pull request:
https://github.com/apache/commons-lang/pull/89
LANG-1115 Add support for varargs in method utils
@britter I have been collaborating with Joe Ferner on this contribution and have prepared this PR which I believe addresses the feedback you provided in [LANG-1115](https://issues.apache.org/jira/browse/LANG-1115). I'm not entirely sure I am following all of you contribution guidelines so please let me know if I need to resubmit. I will also add a comment to the Jira issue.
FYI, the first three commits in this PR replicate the original patch submission provided by Joe. The following seven commits address your feedback.
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/RedSeal-co/commons-lang LANG-1115-Add-support-for-varargs-in-method-utils
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/commons-lang/pull/89.patch
To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:
This closes #89
----
commit f68298488e0c0586b31f5e3076e4dbd60bc97835
Author: Jim Lloyd <ji...@gmail.com>
Date: 2015-05-14T15:55:03Z
Changes to tests
commit a9a60224db84941ae0e9fa7e9545022af53267ad
Author: Jim Lloyd <ji...@gmail.com>
Date: 2015-05-14T15:56:44Z
Revise MemberUtils.getTotalTransformationCost() for varargs.
commit feef24c107ef47f3ff6f17f2b7349b999ac48e9c
Author: Jim Lloyd <ji...@gmail.com>
Date: 2015-05-14T15:57:31Z
ConstructorUtils.java & MethodUtils.java.
commit 05c21917b5b1d2b061b35ac954f47a3651daa529
Author: Jim Lloyd <ji...@gmail.com>
Date: 2015-05-14T19:11:58Z
Use explicit code instead of comments to document getTotalTransformationCost for varargs cases.
commit 2e9165c9e18a4410b4174dd7bf77fa56dec73f8f
Author: Jim Lloyd <ji...@gmail.com>
Date: 2015-05-14T20:23:55Z
Add doc comments for getVarArgs().
commit d49c3e9ac376ddf9bfb0dad53181a44b5d7077e0
Author: Jim Lloyd <ji...@gmail.com>
Date: 2015-05-14T22:51:07Z
Verify correct varargs arguments are received in invoked constructors.
commit 27a547094fa767c209985cdbe18b23e316c96ebe
Author: Jim Lloyd <ji...@gmail.com>
Date: 2015-05-15T01:15:09Z
Verify correct varargs arguments are received in invoked methods.
commit 8c9b90090f8ded74b528fa9ef127943d4e726e58
Author: Jim Lloyd <ji...@gmail.com>
Date: 2015-05-15T19:26:14Z
Implement class MemberUtils.Executable to simplify compareParameterTypes.
commit 04c34e1729084ecbf18f6fabccbbdec62751745d
Author: Jim Lloyd <ji...@gmail.com>
Date: 2015-05-15T19:43:20Z
Move and rename isMatchingMethod() to isMatchingExecutable().
commit 153808cba43184617078079587f34b47b56cf033
Author: Jim Lloyd <ji...@gmail.com>
Date: 2015-05-15T20:04:09Z
Refactor so that Executable is private to MemberUtils.
----
> Add support for varargs in ConstructorUtils, MemberUtils, and MethodUtils
> -------------------------------------------------------------------------
>
> Key: LANG-1115
> URL: https://issues.apache.org/jira/browse/LANG-1115
> Project: Commons Lang
> Issue Type: Improvement
> Components: lang.reflect.*
> Reporter: Joe Ferner
> Priority: Minor
> Fix For: Review Patch
>
> Attachments: LANG-1115.patch
>
>
> Currently when calling methods such as MethodUtils.getMatchingAccessibleMethod variable arguments are not supported. It would be nice if by calling getMatchingAccessibleMethod with arguments that the java compiler would normally match with a vararg method would be matched.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)