Skip to content

Add param name subsection - #9

Open
bvisness wants to merge 2 commits into
mainfrom
param-names
Open

Add param name subsection#9
bvisness wants to merge 2 commits into
mainfrom
param-names

Conversation

@bvisness

Copy link
Copy Markdown
Collaborator

Adds a param name subsection specifically for parameters declared in function types.

Unlike #7, this PR simply adds a new subsection, rather than attempting to extend the behavior of the field name subsection. To me this feels more natural because it avoids confusion about index spaces, aggregate vs. composite types, etc. In particular, it avoids the need to define a new, proper index space that is only used for names, or to extend the definition of an existing address space in a similar way.

Resolves #4.

@bvisness
bvisness requested a review from rossberg August 14, 2026 15:31
@bvisness
bvisness marked this pull request as draft August 14, 2026 17:24
@bvisness

Copy link
Copy Markdown
Collaborator Author

Oh, joyous day, I have just realized that tag types also use (param). Back to draft status for me.

@bvisness
bvisness removed the request for review from rossberg August 14, 2026 17:25
@bvisness
bvisness marked this pull request as ready for review August 14, 2026 18:55
@bvisness
bvisness requested a review from rossberg August 14, 2026 18:55
@bvisness

Copy link
Copy Markdown
Collaborator Author

Actually, because tags use function types, this is still fine as is. Apologies for the noise.

That said, we might need to discuss what to do with tags a bit more in #4.

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.

Support round-tripping parameter names from function type definitions

1 participant