Stream: git-wasmtime

Topic: wasmtime / issue #1583 Cranelift: rename `PrimaryMap` and...


view this post on Zulip Wasmtime GitHub notifications bot (Oct 01 2021 at 21:11):

akirilov-arm labeled issue #1583:

The names PrimaryMap and SecondaryMap are misleading and un-descriptive. For the benefit of readers and maintainers of Cranelift, and especially newcomers, I propose we give them better names.

I suggest to rename PrimaryMap to FixedSizeVec and SecondaryMap to AutoResizeVec:

I believe that accurate naming is important, and this would be a small but important step in making Cranelift easier to understand. If there is general support I will happily prepare a patch.


Last updated: Dec 23 2024 at 12:05 UTC