Implement Concat in terms of existing LINQ operations

MeiChun-Lo / morelinq

Automatically exported from code.google.com/p/morelinq

Apache License 2.0

0 stars 0 forks source link

Implement Concat in terms of existing LINQ operations #1

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago

This is only a suggestion to simplify the implementation of 
Concatenation.Concat operator in MoreLINQ. The attached patch proposes to 
implement Concat in terms of existing LINQ operators, yielding shorter 
code and reusing a tested base. Hopefully, you agree.

Original issue reported on code.google.com by azizatif on 12 Feb 2009 at 10:42

Attachments:

Concatenation.patch

GoogleCodeExporter commented 9 years ago

Yes, I like it. Still no extra objects created or anything like that (when I 
first 
read the description I feared the creation of a single element array) - looks 
good. 
Will patch tonight.

Original comment by jonathan.skeet on 13 Feb 2009 at 6:28

GoogleCodeExporter commented 9 years ago

Original comment by jonathan.skeet on 15 Feb 2009 at 5:15

Changed state: Done

GoogleCodeExporter commented 9 years ago

Implemented in r3.

Original comment by azizatif on 15 Feb 2009 at 6:54