Skip to content
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

Macro + Infix + quote do - compilation error #7589

Closed
mratsim opened this issue Apr 12, 2018 · 3 comments · Fixed by #17256 or #17426
Closed

Macro + Infix + quote do - compilation error #7589

mratsim opened this issue Apr 12, 2018 · 3 comments · Fixed by #17256 or #17426

Comments

@mratsim
Copy link
Collaborator

mratsim commented Apr 12, 2018

cc @krux02

By trying to change the quasi-quoting symbol to allow quote do with infix I stubled upon this:

import macros

macro foo(x: untyped): untyped =

  result = quote do:
    echo `==`(3,4) # echo ["false"]' has no type (or is ambiguous)
  do: "@@"

  echo result.treerepr
@krux02
Copy link
Contributor

krux02 commented Apr 13, 2018

I wasn't able to change the quasi quoting symbol at all. There should be a test case for that.

@stale
Copy link

stale bot commented Aug 4, 2020

This issue has been automatically marked as stale because it has not had recent activity. If you think it is still a valid issue, write a comment below; otherwise it will be closed. Thank you for your contributions.

@stale stale bot added the stale Staled PR/issues; remove the label after fixing them label Aug 4, 2020
@kamilchm
Copy link

Is there any workaround for this?

@stale stale bot removed the stale Staled PR/issues; remove the label after fixing them label Feb 19, 2021
Araq pushed a commit that referenced this issue Mar 5, 2021
Provides a workaround/fix for #7589.
#7589

Updated docs and tutorial to reflect change.

Updated runnableExamples to include an example.

Co-authored-by: name <[email protected]>
ringabout pushed a commit to ringabout/Nim that referenced this issue Mar 22, 2021
Provides a workaround/fix for nim-lang#7589.
nim-lang#7589

Updated docs and tutorial to reflect change.

Updated runnableExamples to include an example.

Co-authored-by: name <[email protected]>
ardek66 pushed a commit to ardek66/Nim that referenced this issue Mar 26, 2021
Provides a workaround/fix for nim-lang#7589.
nim-lang#7589

Updated docs and tutorial to reflect change.

Updated runnableExamples to include an example.

Co-authored-by: name <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
4 participants