A RetroSearch Logo

Home - News ( United States | United Kingdom | Italy | Germany ) - Football scores

Search Query:

Showing content from https://github.com/morelinq/MoreLINQ/issues/462 below:

Remove NestedLoops from public API · Issue #462 · morelinq/MoreLINQ · GitHub

As the comment in NestedLoops.cs says…

// This extension method was developed (primarily) to support the // implementation of the Permutations() extension methods. However, // it is of sufficient generality and usefulness to be elevated to // a public extension method in its own right.

…it was really designed for internal use by Permutations and I don't think it went through a proper design review at the time it was submitted so it should have never been public in the first place!

The reason I think it should be removed is that it's not an extension method for a sequence and the Action it extends doesn't have any arguments! Therefore Action must use side-effects to do any useful work and the resulting sequence of results won't be deterministic or will always require memoization.


RetroSearch is an open source project built by @garambo | Open a GitHub Issue

Search and Browse the WWW like it's 1997 | Search results from DuckDuckGo

HTML: 3.2 | Encoding: UTF-8 | Version: 0.7.4