soraismus / morelinq

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

Consider renaming Zip methods #25

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
(No need for one issue per method.)

We should look at what .NET 4.0 is going to do in terms of Zip methods, and 
try to match those so it's a simple hand-over when moving from 3.5 + 
MoreLinq to 4. We should also make sure that the duplicate naming doesn't 
cause a problem - or have a separate build which doesn't include it.

When .NET 4.0 beta 1 comes out we can take a look.

Original issue reported on code.google.com by jonathan.skeet on 7 Apr 2009 at 9:53

GoogleCodeExporter commented 9 years ago

Original comment by azizatif on 7 Apr 2009 at 10:07

GoogleCodeExporter commented 9 years ago
This issue has been created from suggestions in other issues and which are re-
produced below for convenience:

--------------------
From: azizatif
Source: http://code.google.com/p/morelinq/issues/detail?id=6#c4

- EquiZip:
  Sequences must be equal in length otherwise throw error

- Zip: 
  Truncates to shorter of the two sequences

--------------------
From: jeffmagic
Source: http://code.google.com/p/morelinq/issues/detail?id=6#c11

  - Zip
  - PaddedZip

I don't think the truncated version is required because if you lazily evaluate 
the 
results of Zip and PaddedZip you can just do PaddedZip.Take or Zip.Take to get 
the 
shorter of the two.

--------------------
From: jonathan.skeet
Source: http://code.google.com/p/morelinq/issues/detail?id=24#c2

How about:

Zip - throw error
ZipTruncate - stop at end of shorter sequence
ZipExtend - extend shorter sequence with default values

Original comment by azizatif on 7 Apr 2009 at 10:29

GoogleCodeExporter commented 9 years ago
Closing this issue as stale as v1.0 shipped with Zip, EquiZip and ZipLongest.

Original comment by azizatif on 22 Jun 2013 at 11:54

GoogleCodeExporter commented 9 years ago

Original comment by azizatif on 22 Jun 2013 at 11:56