Added practical example to std.algorithm.iteration.group

[JackStouffer]

Just discovered this useful idiom. I was doing this manually using foreach over ranges and incrementing the count in an AA for my stats plotting code.

All manual foreach loops in user code are bugs!

[wilzbach]

Nice!
Definitely worth to be shown to the user as an example :)

[quickfur]

Awesome example!

[MetaLang]

This should be a regular lambda.

[wilzbach]

It's also the default, so it could be omitted here as well..

[JackStouffer]

Unfortunately it can't be omitted, because then the compiler assumes you're using array.sort rather than std.algorithm.sort().

And for some reason, unlike Phobos, DMD does not stick to actually deprecating things when it says it will. array.sort has been a warning for almost three years now.

[wilzbach]

Unfortunately it can't be omitted, because then the compiler assumes you're using array.sort rather than std.algorithm.sort().

Just use empty parentheses (array.sort()) ;-)

array.sort has been a warning for almost three years now.

Well, I guess nobody has bothered to make a PR so far :/

[quickfur]

Hmm. According to the git log, this warning was turned into a deprecation last year (Apr 29, see dmd commit bc1f04ba. It has been a warning since before the C++ to D switchover in Aug 2015, though, so it has been a while. I think I'll submit a PR for this.

[quickfur]

The deed is done: dlang/dmd#6554