Thread overview
[Issue 20869] `std.algorithm.mutation : move` is overly trusting of `opPostMove`
May 28, 2020
Dlang Bot
May 31, 2020
Dlang Bot
Sep 15, 2020
Dlang Bot
May 28, 2020
https://issues.dlang.org/show_bug.cgi?id=20869

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

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

--- Comment #1 from Dlang Bot <dlang-bot@dlang.rocks> ---
@MoonlightSentinel created dlang/phobos pull request #7502 "Fix Issue 20869 - `move` is overly trusting of `opPostMove`" fixing this issue:

- Fix Issue 20869 - `move` is overly trusting of `opPostMove`

  Remove the manual check whether move is `@safe` and instead let the
  compiler do the attribute interference by adding appropriate @trusted
  blocks.

  Note:
  I could've extend the current checks `trustedMoveImpl` but that could
  easily miss other corner cases.

https://github.com/dlang/phobos/pull/7502

--
May 31, 2020
https://issues.dlang.org/show_bug.cgi?id=20869

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/phobos pull request #7502 "Fix Issue 20869 - `move` is overly trusting of `opPostMove`" was merged into master:

- 082f8300d9d1a423b73cd31320bc86efa78a37e0 by MoonlightSentinel:
  Fix Issue 20869 - `move` is overly trusting of `opPostMove`

  Remove the manual check whether move is `@safe` and instead let the
  compiler do the attribute interference by adding appropriate @trusted
  blocks.

  Note:
  I could've extend the current checks `trustedMoveImpl` but that could
  easily miss other corner cases.

https://github.com/dlang/phobos/pull/7502

--
September 15, 2020
https://issues.dlang.org/show_bug.cgi?id=20869

--- Comment #3 from Dlang Bot <dlang-bot@dlang.rocks> ---
dlang/druntime pull request #3133 "Backport move(Emplace) fixes from Phobos"
was merged into master:

- aa829e3139fe93030bf68a8d216c6956188b4861 by MoonlightSentinel:
  Fix Issue 20869 - `move` is overly trusting of `opPostMove`

  Backport of dlang/phobos#7502

https://github.com/dlang/druntime/pull/3133

--