Review: MoreEnumerable.Incremental()

[atifaziz]

Originally reported on Google Code with ID 46

Purpose of code changes on this branch:

Rev 137: Implementation of the Incremental operator which visits each
adjacent pair of elements in a sequence: {N,N+1}, {N+1,N+2}, {N+2,N+3} ...
and applies a projection to them. An overload is available that is also
supplied the zero-based index of the *second* element in the pair. The
number of items in the result sequence will always be one less than in the
source. Incremental returns an empty sequence if passed a sequence of less
than two items. Incremental operates in a deferred, streaming fashion. 

When reviewing my code changes, please focus on:

* The public interface of the extension method.
* The effectiveness and clarity of the available XML comment documentation.
* How well this operator fits into the MoreLINQ ecosystem of
extension methods.
* The extent of coverage available from the corresponding unit tests.
* Whether edge cases are correctly identified and handled in the
implementation and tests.

After the review, I'll merge this branch into:
/trunk

Reported by ambientlion on 2010-01-23 17:16:35

[atifaziz]

Now migrated to Hg and available in clone:
http://code.google.com/r/azizatif-morelinq-evenmore/
This clone is ready to be cloned :O) for further review. It also has the main/default
branch already merged in to bring it up to date.

Reported by azizatif on 2012-05-25 23:38:38

[atifaziz]

Reported by azizatif on 2012-06-03 21:08:20

• Status changed: Started

[atifaziz]

Seems redundant with Pairwise.

Reported by azizatif on 2012-06-03 21:11:25

[atifaziz]

Following no response from ambientlion (who possibly/assuming lost interest given it
has taken years to get this far), taking ownership of issue to consider as addition
for MoreLINQ 2.0 milestone.

Reported by azizatif on 2013-06-12 22:10:51

• Labels added: Type-Enhancement, Priority-High, Milestone-Release2.0, Component-Logic
• Labels removed: Type-Review, Priority-Medium

[atifaziz]

This issue was closed by revision ecbb044b3f86.

Reported by azizatif on 2013-06-14 05:18:12

• Status changed: Fixed

[atifaziz]

Duplicate with Pairwise that was already published with release 1.0

Reported by azizatif on 2013-06-14 05:55:08

• Status changed: WontFix