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

Parser fails if an identifier contains '@' #437

Closed
yruslan opened this issue Nov 12, 2021 · 0 comments · Fixed by #438
Closed

Parser fails if an identifier contains '@' #437

yruslan opened this issue Nov 12, 2021 · 0 comments · Fixed by #438
Assignees
@yruslan
Labels
bug Something isn't working

Comments

@yruslan
Copy link
Collaborator

yruslan commented Nov 12, 2021

Describe the bug

Given the following copybook:

      01  ROOT-GROUP.
          05  F@TEST PIC X(1).

Got:

Syntax error in the copybook at line 3: Invalid input 'TEST' at position 3:16
za.co.absa.cobrix.cobol.parser.exceptions.SyntaxErrorException: Syntax error in the copybook at line 3: Invalid input 'TEST' at position 3:16

Expected behaviour

Cobrix should replace such characters with '_' as in 0.5.4.
@yruslan yruslan added the bug Something isn't working label Nov 12, 2021
@yruslan yruslan self-assigned this Nov 12, 2021
yruslan added a commit that referenced this issue Nov 15, 2021
@yruslan
#437 Added support for '@' character as a part of identifiers.
5251ae0
@yruslan yruslan linked a pull request Nov 15, 2021 that will close this issue
#437 Added support for '@' character as a part of identifiers. #438
Merged
yruslan added a commit that referenced this issue Nov 15, 2021
@yruslan
#437 Update README.
f806f1e
yruslan added a commit that referenced this issue Nov 15, 2021
@yruslan
#437 Added support for '@' character as a part of identifiers.
52b721d
yruslan added a commit that referenced this issue Nov 15, 2021
@yruslan
#437 Update README.
29163c1
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@yruslan yruslan

Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

#437 Added support for '@' character as a part of identifiers. AbsaOSS/cobrix
1 participant
@yruslan