Bug 2928 – Swap order of paramaters for std.range's "take"

According to the docs, in std.range, "take" takes the range as its second parameter and 'n' as the first parameter. This ordering should be swapped, 1. to be consistent with stride, advance, retreatN, and cycle, and 2. to allow the syntax "a.take(n)" like with advance.

Undecided. take is inspired from Haskell et al where the range comes last.

Keep in mind that Haskell's parameter ordering is based on partial application, a feature that D obviously does not have.

(In reply to comment #2) > Keep in mind that Haskell's parameter ordering is based on partial application, > a feature that D obviously does not have. > Sort of does: import std.functional, std.range, std.stdio; void main() { auto a = [ 1, 2, 3 ][]; alias curry!(take, 2) take2; foreach (e; take2(a)) { writeln(e); } } But I agree there are good arguments in favor of swapping arguments.

Please consider this also for replicate* and any other functions (though replicate and take are the only ones I could find.) Note that in haskell's prelude, there are many functions that also can be found in phobos, such as splitAt and until, for which the same argument could be made. Haskell does seem to be more consistent in argument ordering, always the predicate first and then the list, this makes it easy to remember. It's a bit weird that when uniform function call syntax will be implemented, this will work: iota(0,10).stride(2).until(8) but this won't work: iota(0,10).stride(2).take(4) I'd rather give up currying myself to make this work, even for the benefit of consistency alone. * note that std.string.repeat is defined as string repeat(string s, size_t n);

What is the status on this? I would like to see it it changed to Take!(R) take(R)(R input, size_t n); Take!(Repeat!(T)) replicate(T)(T value, size_t n);

I'll change that soon. Thanks!

Fixed in 2.041 http://digitalmars.com/d/2.0/changelog.html#new2_041