aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorzeertzjq <zeertzjq@outlook.com>2022-08-24 06:28:37 +0800
committerGitHub <noreply@github.com>2022-08-24 06:28:37 +0800
commit64d147b4719310093ca2d728ba68b23f7e0660df (patch)
treecc94abbf0970eeac320671b32dfaf2b257e0902f
parent6cc6e11929ad76a2dc5204aed95cb9ed1dafde23 (diff)
downloadrneovim-64d147b4719310093ca2d728ba68b23f7e0660df.tar.gz
rneovim-64d147b4719310093ca2d728ba68b23f7e0660df.tar.bz2
rneovim-64d147b4719310093ca2d728ba68b23f7e0660df.zip
vim-patch:9.0.0248: duplicate code in finding a script in the execution stack (#19917)
Problem: Duplicate code in finding a script in the execution stack. Solution: Reduce duplicate code. (closes vim/vim#10961) https://github.com/vim/vim/commit/a247142ae45308087b25f91c8af48399c8ac2943
-rw-r--r--src/nvim/runtime.c24
1 files changed, 7 insertions, 17 deletions
diff --git a/src/nvim/runtime.c b/src/nvim/runtime.c
index 038aad12a4..a04ae271e1 100644
--- a/src/nvim/runtime.c
+++ b/src/nvim/runtime.c
@@ -116,25 +116,15 @@ char *estack_sfile(estack_arg_T which)
// where it is defined, at script level the current script path is returned
// instead.
if (which == ESTACK_SCRIPT) {
- assert(entry == ((estack_T *)exestack.ga_data) + exestack.ga_len - 1);
// Walk the stack backwards, starting from the current frame.
for (int idx = exestack.ga_len - 1; idx >= 0; idx--, entry--) {
- if (entry->es_type == ETYPE_UFUNC) {
- const sctx_T *const def_ctx = &entry->es_info.ufunc->uf_script_ctx;
-
- if (def_ctx->sc_sid > 0) {
- return xstrdup((char *)(SCRIPT_ITEM(def_ctx->sc_sid).sn_name));
- } else {
- return NULL;
- }
- } else if (entry->es_type == ETYPE_AUCMD) {
- const sctx_T *const def_ctx = &entry->es_info.aucmd->script_ctx;
-
- if (def_ctx->sc_sid > 0) {
- return xstrdup((char *)(SCRIPT_ITEM(def_ctx->sc_sid).sn_name));
- } else {
- return NULL;
- }
+ if (entry->es_type == ETYPE_UFUNC || entry->es_type == ETYPE_AUCMD) {
+ const sctx_T *const def_ctx = (entry->es_type == ETYPE_UFUNC
+ ? &entry->es_info.ufunc->uf_script_ctx
+ : &entry->es_info.aucmd->script_ctx);
+ return def_ctx->sc_sid > 0
+ ? xstrdup((char *)(SCRIPT_ITEM(def_ctx->sc_sid).sn_name))
+ : NULL;
} else if (entry->es_type == ETYPE_SCRIPT) {
return xstrdup(entry->es_name);
}