Conversation
Signed-off-by: Nicholas Gates <nick@nickgates.com>
|
Warning This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite. This stack of pull requests is managed by Graphite. Learn more about stacking. |
Merging this PR will degrade performance by 21.07%
Performance Changes
Comparing Footnotes
|

As part of #6544
For third-party encodings, this moves constructors to the vtable struct and moves instance methods to an extension trait. When arrays become
Array<V: VTable>it will not be possible for third-party arrays to define instance methods without an extension trait.Note that in the future I'd like to strip the
VTablesuffix from the vtable structs, so it just becomesALP.try_new(...)instead ofALPVTable.try_new(...)