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

elpenguino+D@gmail.com changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |elpenguino+D@gmail.com

--- Comment #1 from elpenguino+D@gmail.com ---
I fully agree. I've been bitten by this same problem many times in the past with incompletely-implemented ranges, and it can be fairly annoying to track down the cause with dbi + a few layers of range interfaces.

If I recall correctly, there were many attempts to solve this in the past (https://github.com/dlang/DIPs/blob/master/DIPs/other/DIP1022.md being one of them), and there are many related issues, both open and closed.

- https://issues.dlang.org/show_bug.cgi?id=11934 ("fixed", but the provided
test case still fails)
- https://issues.dlang.org/show_bug.cgi?id=24232
- https://issues.dlang.org/show_bug.cgi?id=5306 (and the subsequent
https://issues.dlang.org/show_bug.cgi?id=6214 )

--

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

Dlang Bot <dlang-bot@dlang.rocks> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Keywords|                            |pull

--- Comment #2 from Dlang Bot <dlang-bot@dlang.rocks> ---
@ntrel created dlang/dmd pull request #16381 "[foreach] Deprecate `ref` for array index or range parameter" fixing this issue:

- Deprecate `ref` for array index or range foreach parameter

  Fix Bugzilla 24499 - foreach over a ref parameter to an rvalue should be an
error
  Fix Bugzilla 24232 - ref for index of foreach for arrays is not allowed by
spec but accepted by compiler

https://github.com/dlang/dmd/pull/16381

--