Thread overview
[Issue 22519] [dip1000] cannot take address of `ref return`
Nov 23, 2021
Dlang Bot
Nov 25, 2021
Stanislav Blinov
Nov 25, 2021
Dennis
Nov 26, 2021
Dlang Bot
November 23, 2021
https://issues.dlang.org/show_bug.cgi?id=22519

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

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

--- Comment #1 from Dlang Bot <dlang-bot@dlang.rocks> ---
@dkorpel created dlang/dmd pull request #13351 "Fix Issue 22519 - cannot take address of `ref return`" fixing this issue:

- Fix Issue 22519 - cannot take address of `ref return`

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

--
November 25, 2021
https://issues.dlang.org/show_bug.cgi?id=22519

Stanislav Blinov <stanislav.blinov@gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |stanislav.blinov@gmail.com

--- Comment #2 from Stanislav Blinov <stanislav.blinov@gmail.com> ---
How does this interact with -preview=rvaluerefparam ?

I smell a bug with that `addressOf` implementation, though it probably should be addressed in the implementation of the abovementioned switch.

--
November 25, 2021
https://issues.dlang.org/show_bug.cgi?id=22519

--- Comment #3 from Dennis <dkorpel@live.nl> ---
(In reply to Stanislav Blinov from comment #2)
> How does this interact with -preview=rvaluerefparam ?

I thought rvaluerefparam was syntactic sugar for creating a local variable just before the function call and passing that to the function parameter, so it should work just fine.

I also agree that it should be the concern of the rvaluerefparam implementation, especially since it's not in the language yet and the DIP for it hasn't even reached draft review.

--
November 26, 2021
https://issues.dlang.org/show_bug.cgi?id=22519

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

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

--- Comment #4 from Dlang Bot <dlang-bot@dlang.rocks> ---
dlang/dmd pull request #13351 "Fix Issue 22519 - cannot take address of `ref return`" was merged into master:

- 3f1c763b9ed58f323550bd6d99e3dec031016f7b by dkorpel:
  Fix Issue 22519 - cannot take address of `ref return`

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

--