JustForFun88

Move from `String` to `VoltID`

5

Small, preliminary, yet rather cosmetic improvements that improve the readability of the code and the detection of potential errors (#1881). The main improvement here is that now `Volts` is identified...

Removing unnecessary allocations and more

5

This pull request does: - removes a lot of unnecessary memory allocations. For example, previously, multiple allocations of vectors were performed in a loop to store a single character (function...

Remove cloning of the `colors` vector in `lapce_ui::settings::ThemeSection::update_items`

1

We only need to clone the contents of the vector, not the vector itself.

Simplifying `lapce_ui::settings::ThemeSection::new`

1

This pull request simplifies the `lapce_ui::settings::ThemeSection::new` function by removing the allocation of two intermediate vectors (one for `Vec`, the second inside `Itertools::sorted` method). Also, since the colors are just `String`s...

This PR adds a reduce methods to the array structure. This method is very useful if you want to apply a function to all elements of an array. Using `fold`...

Remove unnecessary `add` in `clone_from_impl`

1

The `guard.0 = index + 1` in the implementation of `clone_from _impl` is completely unnecessary, since an invalid index can be set in advance.

Fix `HashSet::get_or_insert_with`

4

Fix #399. Tried to do it without additional overhead. Unless there are additional comparisons.

Move `RawTableInner` and `TableLayout` to separate module

3

All methods of the RawTableInner structure assume one important detail that is not described in the documentation: the `ctrl` and `bucket_mask` fields must remain unchanged throughout the life of the...

Initial implementation of `try_get_many`

1

Attempt to implement `get_many_mut` with a variable number of keys, as requested in #332. cc @Ten0