diff options
author | Yegappan Lakshmanan <yegappan@yahoo.com> | 2023-09-15 20:14:55 +0200 |
---|---|---|
committer | Christian Brabandt <cb@256bit.org> | 2023-09-15 20:14:55 +0200 |
commit | c30a90d9b2c029f794cea502f6b824f71e4876dd (patch) | |
tree | 53c789bae67beb6fe099686fcc6894b2c6fdb7b8 /src/vim9execute.c | |
parent | 35928ee8f80ea721e92bb856c8ecde2cced46bb9 (diff) |
patch 9.0.1898: Vim9: restrict access to static varsv9.0.1898
Problem: Vim9: restrict access to static vars and methods
Solution: Class members are accesible only from the class where they are
defined.
Based on the #13004 discussion, the following changes are made:
1) Static variables and methods are accessible only using the class
name and inside the class where they are defined.
2) Static variables and methods can be used without the class name in
the class where they are defined.
3) Static variables of a super class are not copied to the sub class.
4) A sub class can declare a class variable with the same name as the
super class.
5) When a method or member is found during compilation, use more
specific error messages.
This aligns the Vim9 class variable/method implementation with the Dart
implementation.
Also while at it, ignore duplicate class and object methods.
The access level of an object method can however be changed in a
subclass.
For the tests, use the new CheckSourceFailure() function instead of the
CheckScriptFailure() function in the tests.
closes: #13086
Signed-off-by: Christian Brabandt <cb@256bit.org>
Co-authored-by: Yegappan Lakshmanan <yegappan@yahoo.com>
Diffstat (limited to 'src/vim9execute.c')
-rw-r--r-- | src/vim9execute.c | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/src/vim9execute.c b/src/vim9execute.c index f8e2676410..7c61ff418e 100644 --- a/src/vim9execute.c +++ b/src/vim9execute.c @@ -2172,8 +2172,7 @@ execute_storeindex(isn_T *iptr, ectx_T *ectx) { if (*member == '_') { - semsg(_(e_cannot_access_private_member_str), - m->ocm_name); + semsg(_(e_cannot_access_private_member_str), m->ocm_name); status = FAIL; } @@ -2181,8 +2180,7 @@ execute_storeindex(isn_T *iptr, ectx_T *ectx) } else { - semsg(_(e_member_not_found_on_object_str_str), - cl->class_name, member); + member_not_found_msg(cl, VAR_OBJECT, member, 0); status = FAIL; } } |