CombinationsSequence: underestimatedCount == count

apple / swift-algorithms

Commonly used sequence and collection algorithms for Swift

Apache License 2.0

5.9k stars 435 forks source link

CombinationsSequence: underestimatedCount == count #220

Closed dabrahams closed 7 months ago

dabrahams commented 7 months ago

Not doing this makes it needlessly costly to create an Array of the results. It's a reasonable incremental improvement until you get around to #219, which would handle this automatically.

Checklist

[x] I've added at least one test that validates that my change is working, if appropriate
[x] I've followed the code style of the rest of the project
[x] I've read the Contribution Guidelines
[x] I've updated the documentation if necessary

natecook1000 commented 7 months ago

@swift-ci Please test

natecook1000 commented 7 months ago

@swift-ci Please test