[
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.