Taras Tsugrii

[optim][perf] Simplify _init_group.

4

This version is more concise and avoids second lookup in case `momentum_buffer` is in the `state`.

[dynamo][perf] Avoid list copy in encode_exception_table_varint.

5

List copies are fairly expensive and in this particular case there is no advantage of doing so, since previous value is forgotten anyways.

`cargo check` is not happy with original code ``` = note: call `drop(Box::from_raw(ptr))` if you intend to drop the `Box` = note: `#[warn(unused_must_use)]` on by default help: use `let _...

sled requires Rust 1.62+ and `spin_loop` is availabe starting in 1.42.

[nit] Replace hard-coded bit count with constant.

1

This makes it clear what `8` stands for.

It's much more efficient and is also idiomatic.

`std::fs::read_to_string` removes unnecessary boilerplate.

Config file is opened but never closed which leads to a resource leak.

Reduce duplication in make_direct_map.

1

Summary: it's more concise and reduces chances of accidental out-of-sync branch issues Differential Revision: D64979108