Thread overview
[Issue 13934] Cannot pass 'this' to function by reference
Jan 05, 2015
Kenji Hara
Mar 05, 2015
Kenji Hara
Mar 09, 2015
Kenji Hara
Mar 09, 2015
Dicebot
Mar 09, 2015
Dicebot
Mar 09, 2015
Dicebot
January 05, 2015
https://issues.dlang.org/show_bug.cgi?id=13934

Kenji Hara <k.hara.pg@gmail.com> changed:

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

--- Comment #1 from Kenji Hara <k.hara.pg@gmail.com> ---
It's an intended bugfix for the issue 13116.
'this' and 'super' should not be lvalues to avoid illegal rebindings.

See the detailed cases in the issue ticket.

--
March 05, 2015
https://issues.dlang.org/show_bug.cgi?id=13934

Kenji Hara <k.hara.pg@gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |dragoscarp@gmail.com

--- Comment #2 from Kenji Hara <k.hara.pg@gmail.com> ---
*** Issue 14244 has been marked as a duplicate of this issue. ***

--
March 09, 2015
https://issues.dlang.org/show_bug.cgi?id=13934

Kenji Hara <k.hara.pg@gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |public@dicebot.lv

--- Comment #3 from Kenji Hara <k.hara.pg@gmail.com> ---
*** Issue 14262 has been marked as a duplicate of this issue. ***

--
March 09, 2015
https://issues.dlang.org/show_bug.cgi?id=13934

Dicebot <public@dicebot.lv> changed:

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

--- Comment #4 from Dicebot <public@dicebot.lv> ---
Quoting myself from issue that was closed as duplicate:

"While refusing to compile this seems legit and code clearly needs fixing, error message is not helpful at all. It should explain the issue and suggest a fix more clearly."

Meaningful error messages are very important minimize frustration from broken upgrade.

Looking into it.

--
March 09, 2015
https://issues.dlang.org/show_bug.cgi?id=13934

--- Comment #5 from Dicebot <public@dicebot.lv> ---
btw the fact that 3 different people have reported it as a bug should have been a clear sign that the fix wasn't done clear enough

--
March 09, 2015
https://issues.dlang.org/show_bug.cgi?id=13934

Dicebot <public@dicebot.lv> changed:

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

--- Comment #6 from Dicebot <public@dicebot.lv> ---
Fixed by https://github.com/D-Programming-Language/dmd/pull/4463

Thanks Martin!

--