diff options
| author | Lewis Russell <lewis6991@gmail.com> | 2024-01-02 15:47:55 +0000 |
|---|---|---|
| committer | Lewis Russell <me@lewisr.dev> | 2024-01-03 19:17:52 +0000 |
| commit | 3734519e3b4ba1bf19ca772104170b0ef776be46 (patch) | |
| tree | 33ef73759201265fe99b5c0499d7f792a855aef0 /runtime/lua/vim/keymap.lua | |
| parent | a064ed622927b4c5e30165abbe54db841359c71f (diff) | |
| download | rneovim-3734519e3b4ba1bf19ca772104170b0ef776be46.tar.gz rneovim-3734519e3b4ba1bf19ca772104170b0ef776be46.tar.bz2 rneovim-3734519e3b4ba1bf19ca772104170b0ef776be46.zip | |
feat(lua): add noref to deepcopy
Problem:
Currently `deepcopy` hashes every single tables it copies so it can be
reused. For tables of mostly unique items that are non recursive, this
hashing is unnecessarily expensive
Solution:
Port the `noref` argument from Vimscripts `deepcopy()`.
The below benchmark demonstrates the results for two extreme cases of
tables of different sizes. One table that uses the same table lots of
times and one with all unique tables.
| test | `noref=false` (ms) | `noref=true` (ms) |
| -------------------- | ------------------ | ----------------- |
| unique tables (50) | 6.59 | 2.62 |
| shared tables (50) | 3.24 | 6.40 |
| unique tables (2000) | 23381.48 | 2884.53 |
| shared tables (2000) | 3505.54 | 14038.80 |
The results are basically the inverse of each other where `noref` is
much more performance on tables with unique fields, and `not noref` is
more performant on tables that reuse fields.
Diffstat (limited to 'runtime/lua/vim/keymap.lua')
| -rw-r--r-- | runtime/lua/vim/keymap.lua | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/runtime/lua/vim/keymap.lua b/runtime/lua/vim/keymap.lua index bdea95f9ab..8e4e123fe0 100644 --- a/runtime/lua/vim/keymap.lua +++ b/runtime/lua/vim/keymap.lua @@ -44,7 +44,7 @@ function keymap.set(mode, lhs, rhs, opts) opts = { opts, 't', true }, }) - opts = vim.deepcopy(opts or {}) + opts = vim.deepcopy(opts or {}, true) ---@cast mode string[] mode = type(mode) == 'string' and { mode } or mode |