February 03, 2021 [Issue 21508] private class p in file p.d visible outside the file (module) | ||||
|---|---|---|---|---|
| ||||
 | https://issues.dlang.org/show_bug.cgi?id=21508 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 #12178 "Fix 21508: Do not look inside imports that shadow a symbol" fixing this issue: - Fix 21508: Do not look inside imports that shadow a symbol There was a special case to look inside an import instead of showing the conflict. This isn't consistent with the way imports appear to the user (e.g. `class std {}` will conflict with `import std.stdio`), so we just remove the special case and that solves the issue. https://github.com/dlang/dmd/pull/12178 -- | |||
Permalink
ReplyFebruary 06, 2021 [Issue 21508] private class p in file p.d visible outside the file (module) | ||||
|---|---|---|---|---|
| ||||
| https://issues.dlang.org/show_bug.cgi?id=21508 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 #12178 "Fix 21508: Do not look inside imports that shadow a symbol" was merged into master: - 1d2bdbb597344b6398e6db7d9b4586dcf14735bc by Geod24: Fix 21508: Do not look inside imports that shadow a symbol There was a special case to look inside an import instead of showing the conflict. This isn't consistent with the way imports appear to the user (e.g. `class std {}` will conflict with `import std.stdio`), so we just remove the special case and that solves the issue. https://github.com/dlang/dmd/pull/12178 -- | |||
Copyright © 1999-2021 by the D Language Foundation