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

--- Comment #8 from Steven Schveighoffer <schveiguy@yahoo.com> ---
(In reply to Walter Bright from comment #7)
> Also, this was not a regression.

I distinctly remember at some point in the last 10 years that you HAD to mark opApply scope or it would allocate a closure. How is this not a regression?

It's not terribly important if it gets fixed now, but I'm just surprised if this has always been the behavior.

--

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

--- Comment #9 from Walter Bright <bugzilla@digitalmars.com> ---
Related spec change:

    https://github.com/dlang/dlang.org/pull/1473

--

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

--- Comment #10 from Walter Bright <bugzilla@digitalmars.com> ---
It's possible that was true briefly, but it would have been a regression in itself because of its negative affect on legacy code.

--

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

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

https://github.com/dlang/dmd/commit/f7819c898a05d28297024b7355d4ace94f1e4465 fix Issue 16193 - opApply() doesn't heap allocate closure

https://github.com/dlang/dmd/commit/946a8f4e0eb488313e53da41ad6f81e3a778b9a8 Merge pull request #6135 from WalterBright/fix16193

fix Issue 16193 - opApply() doesn't heap allocate closure

--

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

github-bugzilla@puremagic.com changed:

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

--

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

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

https://github.com/dlang/dmd/commit/f7819c898a05d28297024b7355d4ace94f1e4465 fix Issue 16193 - opApply() doesn't heap allocate closure

https://github.com/dlang/dmd/commit/946a8f4e0eb488313e53da41ad6f81e3a778b9a8 Merge pull request #6135 from WalterBright/fix16193

--

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

--- Comment #13 from github-bugzilla@puremagic.com ---
Commit pushed to stable at https://github.com/dlang/dmd

https://github.com/dlang/dmd/commit/aac715ff2597d256b5ffe8ecb397316abf32fb2e Revert "fix Issue 16193 - opApply() doesn't heap allocate closure"

This reverts commit f7819c898a05d28297024b7355d4ace94f1e4465.

Fixes issue 16678

--

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

--- Comment #14 from github-bugzilla@puremagic.com ---
Commit pushed to master at https://github.com/dlang/dmd

https://github.com/dlang/dmd/commit/aac715ff2597d256b5ffe8ecb397316abf32fb2e Revert "fix Issue 16193 - opApply() doesn't heap allocate closure"

--

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

Dicebot <public@dicebot.lv> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|RESOLVED                    |REOPENED
                 CC|                            |public@dicebot.lv
         Resolution|FIXED                       |---

--

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

--- Comment #15 from github-bugzilla@puremagic.com ---
Commit pushed to scope at https://github.com/dlang/dmd

https://github.com/dlang/dmd/commit/aac715ff2597d256b5ffe8ecb397316abf32fb2e Revert "fix Issue 16193 - opApply() doesn't heap allocate closure"

--