Skip to content

Derive a local's name without the keyword escape - #33

Merged
estebanzimanyi merged 1 commit into
MobilityDB:mainfrom
estebanzimanyi:fix/scratch-name-without-the-escape
Sep 3, 2026
Merged

Derive a local's name without the keyword escape#33
estebanzimanyi merged 1 commit into
MobilityDB:mainfrom
estebanzimanyi:fix/scratch-name-without-the-escape

Conversation

@estebanzimanyi

Copy link
Copy Markdown
Member

A parameter whose name is a C# keyword is escaped, so double *out reaches the
layer as @out — and the locals a body derives from it read _@out, which is
no identifier at all: the escape belongs at the start of a name or nowhere.
Every body that allocates, pins or frees around a call derives such a name, so
one function answers for all of them and the escape is dropped there.

MEOS names an out-parameter out in the four point-cloud accessors
pcpoint_get_x, _get_y, _get_z and _get_dim, and they reach the object
layer as soon as the schema they take has a class, so what the catalog answers
today does not compile. The four read _out and build.

A parameter whose name is a C# keyword is escaped, so `double *out` reaches the
layer as `@out` — and the locals a body derives from it read `_@out`, which is
no identifier at all: the escape belongs at the start of a name or nowhere.
Every body that allocates, pins or frees around a call derives such a name, so
one function answers for all of them and the escape is dropped there.

MEOS names an out-parameter `out` in the four point-cloud accessors
`pcpoint_get_x`, `_get_y`, `_get_z` and `_get_dim`, and they reach the object
layer as soon as the schema they take has a class, so what the catalog answers
today does not compile. The four read `_out` and build.
@estebanzimanyi
estebanzimanyi merged commit 3684ef4 into MobilityDB:main Sep 3, 2026
1 check passed
@estebanzimanyi
estebanzimanyi deleted the fix/scratch-name-without-the-escape branch September 3, 2026 10:13
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.

1 participant