Thread overview
[Issue 24159] BetterC: appending to dynamic arrays no longer errors at compile time
Sep 28, 2023
RazvanN
Nov 10, 2023
Dlang Bot
Nov 13, 2023
Dlang Bot
Nov 20, 2023
Dlang Bot
September 28, 2023
https://issues.dlang.org/show_bug.cgi?id=24159

RazvanN <razvan.nitu1305@gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |razvan.nitu1305@gmail.com
           Severity|normal                      |regression

--
November 10, 2023
https://issues.dlang.org/show_bug.cgi?id=24159

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

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

--- Comment #1 from Dlang Bot <dlang-bot@dlang.rocks> ---
@teodutu updated dlang/dmd pull request #15791 "Fix Issue 21459: Store lowering of `CatAssignExp` in a separate field" fixing this issue:

- Fix Issue 24159: Store lowering of `CatAssignExp` in a separate field

  This preserves the `CatAssignExp` in the AST until the glue layer where
  an error is printed in case this expression is used with `-betterC`.
  This is required to happen in the glue layer as the semantic analysis
  doesn't correctly distinguish when code needs to be generated.

  Signed-off-by: Teodor Dutu <teodor.dutu@gmail.com>

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

--
November 13, 2023
https://issues.dlang.org/show_bug.cgi?id=24159

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/dmd pull request #15791 "Fix Issue 21459: Store lowering of `CatAssignExp` in a separate field" was merged into stable:

- 71518bee8b2779a9cd6064537470582dae4c29c1 by Teodor Dutu:
  Fix Issue 24159: Store lowering of `CatAssignExp` in a separate field

  This preserves the `CatAssignExp` in the AST until the glue layer where
  an error is printed in case this expression is used with `-betterC`.
  This is required to happen in the glue layer as the semantic analysis
  doesn't correctly distinguish when code needs to be generated.

  Signed-off-by: Teodor Dutu <teodor.dutu@gmail.com>

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

--
November 20, 2023
https://issues.dlang.org/show_bug.cgi?id=24159

--- Comment #3 from Dlang Bot <dlang-bot@dlang.rocks> ---
dlang/dmd pull request #15829 "merge stable" was merged into master:

- b9f8e7cf24273f2283a39f703b2367c9cb09a0dc by Teodor Dutu:
  Fix Issue 24159: Store lowering of `CatAssignExp` in a separate field
(#15791)

  This preserves the `CatAssignExp` in the AST until the glue layer where
  an error is printed in case this expression is used with `-betterC`.
  This is required to happen in the glue layer as the semantic analysis
  doesn't correctly distinguish when code needs to be generated.

  Signed-off-by: Teodor Dutu <teodor.dutu@gmail.com>

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

--