On Sun, Feb 10, 2013 at 7:10 PM, Claude Warren <cla...@xenei.com> wrote: > On Sun, Feb 10, 2013 at 6:17 PM, Andy Seaborne <a...@apache.org> wrote: >> On 10/02/13 16:59, Claude Warren wrote: >>> >>> I recently went through the iterator classes and think that there are >>> some that should be removed because the base java classes now support >>> the functionality (e.g. ArrayIterator) or are duplicated elsewhere >>> (e.g. ConcatenatedIterator) Perhaps these should be deprecated? >> >> >> Hi Claude, >> >> Always good to clean up! >> >> Would this be a reason to hold up starting the testing of jena 2.10.0 or >> would just drop into the general development and catch the next release? >> >> What would be the replacements for these? >> >> Andy >> >> >> > > I propose that we mark the following as deprecated with the noted substitution > > ArrayIterator : Arrays.asList( x[] ).iterator(); > > ConcatenatedIterator : NiceIterator.andThen() > > IteratorIterator : add a method to WrappedIterator to create an > ExtendedIterator that does the same thing. > > UniqueExtendedIterator : I am not sure about this one, but I think > that it should be replaced with a UniqueFilter that can be applied to > any ExtendedIterator. > > -- Claude > > > > >
I hit send a but too quickly there. I think we should do the deprecation before testing and then clean up our internal code before final release. I'd be happy to do the work, just need a Jira to track the work. -- Claude -- I like: Like Like - The likeliest place on the web Identity: https://www.identify.nu/user.php?cla...@xenei.com LinkedIn: http://www.linkedin.com/in/claudewarren
