fix(encode): use a `for`-loop, instead of `next()` or `ipairs()`, to prevent loss of data

ThePrimeagen / harpoon

MIT License

6.25k stars 348 forks source link

fix(encode): use a `for`-loop, instead of `next()` or `ipairs()`, to prevent loss of data #573

Open kimabrandt-flx opened 2 months ago

kimabrandt-flx commented 2 months ago

This fixes the issue, when there's an empty line inside the quick-menu, which causes everything after the blank line to be forgotten (after Neovim is restarted).

This also restores the blank lines, which I'm not certain of is expected or not?!

kimabrandt-flx commented 2 months ago

There still seems to be an issue, when there are many blank items!? Some of them are just removed!? But, non-nil items seem to get restored alright!?