commons-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Henri Yandell (JIRA)" <>
Subject [jira] Commented: (LANG-588) Create a basic Pair<L, R> class
Date Sun, 07 Feb 2010 05:22:28 GMT


Henri Yandell commented on LANG-588:

It was for IP sake - I wrote it as an exercise instead of for any use. Happy to use yours

Comments on your patches:

* Javadoc for pairOf/matchedPairOf needs filling in.
* @since should be '3.0' and not 'Lang 3.0'
* The static import in the tests makes me cringe, but I know you like using them. First time
I looked at the code (as I looked at tests first) I went looking in the file to see where
the utility pairOf method was, while wondering why you'd done such a thing.
* I'm not sure about the Iterable concept for MatchedPair.
* I do like the notion of Iterable if it becomes a tree walker - i.e. it iterates down Pairs
of Pairs. It could then be in the parent and could be:  depthFirstIterate() & breadthFirstIterate().

> Create a basic Pair<L, R> class
> -------------------------------
>                 Key: LANG-588
>                 URL:
>             Project: Commons Lang
>          Issue Type: New Feature
>          Components: lang.*
>            Reporter: Matt Benson
>             Fix For: 3.0
>         Attachments:,,,
> [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.

View raw message