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

SPT parse to ATerm constructor names don't support underscores #67

Closed
RvanBaarle opened this issue Sep 28, 2021 · 1 comment
Closed
Labels
C-Bug Category: bug. Not working as expected P-High Priority: high. Should take priority over low and non-prioritized

Comments

@RvanBaarle
Copy link
Contributor

Summary

Constructor names containing underscores aren't parsed correctly in SPT files.

What you did

Defining a constructor such as

Start.Hello_World = <Hello world>

And a test such a

test hello [[
Hello world
]] parse to Hello_World()

What you expected to happen

The test should pass

What actually happened

The test fails with the error "Expected parse to (), but got Hello_World()"

Context

  • Spoofax version: devenv (spoofax.pie commit c7ed510)
  • Operating system & version: Ubuntu 20.04
  • Java version (if applicable): 11.0.11
  • Eclipse version (if applicable): Version: 2021-03 (4.19) Build id: I20210303-1800
  • Gradle version (if applicable): 6.8

Additional information

The SPT file does evaluate, but "Show parsed AST" does not produce an AST.

@RvanBaarle RvanBaarle added the C-Bug Category: bug. Not working as expected label Sep 28, 2021
@Gohla Gohla added the P-High Priority: high. Should take priority over low and non-prioritized label Sep 28, 2021
@Gohla
Copy link
Member

Gohla commented Oct 1, 2021

Solved by metaborg/spt@a5253d1

@Gohla Gohla closed this as completed Oct 1, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-Bug Category: bug. Not working as expected P-High Priority: high. Should take priority over low and non-prioritized
Projects
None yet
Development

No branches or pull requests

2 participants