Thread overview
[Issue 24353] Misleading error for foreach when opApply has wrong qualifier
5 days ago
Dlang Bot
5 days ago
Dlang Bot
5 days ago
Paul Backus
5 days ago
https://issues.dlang.org/show_bug.cgi?id=24353

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

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

--- Comment #1 from Dlang Bot <dlang-bot@dlang.rocks> ---
@royalpinto007 created dlang/dmd pull request #17071 "fix Bugzilla 24353 - add mutability check for foreach with opApply" fixing this issue:

- fix Bugzilla 24353 - add mutability check for foreach with opApply

  Signed-off-by: royalpinto007 <royalpinto007@gmail.com>

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

--
5 days ago
https://issues.dlang.org/show_bug.cgi?id=24353

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

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
         Resolution|---                         |FIXED

--- Comment #2 from Dlang Bot <dlang-bot@dlang.rocks> ---
dlang/dmd pull request #17071 "fix Bugzilla 24353 - add mutability check for foreach with opApply" was merged into master:

- 7ce408a4370a7f308c173b9f233ed7f242b926dc by royalpinto007:
  fix Bugzilla 24353 - add mutability check for foreach with opApply

  Signed-off-by: royalpinto007 <royalpinto007@gmail.com>

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

--
5 days ago
https://issues.dlang.org/show_bug.cgi?id=24353

Paul Backus <snarwin+bugzilla@gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|RESOLVED                    |REOPENED
         Resolution|FIXED                       |---

--- Comment #3 from Paul Backus <snarwin+bugzilla@gmail.com> ---
The linked PR is an incomplete fix. The example with `const` now works, but the same problem still exists with other qualifiers, such as `shared` and `immutable`.

--