[Issue 20908] -preview=nosharedaccess requires zero-initializion for aggregates

Jun 08, 2020

moonlightsentinel@disroot.org

Sep 13, 2020

Dlang Bot

Dec 17, 2022

Mar 16, 2023

Mar 22, 2023

Mar 22, 2023

Mar 22, 2023

Apr 01, 2023

https://issues.dlang.org/show_bug.cgi?id=20908 moonlightsentinel@disroot.org changed: What |Removed |Added ---------------------------------------------------------------------------- See Also| |https://issues.dlang.org/sh | |ow_bug.cgi?id=20195 --

September 13, 2020

[Issue 20908] -preview=nosharedaccess requires zero-initializion for aggregates

Posted by Dlang Bot

Permalink

Dlang Bot

Permalink

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

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

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

--- Comment #1 from Dlang Bot <dlang-bot@dlang.rocks> ---
@Geod24 created dlang/dmd pull request #11726 "Fix 20908 - Improve `shared` access detection" fixing this issue:

- Fix 20908 - Improve `shared` access detection

  There were quite a few positives with the previous approach.
  Essentially, we cannot perform it during expression semantic,
  within things like `visit(PtrExp)` methods, because we need
  to be aware of our context.

  Instead of adding yet another flag to `Scope` (which would need
  to be flipped on and off), we use a visitor called from a higher
  level (namely, the unary / binary expression semantic).

  This should also make it easier to extend in the future,
  for example by adding support for bypassing the check if
  the constructor is `pure`.

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

--

https://issues.dlang.org/show_bug.cgi?id=20908 Iain Buclaw <ibuclaw@gdcproject.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Priority|P1 |P3 --

https://issues.dlang.org/show_bug.cgi?id=20908 Nick Treleaven <nick@geany.org> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |nick@geany.org --- Comment #2 from Nick Treleaven <nick@geany.org> --- Still happens with v2.102.2. I don't think the pull really got merged. --

https://issues.dlang.org/show_bug.cgi?id=20908 --- Comment #3 from Dlang Bot <dlang-bot@dlang.rocks> --- @RazvanN7 created dlang/dmd pull request #15023 "Fix Issue 20908 - -preview=nosharedaccess requires zero-initializion for aggregates" fixing this issue: - Fix Issue 20908 - -preview=nosharedaccess requires zero-initializion for aggregates https://github.com/dlang/dmd/pull/15023 --

https://issues.dlang.org/show_bug.cgi?id=20908 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 #15023 "Fix Issue 20908 - -preview=nosharedaccess requires zero-initializion for aggregates" was merged into stable: - 06fe8c4c09c97e7c666418cd568e30eb6f5b195d by RazvanN7: Fix Issue 20908 - -preview=nosharedaccess requires zero-initializion for aggregates https://github.com/dlang/dmd/pull/15023 --

https://issues.dlang.org/show_bug.cgi?id=20908 --- Comment #5 from Dlang Bot <dlang-bot@dlang.rocks> --- dlang/dmd pull request #15066 "merge stable" was merged into master: - ec071370017911d5984675c03c28e9b7603c3b14 by Razvan Nitu: Fix Issue 20908 - -preview=nosharedaccess requires zero-initializion for aggregates (#15023) https://github.com/dlang/dmd/pull/15066 --

Forums