[Issue 21590] assignment inside assert accepted if -checkaction=context is given

d-bugmail at puremagic.com d-bugmail at puremagic.com
Fri Jan 29 13:48:57 UTC 2021


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

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

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

--- Comment #2 from Dlang Bot <dlang-bot at dlang.rocks> ---
@MoonlightSentinel created dlang/dmd pull request #12165 "Fix 21590 -
assignment inside assert accepted for -checkaction=context" fixing this issue:

- Fix 21590 - assignment inside assert accepted for -checkaction=context

  The rewrite introduced a temporary which hid the assignment inside of
  `assert(...)` and hence prevented the error.

  The fix is to omit the additional temporary for `AssignExp` and use
  the assigned variable directly. The temporary is unecessary anyways
  and the following semantic analysis will raise an appropriate error.

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

--


More information about the Digitalmars-d-bugs mailing list