Jump to page: 1 2
Thread overview
[Issue 20717] Unsilenced bogus "undefined identifier" error from speculative collision
Apr 01, 2020
Vladimir Panteleev
Apr 03, 2020
Basile-z
Apr 03, 2020
Basile-z
Apr 03, 2020
Basile-z
Apr 03, 2020
Basile-z
Apr 04, 2020
Boris Carvajal
Apr 04, 2020
Basile-z
Apr 04, 2020
Basile-z
Apr 04, 2020
Basile-z
Mar 29, 2022
Dlang Bot
Mar 30, 2022
Dlang Bot
Apr 01, 2022
Dlang Bot
April 01, 2020
https://issues.dlang.org/show_bug.cgi?id=20717

Vladimir Panteleev <dlang-bugzilla@thecybershadow.net> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Keywords|                            |rejects-valid

--
April 03, 2020
https://issues.dlang.org/show_bug.cgi?id=20717

--- Comment #1 from Basile-z <b2.temp@gmx.com> ---
The problem is that the semantic of the alias is run at the wrong time, i.e the error are not gagged but in the present context they should. The gagging is activated when the IsExp is visited. But the alias sema is run when the overload is inserted.

--
April 03, 2020
https://issues.dlang.org/show_bug.cgi?id=20717

--- Comment #2 from Basile-z <b2.temp@gmx.com> ---
Created attachment 1779
  --> https://issues.dlang.org/attachment.cgi?id=1779&action=edit
workaround

As officially I dont participate to dlang anymore but also as a commit of mine causes the regression I send a patch.

The fix is not really good but comes with explanations about what was done in first place, i.e for issue 15795. Ideally issue 15795 should be fixed in another way but I remember that I could not find a better way when I made the PR.

--
April 03, 2020
https://issues.dlang.org/show_bug.cgi?id=20717

Basile-z <b2.temp@gmx.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
                 CC|                            |b2.temp@gmx.com
         Resolution|---                         |FIXED

--- Comment #3 from Basile-z <b2.temp@gmx.com> ---
good news, this problem is fixed by https://github.com/dlang/dmd/pull/10952/

--
April 03, 2020
https://issues.dlang.org/show_bug.cgi?id=20717

Basile-z <b2.temp@gmx.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|b2.temp@gmx.com             |

--
April 04, 2020
https://issues.dlang.org/show_bug.cgi?id=20717

Boris Carvajal <boris2.9@gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|RESOLVED                    |REOPENED
                 CC|                            |boris2.9@gmail.com
         Resolution|FIXED                       |---

--- Comment #4 from Boris Carvajal <boris2.9@gmail.com> ---
I just checked the problem is still there.

--
April 04, 2020
https://issues.dlang.org/show_bug.cgi?id=20717

Basile-z <b2.temp@gmx.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |b2.temp@gmx.com

--- Comment #5 from Basile-z <b2.temp@gmx.com> ---
indeed, the patch I sent (workaround) was still applied.

--
April 04, 2020
https://issues.dlang.org/show_bug.cgi?id=20717

--- Comment #6 from Basile-z <b2.temp@gmx.com> ---
Ok I've find what really happens. When you merge the two tests in a single
module, so the one reported here and the one here
https://github.com/dlang/dmd/pull/10078/files#diff-b33aefca6dc2d2014d789a2d8727fb1eR1,
then this compiles because symbols are solved differently.

--
April 04, 2020
https://issues.dlang.org/show_bug.cgi?id=20717

Basile-z <b2.temp@gmx.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|b2.temp@gmx.com             |

--
March 29, 2022
https://issues.dlang.org/show_bug.cgi?id=20717

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

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

--- Comment #7 from Dlang Bot <dlang-bot@dlang.rocks> ---
@RazvanN7 created dlang/dmd pull request #13922 "Fix Issue 20717 - Unsilenced bogus undefined identifier error from speculative collision" fixing this issue:

- Fix Issue 20717 - Unsilenced bogus undefined identifier error from speculative collision

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

--
« First   ‹ Prev
1 2