0
0
mirror of https://github.com/vim/vim.git synced 2025-09-24 03:44:06 -04:00

patch 9.0.1483: += operator does not work on class member

Problem:    += operator does not work on class member.
Solution:   Do not skip as if "this." was used. (Christian Brabandt,
            closes #12263)
This commit is contained in:
Bram Moolenaar
2023-04-24 17:15:25 +01:00
parent 2354b66ac5
commit 22363c6994
3 changed files with 31 additions and 7 deletions

View File

@@ -401,6 +401,13 @@ def Test_assignment_with_operator()
var f = Foo.new(3) var f = Foo.new(3)
f.Add(17) f.Add(17)
assert_equal(20, f.x) assert_equal(20, f.x)
def AddToFoo(obj: Foo)
obj.x += 3
enddef
AddToFoo(f)
assert_equal(23, f.x)
END END
v9.CheckScriptSuccess(lines) v9.CheckScriptSuccess(lines)
enddef enddef

View File

@@ -695,6 +695,8 @@ static char *(features[]) =
static int included_patches[] = static int included_patches[] =
{ /* Add new patch number below this line */ { /* Add new patch number below this line */
/**/
1483,
/**/ /**/
1482, 1482,
/**/ /**/

View File

@@ -2065,16 +2065,31 @@ compile_load_lhs_with_index(lhs_T *lhs, char_u *var_start, cctx_T *cctx)
{ {
if (lhs->lhs_type->tt_type == VAR_OBJECT) if (lhs->lhs_type->tt_type == VAR_OBJECT)
{ {
// "this.value": load "this" object and get the value at index // "this.value": load "this" object and get the value at index for an
// for an object or class member get the type of the member // object or class member get the type of the member.
// Also for "obj.value".
char_u *dot = vim_strchr(var_start, '.');
if (dot == NULL)
return FAIL;
class_T *cl = lhs->lhs_type->tt_class; class_T *cl = lhs->lhs_type->tt_class;
type_T *type = class_member_type(cl, var_start + 5, type_T *type = class_member_type(cl, dot + 1,
lhs->lhs_end, &lhs->lhs_member_idx); lhs->lhs_end, &lhs->lhs_member_idx);
if (lhs->lhs_member_idx < 0) if (lhs->lhs_member_idx < 0)
return FAIL; return FAIL;
if (dot - var_start == 4 && STRNCMP(var_start, "this", 4) == 0)
{
// load "this"
if (generate_LOAD(cctx, ISN_LOAD, 0, NULL, lhs->lhs_type) == FAIL) if (generate_LOAD(cctx, ISN_LOAD, 0, NULL, lhs->lhs_type) == FAIL)
return FAIL; return FAIL;
}
else
{
// load object variable or argument
if (compile_load_lhs(lhs, var_start, lhs->lhs_type, cctx) == FAIL)
return FAIL;
}
if (cl->class_flags & CLASS_INTERFACE) if (cl->class_flags & CLASS_INTERFACE)
return generate_GET_ITF_MEMBER(cctx, cl, lhs->lhs_member_idx, type); return generate_GET_ITF_MEMBER(cctx, cl, lhs->lhs_member_idx, type);
return generate_GET_OBJ_MEMBER(cctx, lhs->lhs_member_idx, type); return generate_GET_OBJ_MEMBER(cctx, lhs->lhs_member_idx, type);
@@ -2169,7 +2184,7 @@ compile_assign_unlet(
if (cctx->ctx_skip == SKIP_YES) if (cctx->ctx_skip == SKIP_YES)
return OK; return OK;
// Load the dict or list. On the stack we then have: // Load the dict, list or object. On the stack we then have:
// - value (for assignment, not for :unlet) // - value (for assignment, not for :unlet)
// - index // - index
// - for [a : b] second index // - for [a : b] second index
@@ -2731,7 +2746,7 @@ compile_assignment(
if (lhs.lhs_has_index) if (lhs.lhs_has_index)
{ {
// Use the info in "lhs" to store the value at the index in the // Use the info in "lhs" to store the value at the index in the
// list or dict. // list, dict or object.
if (compile_assign_unlet(var_start, &lhs, TRUE, rhs_type, cctx) if (compile_assign_unlet(var_start, &lhs, TRUE, rhs_type, cctx)
== FAIL) == FAIL)
{ {