Remove NestedLoops from public API

As the comment in NestedLoops.cs says…

Lines 26 to 29 in 85c525c

    
           // 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.

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Metadata

Assignees

Labels

Type

Projects

Milestone

Relationships

Development

	// 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.

Description

Metadata

Metadata

Assignees

Labels

Type

Projects

Milestone

Relationships

Development

Issue actions