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

Stephen Colebourne commented on LANG-588:
-----------------------------------------

I should note that EnumSet and JSR-310 use Classname.of - its becoming a new 
standard. The benefit of static imports seems to wane by comparison. So, +1 to 
Pair.of()

As explained above this class shouldn't cache the hash code, as mutable objects 
might be stashed within.

I should say that I find the L/R notation perfectly acceptable for this class - 
it doesn't overly imply anything to me.

> Create a basic Pair<L, R> class
> -------------------------------
>
>                 Key: LANG-588
>                 URL: https://issues.apache.org/jira/browse/LANG-588
>             Project: Commons Lang
>          Issue Type: New Feature
>          Components: lang.*
>            Reporter: Matt Benson
>             Fix For: 3.0
>
>         Attachments: Pair.java, PairTest.java
>
>
> [lang] is the perfect place to provide a basic typed Pair class.  I have 
> written such a class for my employer (who hasn't?) but can/will rewrite blind 
> to avoid IP issues.  I think it's also nice to go ahead and extend this to 
> MatchedPair<T> extends Pair<T, T> as well.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply via email to