Skip to content

Rename Array VTables#6907

Open
gatesn wants to merge 2 commits intodevelopfrom
ngates/rename-vtables
Open

Rename Array VTables#6907
gatesn wants to merge 2 commits intodevelopfrom
ngates/rename-vtables

Conversation

@gatesn
Copy link
Contributor

@gatesn gatesn commented Mar 11, 2026

We are about to move more functions onto the VTable struct. To make this API nicer for users we're renaming FooVTable to Foo. Since users don't really interact with these structs at the moment, it should have a relatively small blast radius.

That said, anyone who has code like this should update it

- if let Some(foo) = array.as_opt::<FooVTable>() { ... }
+ if let Some(foo) = array.as_opt::<Foo>() { ... }

Signed-off-by: Nicholas Gates <nick@nickgates.com>
@gatesn gatesn requested a review from joseph-isaacs March 11, 2026 23:03
@gatesn gatesn enabled auto-merge (squash) March 11, 2026 23:03
@gatesn gatesn added the changelog/break A breaking API change label Mar 12, 2026
Signed-off-by: Nicholas Gates <nick@nickgates.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

changelog/break A breaking API change

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant