https://issues.dlang.org/show_bug.cgi?id=13596

hsteoh@quickfur.ath.cx changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |hsteoh@quickfur.ath.cx

--

https://issues.dlang.org/show_bug.cgi?id=13596

Vladimir Panteleev <thecybershadow@gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Priority|P1                          |P3
           Hardware|x86                         |All
                 OS|Windows                     |All

--

https://issues.dlang.org/show_bug.cgi?id=13596

Vladimir Panteleev <thecybershadow@gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Keywords|                            |pull
                 CC|                            |thecybershadow@gmail.com

--- Comment #1 from Vladimir Panteleev <thecybershadow@gmail.com> ---
Wrote something like this before finding this issue:

https://github.com/D-Programming-Language/phobos/pull/3482

--

https://issues.dlang.org/show_bug.cgi?id=13596

--- Comment #2 from github-bugzilla@puremagic.com ---
Commit pushed to master at https://github.com/D-Programming-Language/phobos

https://github.com/D-Programming-Language/phobos/commit/6079e23d827ffeaf15b6587eb713fa95a20e4e6f std.algorithm.iteration: Add permutations (fix issue 13596)

--

https://issues.dlang.org/show_bug.cgi?id=13596

--- Comment #3 from bearophile_hugs@eml.cc ---
(In reply to github-bugzilla from comment #2)
> Commit pushed to master at https://github.com/D-Programming-Language/phobos
> 
> https://github.com/D-Programming-Language/phobos/commit/ 6079e23d827ffeaf15b6587eb713fa95a20e4e6f std.algorithm.iteration: Add permutations (fix issue 13596)

You have missed the "doCopy" compilation argument, that gives safety.

--

https://issues.dlang.org/show_bug.cgi?id=13596

--- Comment #4 from bearophile_hugs@eml.cc ---
(In reply to bearophile_hugs from comment #3)
> (In reply to github-bugzilla from comment #2)
> > Commit pushed to master at https://github.com/D-Programming-Language/phobos
> > 
> > https://github.com/D-Programming-Language/phobos/commit/ 6079e23d827ffeaf15b6587eb713fa95a20e4e6f std.algorithm.iteration: Add permutations (fix issue 13596)
> 
> You have missed the "doCopy" compilation argument, that gives safety.

You can also add an optional runtime argument, to supply the array of indexes, to make everything @nogc if desired.

--

https://issues.dlang.org/show_bug.cgi?id=13596

--- Comment #5 from github-bugzilla@puremagic.com ---
Commit pushed to stable at https://github.com/D-Programming-Language/phobos

https://github.com/D-Programming-Language/phobos/commit/6079e23d827ffeaf15b6587eb713fa95a20e4e6f std.algorithm.iteration: Add permutations (fix issue 13596)

--