IntenseFinance/neovim
3
0
Fork 0
mirror of https://github.com/neovim/neovim.git synced 2025-02-25 18:55:25 -06:00
Code Issues Packages Projects Releases Wiki Activity

vim-patch:8.2.1210: using ht_used when looping through a hashtab is less reliable

Browse Source 
Problem:    Using ht_used when looping through a hashtab is less reliable.
Solution:   Use ht_changed in a few more places.

1f22cc5cdb

Co-authored-by: Bram Moolenaar <Bram@vim.org>
This commit is contained in:
zeertzjq 2022-12-17 08:50:17 +08:00
parent 73a6000120
commit b969844398
1 changed files with 9 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

16
src/nvim/eval/userfunc.c
View File

@ -1297,7 +1297,7 @@ void free_all_functions(void)
ufunc_T *fp; ufunc_T *fp;
uint64_t skipped = 0; uint64_t skipped = 0;
uint64_t todo = 1; uint64_t todo = 1;
uint64_t used; int changed;
// Clean up the current_funccal chain and the funccal stack. // Clean up the current_funccal chain and the funccal stack.
while (current_funccal != NULL) { while (current_funccal != NULL) {
@ -1321,9 +1321,9 @@ void free_all_functions(void)
if (func_name_refcount((char_u *)fp->uf_name)) { if (func_name_refcount((char_u *)fp->uf_name)) {
skipped++; skipped++;
} else { } else {
used = func_hashtab.ht_used; changed = func_hashtab.ht_changed;
func_clear(fp, true); func_clear(fp, true);
if (used != func_hashtab.ht_used) { if (changed != func_hashtab.ht_changed) {
skipped = 0; skipped = 0;
break; break;
} }
@ -1993,8 +1993,8 @@ char *save_function_name(char **name, bool skip, int flags, funcdict_T *fudi)
/// Otherwise functions matching "regmatch". /// Otherwise functions matching "regmatch".
static void list_functions(regmatch_T *regmatch) static void list_functions(regmatch_T *regmatch)
{ {
const size_t used = func_hashtab.ht_used; const int changed = func_hashtab.ht_changed;
size_t todo = used; size_t todo = func_hashtab.ht_used;
const hashitem_T *const ht_array = func_hashtab.ht_array; const hashitem_T *const ht_array = func_hashtab.ht_array;
for (const hashitem_T *hi = ht_array; todo > 0 && !got_int; hi++) { for (const hashitem_T *hi = ht_array; todo > 0 && !got_int; hi++) {
@ -2008,7 +2008,7 @@ static void list_functions(regmatch_T *regmatch)
: (!isdigit(*fp->uf_name) : (!isdigit(*fp->uf_name)
&& vim_regexec(regmatch, (char *)fp->uf_name, 0)))) { && vim_regexec(regmatch, (char *)fp->uf_name, 0)))) {
list_func_head(fp, false, false); list_func_head(fp, false, false);
if (used != func_hashtab.ht_used || ht_array != func_hashtab.ht_array) { if (changed != func_hashtab.ht_changed) {
emsg(_("E454: function list was modified")); emsg(_("E454: function list was modified"));
return; return;
} }
@ -2730,15 +2730,17 @@ bool function_exists(const char *const name, bool no_deref)
char *get_user_func_name(expand_T *xp, int idx) char *get_user_func_name(expand_T *xp, int idx)
{ {
static size_t done; static size_t done;
static int changed;
static hashitem_T *hi; static hashitem_T *hi;
ufunc_T *fp; ufunc_T *fp;
if (idx == 0) { if (idx == 0) {
done = 0; done = 0;
hi = func_hashtab.ht_array; hi = func_hashtab.ht_array;
changed = func_hashtab.ht_changed;
} }
assert(hi); assert(hi);
if (done < func_hashtab.ht_used) { if (changed == func_hashtab.ht_changed && done < func_hashtab.ht_used) {
if (done++ > 0) { if (done++ > 0) {
hi++; hi++;
} }