search

devurmel / 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:

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

GoogleCodeExporter commented 9 years ago 
Implemented in r3.

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