[Issue 15860] lockstep should support foreach_reverse

[via Digitalmars-d-bugs]

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

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

https://github.com/dlang/phobos/commit/154b0e9cb8f9b0d93a59faf913006a4b66862bc9
Fix Issue 15860 - lockstep should support foreach_reverse

https://github.com/dlang/phobos/commit/23fcb244efbb545485fb5fd8635144340e4193d3
Merge pull request #4138 from Biotronic/fix-15860

Fix Issue 15860 - lockstep should support foreach_reverse

--

[Issue 15860] lockstep should support foreach_reverse

[via Digitalmars-d-bugs]

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

github-bugzi...@puremagic.com changed:

   What|Removed |Added

 Status|NEW |RESOLVED
 Resolution|--- |FIXED

--

[Issue 15860] lockstep should support foreach_reverse

[via Digitalmars-d-bugs]

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

--- Comment #2 from Simen Kjaeraas  ---
The pull request above adds foreach_reverse without index to any set of
bidirectional ranges (but if one or more is not bidirectional, foreach_reverse
is of course not possible).

Indexed reverse iteration has more requirements:

All iterated ranges must have have a length, and the stoppingPolicy must be
requireSameLength. Sadly, the latter is a runtime value, so it has to be
handled with enforce. I'd have prefered to disallow it at compile-time.

--

[Issue 15860] lockstep should support foreach_reverse

[via Digitalmars-d-bugs]

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

Simen Kjaeraas  changed:

   What|Removed |Added

   Keywords||pull
 CC||simen.kja...@gmail.com

--- Comment #1 from Simen Kjaeraas  ---
Fixed in PR 4138:
https://github.com/D-Programming-Language/phobos/pull/4138

--

[Issue 15860] lockstep should support foreach_reverse

[via Digitalmars-d-bugs]

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

ZombineDev  changed:

   What|Removed |Added

   Assignee|nob...@puremagic.com|petar.p.ki...@gmail.com

--