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.

Sign up for GitHub

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

Jump to bottom

funcId in grammar.ohm of Tact doesn't match the possible range of identifiers and function names in FunC #627

Closed
novusnota opened this issue Jul 26, 2024 · 2 comments · Fixed by #628
Closed

funcId in grammar.ohm of Tact doesn't match the possible range of identifiers and function names in FunC #627

novusnota opened this issue Jul 26, 2024 · 2 comments · Fixed by #628
Assignees
@novusnota
Labels
kind: bug Something isn't working or isn't right scope: parser Implementation of parser (src/grammar)
Milestone
v1.4.1

Comments

@novusnota
Copy link
Member

See:
@novusnota novusnota added kind: bug Something isn't working or isn't right syntax labels Jul 26, 2024
@novusnota novusnota added this to the v1.4.1 milestone Jul 26, 2024
@novusnota novusnota self-assigned this Jul 26, 2024
@anton-trunov
Copy link
Member

btw, the two descriptions do not really correspond. It seems to me the grammar in the TON IntelliJ plugin admits 42 as a valid FunC identifier

@novusnota
Copy link
Member Author

novusnota commented Jul 26, 2024 •
edited
Loading

btw, the two descriptions do not really correspond. It seems to me the grammar in the TON IntelliJ plugin admits 42 as a valid FunC identifier

Yup, I discovered divergencies in tests too. I'll stick to the TON docs and the source code of FunC

This was referenced Jul 26, 2024
Merged
Closed
Closed
@verytactical verytactical added scope: parser Implementation of parser (src/grammar) and removed syntax labels Jan 20, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@novusnota novusnota

Labels
kind: bug Something isn't working or isn't right scope: parser Implementation of parser (src/grammar)
Projects
None yet
Milestone
v1.4.1
Development

Successfully merging a pull request may close this issue.

fix: mostly accurate funcId representation tact-lang/tact
3 participants
@anton-trunov @novusnota @verytactical