diff options
-rw-r--r-- | runtime/doc/api.txt | 4 | ||||
-rw-r--r-- | runtime/doc/news.txt | 3 | ||||
-rw-r--r-- | src/nvim/api/keysets.lua | 1 | ||||
-rw-r--r-- | src/nvim/api/options.c | 72 | ||||
-rw-r--r-- | src/nvim/edit.c | 4 | ||||
-rw-r--r-- | src/nvim/ex_docmd.c | 8 | ||||
-rw-r--r-- | src/nvim/ex_getln.c | 3 | ||||
-rw-r--r-- | src/nvim/globals.h | 2 | ||||
-rw-r--r-- | src/nvim/mapping.c | 6 | ||||
-rw-r--r-- | src/nvim/undo.c | 3 |
10 files changed, 95 insertions, 11 deletions
diff --git a/runtime/doc/api.txt b/runtime/doc/api.txt index b8590026c8..04825381ff 100644 --- a/runtime/doc/api.txt +++ b/runtime/doc/api.txt @@ -1969,6 +1969,10 @@ nvim_get_option_value({name}, {*opts}) *nvim_get_option_value()* • win: |window-ID|. Used for getting window local options. • buf: Buffer number. Used for getting buffer local options. Implies {scope} is "local". + • filetype: |filetype|. Used to get the default option for a + specific filetype. Cannot be used with any other option. + Note: this is expensive, it is recommended to cache this + value. Return: ~ Option value diff --git a/runtime/doc/news.txt b/runtime/doc/news.txt index 64cf5d78ba..b7d5694802 100644 --- a/runtime/doc/news.txt +++ b/runtime/doc/news.txt @@ -210,6 +210,9 @@ The following new APIs or features were added. https://tree-sitter.github.io/tree-sitter/syntax-highlighting#language-injection . Support for the previous format will be removed in a future release. +• |nvim_get_option_value()| now has a `filetype` option so it can return the + default option for a specific filetype. + ============================================================================== CHANGED FEATURES *news-changes* diff --git a/src/nvim/api/keysets.lua b/src/nvim/api/keysets.lua index 30dcef6127..2c7532bd27 100644 --- a/src/nvim/api/keysets.lua +++ b/src/nvim/api/keysets.lua @@ -102,6 +102,7 @@ return { "scope"; "win"; "buf"; + "filetype"; }}; { 'highlight', { "bold"; diff --git a/src/nvim/api/options.c b/src/nvim/api/options.c index 2d9ffcba06..a0351cc6cd 100644 --- a/src/nvim/api/options.c +++ b/src/nvim/api/options.c @@ -24,7 +24,7 @@ #endif static int validate_option_value_args(Dict(option) *opts, int *scope, int *opt_type, void **from, - Error *err) + char **filetype, Error *err) { if (HAS_KEY(opts->scope)) { VALIDATE_T("scope", kObjectTypeString, opts->scope.type, { @@ -44,6 +44,14 @@ static int validate_option_value_args(Dict(option) *opts, int *scope, int *opt_t *opt_type = SREQ_GLOBAL; + if (filetype != NULL && HAS_KEY(opts->filetype)) { + VALIDATE_T("scope", kObjectTypeString, opts->filetype.type, { + return FAIL; + }); + + *filetype = opts->filetype.data.string.data; + } + if (HAS_KEY(opts->win)) { VALIDATE_T("win", kObjectTypeInteger, opts->win.type, { return FAIL; @@ -69,10 +77,17 @@ static int validate_option_value_args(Dict(option) *opts, int *scope, int *opt_t } } + VALIDATE((!HAS_KEY(opts->filetype) + || !(HAS_KEY(opts->buf) || HAS_KEY(opts->scope) || HAS_KEY(opts->win))), + "%s", "cannot use 'filetype' with 'scope', 'buf' or 'win'", { + return FAIL; + }); + VALIDATE((!HAS_KEY(opts->scope) || !HAS_KEY(opts->buf)), "%s", "cannot use both 'scope' and 'buf'", { return FAIL; }); + VALIDATE((!HAS_KEY(opts->win) || !HAS_KEY(opts->buf)), "%s", "cannot use both 'buf' and 'win'", { return FAIL; }); @@ -80,6 +95,30 @@ static int validate_option_value_args(Dict(option) *opts, int *scope, int *opt_t return OK; } +/// Create a dummy buffer and run the FileType autocmd on it. +static buf_T *do_ft_buf(char *filetype, aco_save_T *aco, Error *err) +{ + if (filetype == NULL) { + return NULL; + } + + // Allocate a buffer without putting it in the buffer list. + buf_T *ftbuf = buflist_new(NULL, NULL, 1, BLN_DUMMY); + if (ftbuf == NULL) { + api_set_error(err, kErrorTypeException, "Could not create internal buffer"); + return NULL; + } + + // Set curwin/curbuf to buf and save a few things. + aucmd_prepbuf(aco, ftbuf); + + ftbuf->b_p_ft = xstrdup(filetype); + + apply_autocmds(EVENT_FILETYPE, ftbuf->b_p_ft, ftbuf->b_fname, true, ftbuf); + + return ftbuf; +} + /// Gets the value of an option. The behavior of this function matches that of /// |:set|: the local value of an option is returned if it exists; otherwise, /// the global value is returned. Local values always correspond to the current @@ -92,6 +131,10 @@ static int validate_option_value_args(Dict(option) *opts, int *scope, int *opt_t /// - win: |window-ID|. Used for getting window local options. /// - buf: Buffer number. Used for getting buffer local options. /// Implies {scope} is "local". +/// - filetype: |filetype|. Used to get the default option for a +/// specific filetype. Cannot be used with any other option. +/// Note: this is expensive, it is recommended to cache this +/// value. /// @param[out] err Error details, if any /// @return Option value Object nvim_get_option_value(String name, Dict(option) *opts, Error *err) @@ -102,14 +145,37 @@ Object nvim_get_option_value(String name, Dict(option) *opts, Error *err) int scope = 0; int opt_type = SREQ_GLOBAL; void *from = NULL; - if (!validate_option_value_args(opts, &scope, &opt_type, &from, err)) { + char *filetype = NULL; + + if (!validate_option_value_args(opts, &scope, &opt_type, &from, &filetype, err)) { + return rv; + } + + aco_save_T aco; + + buf_T *ftbuf = do_ft_buf(filetype, &aco, err); + if (ERROR_SET(err)) { return rv; } + if (ftbuf != NULL) { + assert(!from); + from = ftbuf; + } + long numval = 0; char *stringval = NULL; getoption_T result = access_option_value_for(name.data, &numval, &stringval, scope, opt_type, from, true, err); + + if (ftbuf != NULL) { + // restore curwin/curbuf and a few other things + aucmd_restbuf(&aco); + + assert(curbuf != ftbuf); // safety check + wipe_buffer(ftbuf, false); + } + if (ERROR_SET(err)) { return rv; } @@ -164,7 +230,7 @@ void nvim_set_option_value(uint64_t channel_id, String name, Object value, Dict( int scope = 0; int opt_type = SREQ_GLOBAL; void *to = NULL; - if (!validate_option_value_args(opts, &scope, &opt_type, &to, err)) { + if (!validate_option_value_args(opts, &scope, &opt_type, &to, NULL, err)) { return; } diff --git a/src/nvim/edit.c b/src/nvim/edit.c index 66f6f06062..7bc71c1477 100644 --- a/src/nvim/edit.c +++ b/src/nvim/edit.c @@ -1245,7 +1245,9 @@ bool edit(int cmdchar, bool startln, long count) // Don't allow changes in the buffer while editing the cmdline. The // caller of getcmdline() may get confused. // Don't allow recursive insert mode when busy with completion. - if (textlock != 0 || ins_compl_active() || compl_busy || pum_visible()) { + // Allow in dummy buffers since they are only used internally + if (textlock != 0 || ins_compl_active() || compl_busy || pum_visible() + || expr_map_locked()) { emsg(_(e_textlock)); return false; } diff --git a/src/nvim/ex_docmd.c b/src/nvim/ex_docmd.c index 061a8e699e..7b94e3184b 100644 --- a/src/nvim/ex_docmd.c +++ b/src/nvim/ex_docmd.c @@ -6326,6 +6326,11 @@ void restore_current_state(save_state_T *sst) ui_cursor_shape(); // may show different cursor shape } +bool expr_map_locked(void) +{ + return expr_map_lock > 0 && !(curbuf->b_flags & BF_DUMMY); +} + /// ":normal[!] {commands}": Execute normal mode commands. static void ex_normal(exarg_T *eap) { @@ -6335,10 +6340,11 @@ static void ex_normal(exarg_T *eap) } char *arg = NULL; - if (ex_normal_lock > 0) { + if (expr_map_locked()) { emsg(_(e_secure)); return; } + if (ex_normal_busy >= p_mmd) { emsg(_("E192: Recursive use of :normal too deep")); return; diff --git a/src/nvim/ex_getln.c b/src/nvim/ex_getln.c index 0f823383a4..5112505eda 100644 --- a/src/nvim/ex_getln.c +++ b/src/nvim/ex_getln.c @@ -2740,6 +2740,9 @@ bool text_locked(void) if (cmdwin_type != 0) { return true; } + if (expr_map_locked()) { + return true; + } return textlock != 0; } diff --git a/src/nvim/globals.h b/src/nvim/globals.h index f3bfbdabf6..816471d7c1 100644 --- a/src/nvim/globals.h +++ b/src/nvim/globals.h @@ -715,7 +715,7 @@ EXTERN typebuf_T typebuf INIT(= { NULL, NULL, 0, 0, 0, 0, 0, 0, 0 }); EXTERN bool typebuf_was_empty INIT(= false); EXTERN int ex_normal_busy INIT(= 0); // recursiveness of ex_normal() -EXTERN int ex_normal_lock INIT(= 0); // forbid use of ex_normal() +EXTERN int expr_map_lock INIT(= 0); // running expr mapping, prevent use of ex_normal() and text changes EXTERN int ignore_script INIT(= false); // ignore script input EXTERN int stop_insert_mode; // for ":stopinsert" EXTERN bool KeyTyped; // true if user typed current char diff --git a/src/nvim/mapping.c b/src/nvim/mapping.c index 3c4ce0ee2b..5cedc5e97d 100644 --- a/src/nvim/mapping.c +++ b/src/nvim/mapping.c @@ -1615,8 +1615,7 @@ char *eval_map_expr(mapblock_T *mp, int c) // Forbid changing text or using ":normal" to avoid most of the bad side // effects. Also restore the cursor position. - textlock++; - ex_normal_lock++; + expr_map_lock++; set_vim_var_char(c); // set v:char to the typed character const pos_T save_cursor = curwin->w_cursor; const int save_msg_col = msg_col; @@ -1637,8 +1636,7 @@ char *eval_map_expr(mapblock_T *mp, int c) p = eval_to_string(expr, NULL, false); xfree(expr); } - textlock--; - ex_normal_lock--; + expr_map_lock--; curwin->w_cursor = save_cursor; msg_col = save_msg_col; msg_row = save_msg_row; diff --git a/src/nvim/undo.c b/src/nvim/undo.c index 3902d9c2aa..ad33ba4667 100644 --- a/src/nvim/undo.c +++ b/src/nvim/undo.c @@ -96,6 +96,7 @@ #include "nvim/eval/typval.h" #include "nvim/eval/typval_defs.h" #include "nvim/ex_cmds_defs.h" +#include "nvim/ex_docmd.h" #include "nvim/ex_getln.h" #include "nvim/extmark.h" #include "nvim/fileio.h" @@ -300,7 +301,7 @@ bool undo_allowed(buf_T *buf) // Don't allow changes in the buffer while editing the cmdline. The // caller of getcmdline() may get confused. - if (textlock != 0) { + if (textlock != 0 || expr_map_locked()) { emsg(_(e_textlock)); return false; } |