You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@commons.apache.org by "Peter Verhas (Jira)" <ji...@apache.org> on 2019/11/07 03:50:03 UTC

[jira] [Commented] (LANG-1482) Functions

    [ https://issues.apache.org/jira/browse/LANG-1482?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16968903#comment-16968903 ] 

Peter Verhas commented on LANG-1482:
------------------------------------

This is a long outstanding pull request. In the meantime, there were some changes that made conflict with the master. I made the resolution. Could someone please review it?

> Functions 
> ----------
>
>                 Key: LANG-1482
>                 URL: https://issues.apache.org/jira/browse/LANG-1482
>             Project: Commons Lang
>          Issue Type: Bug
>          Components: lang.*
>            Reporter: Peter Verhas
>            Priority: Minor
>             Fix For: 3.10
>
>   Original Estimate: 1h
>  Remaining Estimate: 1h
>
> The implementation of the methods {{asRunnable()}}, {{asConsumer()}}, {{asCallable()}}, {{asBiConsumer()}} etc. is redundant and copy-paste. They are implemented with the structure (example from {{asRunnable()}}:
> {code}
> return () -> {
>             try {
>                 pRunnable.run();
>             } catch (Throwable t) {
>                 throw rethrow(t);
>             }
>         };
> {code}
> This try-catch structure is already implemented in the class and can be used here simplifying the method to 
> {code}
> return () -> run(pRunnable);
> {code}
> Also, the tests for {{asPredicate()}} and {{asBiPredicate()}} are missing.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)