Closed GoogleCodeExporter closed 8 years ago
Fabulous - this was certainly something I was going to do, so it's wonderful to
have
it done for me :) Will check out the code tonight and almost certainly just put
it
in.
Original comment by jonathan.skeet
on 13 Feb 2009 at 6:27
Looks okay at a quick look, but I have two suggestions:
- I think we want four possible strategies for the case where the two sequences
aren't the same length: IgnoreLonger (current implementation); FillWithDefault
(e.g.
{1,2} and {"a", "b", "c"} would zip to {{1,"a"}, {2,"b"}, {0,"c"}} and Throw
(throw
InvalidOperationException).
- The current implementation doesn't dispose of the iterators after it's done.
I'll look at adding both of these aspects tonight.
Original comment by jonathan.skeet
on 15 Feb 2009 at 4:56
Okay, making a few changes now - should commit tonight.
Original comment by jonathan.skeet
on 15 Feb 2009 at 5:54
Original comment by azizatif
on 15 Feb 2009 at 6:51
Completed in r11.
Original comment by jonathan.skeet
on 15 Feb 2009 at 8:25
Original issue reported on code.google.com by
azizatif
on 13 Feb 2009 at 12:01Attachments: