patch 8.2.4367: calling in_vim9script() multiple times
Problem: Calling in_vim9script() multiple times.
Solution: Call it once and keep the result.
diff --git a/src/eval.c b/src/eval.c
index 261a397..91e9bbb 100644
--- a/src/eval.c
+++ b/src/eval.c
@@ -865,6 +865,7 @@
hashtab_T *ht = NULL;
int quiet = flags & GLV_QUIET;
int writing;
+ int vim9script = in_vim9script();
// Clear everything in "lp".
CLEAR_POINTER(lp);
@@ -879,7 +880,7 @@
}
// Cannot use "s:var" at the Vim9 script level. "s: type" is OK.
- if (in_vim9script() && at_script_level()
+ if (vim9script && at_script_level()
&& name[0] == 's' && name[1] == ':' && !VIM_ISWHITE(name[2]))
{
semsg(_(e_cannot_use_s_colon_in_vim9_script_str), name);
@@ -918,7 +919,7 @@
{
lp->ll_name = name;
- if (in_vim9script())
+ if (vim9script)
{
// "a: type" is declaring variable "a" with a type, not "a:".
if (p == name + 2 && p[-1] == ':')
@@ -996,7 +997,7 @@
if ((*p != '[' && *p != '.'))
return p;
- if (in_vim9script() && lval_root != NULL)
+ if (vim9script && lval_root != NULL)
{
// using local variable
lp->ll_tv = lval_root;
@@ -1018,7 +1019,7 @@
lp->ll_tv = &v->di_tv;
}
- if (in_vim9script() && (flags & GLV_NO_DECL) == 0)
+ if (vim9script && (flags & GLV_NO_DECL) == 0)
{
if (!quiet)
semsg(_(e_variable_already_declared), lp->ll_name);
@@ -1061,7 +1062,7 @@
return NULL;
}
- if (in_vim9script() && lp->ll_valtype == NULL
+ if (vim9script && lp->ll_valtype == NULL
&& v != NULL
&& lp->ll_tv == &v->di_tv
&& ht != NULL && ht == get_script_local_ht())
@@ -2612,7 +2613,7 @@
*arg = eval_next_line(evalarg_used);
else
{
- if (evaluate && in_vim9script() && !VIM_ISWHITE(p[-1]))
+ if (evaluate && vim9script && !VIM_ISWHITE(p[-1]))
{
error_white_both(p, 2);
clear_tv(rettv);
@@ -2624,7 +2625,7 @@
/*
* Get the second variable.
*/
- if (evaluate && in_vim9script() && !IS_WHITE_OR_NUL((*arg)[2]))
+ if (evaluate && vim9script && !IS_WHITE_OR_NUL((*arg)[2]))
{
error_white_both(*arg, 2);
clear_tv(rettv);
@@ -2750,7 +2751,7 @@
/*
* Get the second variable.
*/
- if (evaluate && in_vim9script() && !IS_WHITE_OR_NUL((*arg)[2]))
+ if (evaluate && vim9script && !IS_WHITE_OR_NUL((*arg)[2]))
{
error_white_both(*arg, 2);
clear_tv(rettv);
@@ -3527,7 +3528,8 @@
char_u *start_leader, *end_leader;
int ret = OK;
char_u *alias;
- static int recurse = 0;
+ static int recurse = 0;
+ int vim9script = in_vim9script();
/*
* Initialise variable so that clear_tv() can't mistake this for a
@@ -3539,7 +3541,7 @@
* Skip '!', '-' and '+' characters. They are handled later.
*/
start_leader = *arg;
- if (eval_leader(arg, in_vim9script()) == FAIL)
+ if (eval_leader(arg, vim9script) == FAIL)
return FAIL;
end_leader = *arg;
@@ -3614,7 +3616,7 @@
/*
* Dictionary: #{key: val, key: val}
*/
- case '#': if (in_vim9script())
+ case '#': if (vim9script)
{
ret = vim9_bad_comment(*arg) ? FAIL : NOTDONE;
}
@@ -3631,10 +3633,10 @@
* Lambda: {arg, arg -> expr}
* Dictionary: {'key': val, 'key': val}
*/
- case '{': if (in_vim9script())
+ case '{': if (vim9script)
ret = NOTDONE;
else
- ret = get_lambda_tv(arg, rettv, in_vim9script(), evalarg);
+ ret = get_lambda_tv(arg, rettv, vim9script, evalarg);
if (ret == NOTDONE)
ret = eval_dict(arg, rettv, evalarg, FALSE);
break;
@@ -3657,9 +3659,9 @@
case '@': ++*arg;
if (evaluate)
{
- if (in_vim9script() && IS_WHITE_OR_NUL(**arg))
+ if (vim9script && IS_WHITE_OR_NUL(**arg))
semsg(_(e_syntax_error_at_str), *arg);
- else if (in_vim9script() && !valid_yank_reg(**arg, FALSE))
+ else if (vim9script && !valid_yank_reg(**arg, FALSE))
emsg_invreg(**arg);
else
{
@@ -3677,7 +3679,7 @@
* or lambda: (arg) => expr
*/
case '(': ret = NOTDONE;
- if (in_vim9script())
+ if (vim9script)
{
ret = get_lambda_tv(arg, rettv, TRUE, evalarg);
if (ret == OK && evaluate)
@@ -3735,18 +3737,18 @@
{
int flags = evalarg == NULL ? 0 : evalarg->eval_flags;
- if (evaluate && in_vim9script() && len == 1 && *s == '_')
+ if (evaluate && vim9script && len == 1 && *s == '_')
{
emsg(_(e_cannot_use_underscore_here));
ret = FAIL;
}
- else if (evaluate && in_vim9script() && len > 2
+ else if (evaluate && vim9script && len > 2
&& s[0] == 's' && s[1] == ':')
{
semsg(_(e_cannot_use_s_colon_in_vim9_script_str), s);
ret = FAIL;
}
- else if ((in_vim9script() ? **arg : *skipwhite(*arg)) == '(')
+ else if ((vim9script ? **arg : *skipwhite(*arg)) == '(')
{
// "name(..." recursive!
*arg = skipwhite(*arg);
@@ -3757,21 +3759,19 @@
else if (evaluate)
{
// get the value of "true", "false" or a variable
- if (len == 4 && in_vim9script() && STRNCMP(s, "true", 4) == 0)
+ if (len == 4 && vim9script && STRNCMP(s, "true", 4) == 0)
{
rettv->v_type = VAR_BOOL;
rettv->vval.v_number = VVAL_TRUE;
ret = OK;
}
- else if (len == 5 && in_vim9script()
- && STRNCMP(s, "false", 5) == 0)
+ else if (len == 5 && vim9script && STRNCMP(s, "false", 5) == 0)
{
rettv->v_type = VAR_BOOL;
rettv->vval.v_number = VVAL_FALSE;
ret = OK;
}
- else if (len == 4 && in_vim9script()
- && STRNCMP(s, "null", 4) == 0)
+ else if (len == 4 && vim9script && STRNCMP(s, "null", 4) == 0)
{
rettv->v_type = VAR_SPECIAL;
rettv->vval.v_number = VVAL_NULL;
@@ -3826,6 +3826,7 @@
int error = FALSE;
varnumber_T val = 0;
vartype_T type = rettv->v_type;
+ int vim9script = in_vim9script();
#ifdef FEAT_FLOAT
float_T f = 0.0;
@@ -3836,7 +3837,7 @@
{
while (VIM_ISWHITE(end_leader[-1]))
--end_leader;
- if (in_vim9script() && end_leader[-1] == '!')
+ if (vim9script && end_leader[-1] == '!')
val = tv2bool(rettv);
else
val = tv_get_number_chk(rettv, &error);
@@ -3861,7 +3862,7 @@
#ifdef FEAT_FLOAT
if (rettv->v_type == VAR_FLOAT)
{
- if (in_vim9script())
+ if (vim9script)
{
rettv->v_type = VAR_BOOL;
val = f == 0.0 ? VVAL_TRUE : VVAL_FALSE;
@@ -3899,7 +3900,7 @@
#endif
{
clear_tv(rettv);
- if (in_vim9script())
+ if (vim9script)
rettv->v_type = type;
else
rettv->v_type = VAR_NUMBER;
@@ -4151,7 +4152,7 @@
int range = FALSE;
char_u *key = NULL;
int keylen = -1;
- int vim9 = in_vim9script();
+ int vim9script = in_vim9script();
if (check_can_index(rettv, evaluate, verbose) == FAIL)
return FAIL;
@@ -4182,7 +4183,7 @@
empty1 = TRUE;
else if (eval1(arg, &var1, evalarg) == FAIL) // recursive!
return FAIL;
- else if (vim9 && **arg == ':')
+ else if (vim9script && **arg == ':')
{
semsg(_(e_white_space_required_before_and_after_str_at_str),
":", *arg);
@@ -4195,14 +4196,14 @@
#ifdef FEAT_FLOAT
// allow for indexing with float
- if (vim9 && rettv->v_type == VAR_DICT
+ if (vim9script && rettv->v_type == VAR_DICT
&& var1.v_type == VAR_FLOAT)
{
var1.vval.v_string = typval_tostring(&var1, TRUE);
var1.v_type = VAR_STRING;
}
#endif
- if (vim9 && rettv->v_type == VAR_LIST)
+ if (vim9script && rettv->v_type == VAR_LIST)
tv_get_number_chk(&var1, &error);
else
error = tv_get_string_chk(&var1) == NULL;
@@ -4222,7 +4223,7 @@
{
range = TRUE;
++*arg;
- if (vim9 && !IS_WHITE_OR_NUL(**arg) && **arg != ']')
+ if (vim9script && !IS_WHITE_OR_NUL(**arg) && **arg != ']')
{
semsg(_(e_white_space_required_before_and_after_str_at_str),
":", *arg - 1);