Skip to content

Support path function calls in CallExpr#232

Merged
makenowjust merged 2 commits intocloudspannerecosystem:mainfrom
apstndb:feature/call-expr-path
Dec 20, 2024
Merged

Support path function calls in CallExpr#232
makenowjust merged 2 commits intocloudspannerecosystem:mainfrom
apstndb:feature/call-expr-path

Conversation

@apstndb
Copy link
Copy Markdown
Contributor

@apstndb apstndb commented Dec 18, 2024

This PR enhances CallExpr to support path as function name.
This change is for:

  • Supporting SAFE. prefix
  • Supporting some category of functions

Breaking Chenges

  • Func field of CallExpr is changed to *ast.Path from *ast.Ident.

Related Issue

Copy link
Copy Markdown
Collaborator

@makenowjust makenowjust left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 👍

@makenowjust makenowjust merged commit 11a7b9c into cloudspannerecosystem:main Dec 20, 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

Successfully merging this pull request may close these issues.

Incompatibility: SAFE functions are not supported

2 participants