-
-
Notifications
You must be signed in to change notification settings - Fork 1.5k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
dotOperators
, callOperator
produce bad/misleading undeclared field error msgs
#13063
Labels
Comments
timotheecour
changed the title
{.experimental: "dotOperators".} produce bad error msgs
{.experimental: "dotOperators".} produce bad/misleading undeclared field error msgs
Jan 7, 2020
timotheecour
changed the title
{.experimental: "dotOperators".} produce bad/misleading undeclared field error msgs
Feb 3, 2021
dotOperators
, callOperator
produce bad/misleading undeclared field error msgs
@hlaaftana do you have any interest in working on this since you've looked at related code with #16924? this issue makes it awkward to use things like wrapnils or jsffi etc, since (even after your PR, albeit less so) you get bad errors |
I love dotOperators, except for this bug they are perfect. Can't wait for it to be fixed. |
treeform
added a commit
to treeform/Nim
that referenced
this issue
Nov 19, 2021
Closed
treeform
added a commit
to treeform/vmath
that referenced
this issue
Nov 20, 2021
metagn
added a commit
to metagn/Nim
that referenced
this issue
Nov 26, 2022
Araq
pushed a commit
that referenced
this issue
Nov 28, 2022
survivorm
pushed a commit
to survivorm/Nim
that referenced
this issue
Feb 28, 2023
* better error messages for dot operators [backport] fixes nim-lang#13063 * also fixes nim-lang#7777 * fix nim-lang#6981 and nim-lang#9831 too * fix * minor improvement * sus test fixes * make test multiplatform lol * fix nimsuggest test, extra improvements
capocasa
pushed a commit
to capocasa/Nim
that referenced
this issue
Mar 31, 2023
* better error messages for dot operators [backport] fixes nim-lang#13063 * also fixes nim-lang#7777 * fix nim-lang#6981 and nim-lang#9831 too * fix * minor improvement * sus test fixes * make test multiplatform lol * fix nimsuggest test, extra improvements
bung87
pushed a commit
to bung87/Nim
that referenced
this issue
Jul 29, 2023
* better error messages for dot operators [backport] fixes nim-lang#13063 * also fixes nim-lang#7777 * fix nim-lang#6981 and nim-lang#9831 too * fix * minor improvement * sus test fixes * make test multiplatform lol * fix nimsuggest test, extra improvements
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
when a dotOperators is defined (with
{.experimental: "dotOperators".}
), it affects unrelated undeclared field error messages.Example 1
Current Output
Expected Output
should contain the following errmsg which is what we get without
template
.(a: Foo, b: untyped): untyped = 123
:in addition to the above errmsg
Error: type mismatch: got <Bar>...
Example 2
callOperator
has the same problemsAdditional Information
The text was updated successfully, but these errors were encountered: