Bug 16170 – std.algorithm.sorting.partition has many issues

* The SwapStrategy.semistable branch does not call save for result, therefore this breaks with forward ranges * The SwapStrategy.unstable !isDynamicArray branch does not call save for result, therefore this breaks with forward ranges * The SwapStrategy.unstable !isDynamicArray branch requires a bidirectional range, only forward range primitives are tested for * is(typeof(swap(result.front, r.front))) is never tested for A lot of this stems from the fact that arrays are the only means of testing for this function

Commits pushed to master at https://github.com/dlang/phobos https://github.com/dlang/phobos/commit/e21f2729ba3f99605c7db5572c96ad83b1b7ae86 [Issue 16170] Seperate std.algorithm.sorting.partition into various overloads in order to facilitate further improvements https://github.com/dlang/phobos/commit/b2978040ca3c0f061f283e4f433507b635337efb Merge pull request #4429 from JackStouffer/issue16170 [Issue 16170] Partial Fix for Broken std.algorithm.sorting.partition

Commits pushed to stable at https://github.com/dlang/phobos https://github.com/dlang/phobos/commit/e21f2729ba3f99605c7db5572c96ad83b1b7ae86 [Issue 16170] Seperate std.algorithm.sorting.partition into various overloads in order to facilitate further improvements https://github.com/dlang/phobos/commit/b2978040ca3c0f061f283e4f433507b635337efb Merge pull request #4429 from JackStouffer/issue16170

THIS ISSUE HAS BEEN MOVED TO GITHUB https://github.com/dlang/phobos/issues/10182 DO NOT COMMENT HERE ANYMORE, NOBODY WILL SEE IT, THIS ISSUE HAS BEEN MOVED TO GITHUB