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

Andrei Alexandrescu <andrei@erdani.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Keywords|                            |bootcamp
                 CC|                            |andrei@erdani.com

--

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

Nick Treleaven <nick@geany.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |nick@geany.org

--- Comment #1 from Nick Treleaven <nick@geany.org> ---
Alternatively we could use std.range.enumerate with minPos. minPos could take a unary map function like minElement does:

auto index = range.enumerate.minPos!`a.value`.front.value;

--

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

--- Comment #2 from Nick Treleaven <nick@geany.org> ---
Oops, should be .front.index at the end.

--

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

--- Comment #3 from github-bugzilla@puremagic.com ---
Commits pushed to master at https://github.com/dlang/phobos

https://github.com/dlang/phobos/commit/d2c7d3761b73405ee39da3fd7fe5030dee35a39e Issue 8573 - A simpler Phobos function that returns the index of the mix or max item

Issue 8573 - A simpler Phobos function that returns the index of the mix or max item

added some review fixes

fixed an issue with a mutable variable

Applied review feedback

Renamed functions to minIndex and maxIndex + used sizediff_t for return value type

Updated function so that it works optimally even for lazy ranges and algorithms

Reverted to having only copyable elements in ranges

Added more unittests; implemented an array path; fixed documentation

Squashed commits

https://github.com/dlang/phobos/commit/f3a840144a156fce5efee82ca13152c2ac1ef1c3 Merge pull request #4921 from RazvanN7/Issue_8573

Issue 8573 - A simpler Phobos function that returns the index of the …

--

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

--- Comment #4 from github-bugzilla@puremagic.com ---
Commits pushed to stable at https://github.com/dlang/phobos

https://github.com/dlang/phobos/commit/d2c7d3761b73405ee39da3fd7fe5030dee35a39e Issue 8573 - A simpler Phobos function that returns the index of the mix or max item

https://github.com/dlang/phobos/commit/f3a840144a156fce5efee82ca13152c2ac1ef1c3 Merge pull request #4921 from RazvanN7/Issue_8573

--

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

--- Comment #5 from github-bugzilla@puremagic.com ---
Commits pushed to newCTFE at https://github.com/dlang/phobos

https://github.com/dlang/phobos/commit/d2c7d3761b73405ee39da3fd7fe5030dee35a39e Issue 8573 - A simpler Phobos function that returns the index of the mix or max item

https://github.com/dlang/phobos/commit/f3a840144a156fce5efee82ca13152c2ac1ef1c3 Merge pull request #4921 from RazvanN7/Issue_8573

--

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

Jakub Łabaj <uaaabbjjkl@gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
                 CC|                            |uaaabbjjkl@gmail.com
         Resolution|---                         |FIXED

--- Comment #6 from Jakub Łabaj <uaaabbjjkl@gmail.com> ---
Implemented as stated in comments.

--