soraismus / morelinq

Automatically exported from code.google.com/p/morelinq
Apache License 2.0
0 stars 0 forks source link

Code review request for Split #73

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
Purpose of code changes on this branch:

When I added the Batch operator, I thought I could implement it in terms 
of a generalized Split operator. I was wrong, but the damage was already 
done and I had implemented and factored out all the work to implement a 
Split operator for any enumerable source. However, I have doubts about its 
general usefulness and is the reason why I am putting it in a branch.

When reviewing my code changes, please focus on:

I would appreciate it if you could review and comment on "utility" and 
general application of this operator. If you find that it makes good sense 
then I can merge it back into the turnk. Otherwise the branch can just 
serve as an archive.

Post your review to the following clone:
http://code.google.com/r/azizatif-morelinq-split/source/list

After review, I'll merge the clone with main.

Original issue reported on code.google.com by azizatif on 25 May 2012 at 4:40

GoogleCodeExporter commented 9 years ago
Issue 8 has been merged into this issue.

Original comment by azizatif on 25 May 2012 at 4:41

GoogleCodeExporter commented 9 years ago
This issue was closed by revision 79a03c125475.

Original comment by azizatif on 3 Jun 2012 at 7:48