diff options
author | Bram Moolenaar <Bram@vim.org> | 2016-08-27 21:25:44 +0200 |
---|---|---|
committer | Bram Moolenaar <Bram@vim.org> | 2016-08-27 21:25:44 +0200 |
commit | 3056735ae8a366aa7fcb51872520895251858637 (patch) | |
tree | 329de3dbeea7c06972c0ea41929b636f9ef47540 /src/evalfunc.c | |
parent | 9f8187c335b4fb07be9095dfdd0fc52670ba3c3f (diff) |
patch 7.4.2273v7.4.2273
Problem: getwininfo() and getbufinfo() are inefficient.
Solution: Do not make a copy of all window/buffer-local options. Make it
possible to get them with gettabwinvar() or getbufvar().
Diffstat (limited to 'src/evalfunc.c')
-rw-r--r-- | src/evalfunc.c | 32 |
1 files changed, 17 insertions, 15 deletions
diff --git a/src/evalfunc.c b/src/evalfunc.c index 1a2ea8621c..7ccf524e61 100644 --- a/src/evalfunc.c +++ b/src/evalfunc.c @@ -3921,7 +3921,6 @@ get_buffer_signs(buf_T *buf, list_T *l) get_buffer_info(buf_T *buf) { dict_T *dict; - dict_T *opts; tabpage_T *tp; win_T *wp; list_T *windows; @@ -3945,11 +3944,6 @@ get_buffer_info(buf_T *buf) /* Get a reference to buffer variables */ dict_add_dict(dict, "variables", buf->b_vars); - /* Copy buffer options */ - opts = get_winbuf_options(TRUE); - if (opts != NULL) - dict_add_dict(dict, "options", opts); - /* List of windows displaying this buffer */ windows = list_alloc(); if (windows != NULL) @@ -4156,9 +4150,23 @@ f_getbufvar(typval_T *argvars, typval_T *rettv) save_curbuf = curbuf; curbuf = buf; - if (*varname == '&') /* buffer-local-option */ + if (*varname == '&') { - if (get_option_tv(&varname, rettv, TRUE) == OK) + if (varname[1] == NUL) + { + /* get all buffer-local options in a dict */ + dict_T *opts = get_winbuf_options(TRUE); + + if (opts != NULL) + { + rettv->v_type = VAR_DICT; + rettv->vval.v_dict = opts; + ++opts->dv_refcount; + done = TRUE; + } + } + else if (get_option_tv(&varname, rettv, TRUE) == OK) + /* buffer-local-option */ done = TRUE; } else if (STRCMP(varname, "changedtick") == 0) @@ -5112,7 +5120,6 @@ f_gettabwinvar(typval_T *argvars, typval_T *rettv) get_win_info(win_T *wp, short tpnr, short winnr) { dict_T *dict; - dict_T *opts; dict = dict_alloc(); if (dict == NULL) @@ -5131,14 +5138,9 @@ get_win_info(win_T *wp, short tpnr, short winnr) (bt_quickfix(wp->w_buffer) && wp->w_llist_ref != NULL), NULL); #endif - /* Make a reference to window variables */ + /* Add a reference to window variables */ dict_add_dict(dict, "variables", wp->w_vars); - /* Copy window options */ - opts = get_winbuf_options(FALSE); - if (opts != NULL) - dict_add_dict(dict, "options", opts); - return dict; } #endif |