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

Parameters of external procedure call are not parsed #65

Open
eglekaz opened this issue Sep 4, 2024 · 3 comments
Open

Parameters of external procedure call are not parsed #65

eglekaz opened this issue Sep 4, 2024 · 3 comments

Comments

@eglekaz
Copy link

eglekaz commented Sep 4, 2024

Example:
run getMyNumber.p(piFirstNum, piSecNum).

Calls for internal procedures are being parsed correctly.

Reference:
https://documentation.progress.com/output/ua/OpenEdge_latest/pdsoe/PLUGINS_ROOT/com.openedge.pdt.langref.help/rfi1424920323961.html

@jkbz64
Copy link
Member

jkbz64 commented Sep 4, 2024

If you see abl_statement node in the AST output it means the statement is simply not implemented yet, abl_statement node is just a filler to make more code "parsable" and it is meant to be deleted somewhere along the line if the parser manages to implement every statement (press x to doubt)

@eglekaz
Copy link
Author

eglekaz commented Sep 5, 2024

Thank you for explanation!
These errors do not break our logic, so I will close this issue for now.

@eglekaz eglekaz closed this as completed Sep 5, 2024
@jkbz64
Copy link
Member

jkbz64 commented Sep 5, 2024

Nah, it's fine to reopen this issue, it's that this issue is not about parameters, it's more "Run statement is not yet implemented".

@jkbz64 jkbz64 reopened this Sep 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants