aniani/vim
0
0
Fork 1
mirror of https://github.com/vim/vim.git synced 2025-10-05 05:34:07 -04:00
Code Activity

patch 9.0.2085: Vim9: abstract can be used in interface

Browse Source 
Problem:  Vim9: abstract can be used in interface
Solution: Disallow the use of abstract in an interface

fixes: #13456
closes: #13464

Signed-off-by: Yegappan Lakshmanan <yegappan@yahoo.com>
Signed-off-by: Christian Brabandt <cb@256bit.org>
This commit is contained in:
Yegappan Lakshmanan
2023-11-02 20:57:32 +01:00
committed by Christian Brabandt
parent ef9e3f8924
commit 2b358adde0
4 changed files with 44 additions and 20 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
src/errors.h
View File

@@ -3560,8 +3560,10 @@ EXTERN char e_using_typealias_as_string[]
INIT(= N_("E1402: Using type alias \"%s\" as a String")); INIT(= N_("E1402: Using type alias \"%s\" as a String"));
EXTERN char e_using_typealias_as_value[] EXTERN char e_using_typealias_as_value[]
INIT(= N_("E1403: Type alias \"%s\" cannot be used as a value")); INIT(= N_("E1403: Type alias \"%s\" cannot be used as a value"));
EXTERN char e_abstract_cannot_be_used_in_interface[]
INIT(= N_("E1404: Abstract cannot be used in an interface"));
#endif #endif
// E1404 - E1499 unused (reserved for Vim9 class support) // E1405 - E1499 unused (reserved for Vim9 class support)
EXTERN char e_cannot_mix_positional_and_non_positional_str[] EXTERN char e_cannot_mix_positional_and_non_positional_str[]
INIT(= N_("E1500: Cannot mix positional and non-positional arguments: %s")); INIT(= N_("E1500: Cannot mix positional and non-positional arguments: %s"));
EXTERN char e_fmt_arg_nr_unused_str[] EXTERN char e_fmt_arg_nr_unused_str[]

21
src/testdir/test_vim9_class.vim
View File

@@ -5567,7 +5567,26 @@ def Test_abstract_method()
enddef enddef
endclass endclass
END END
v9.CheckSourceSuccess(lines) v9.CheckSourceFailure(lines, 'E1404: Abstract cannot be used in an interface', 3)
# Use abstract static method in an interface
lines =<< trim END
vim9script
interface A
abstract static def Foo()
enddef
endinterface
END
v9.CheckSourceFailure(lines, 'E1404: Abstract cannot be used in an interface', 3)
# Use abstract static variable in an interface
lines =<< trim END
vim9script
interface A
abstract static foo: number = 10
endinterface
END
v9.CheckSourceFailure(lines, 'E1404: Abstract cannot be used in an interface', 3)
# Abbreviate the "abstract" keyword # Abbreviate the "abstract" keyword
lines =<< trim END lines =<< trim END

2
src/version.c
View File

@@ -704,6 +704,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 */
/**/
2085,
/**/ /**/
2084, 2084,
/**/ /**/

23
src/vim9class.c
View File

@@ -1557,12 +1557,20 @@ early_ret:
break; break;
} }
if (!is_class)
// ignore "abstract" in an interface (as all the methods in an
// interface are abstract.
p = skipwhite(pa + 8); p = skipwhite(pa + 8);
else if (STRNCMP(p, "def", 3) != 0 && STRNCMP(p, "static", 6) != 0)
{ {
emsg(_(e_abstract_must_be_followed_by_def_or_static));
break;
}
if (!is_class)
{
// "abstract" not supported in an interface
emsg(_(e_abstract_cannot_be_used_in_interface));
break;
}
if (!is_abstract) if (!is_abstract)
{ {
semsg(_(e_abstract_method_in_concrete_class), pa); semsg(_(e_abstract_method_in_concrete_class), pa);
@@ -1570,13 +1578,6 @@ early_ret:
} }
abstract_method = TRUE; abstract_method = TRUE;
p = skipwhite(pa + 8);
if (STRNCMP(p, "def", 3) != 0)
{
emsg(_(e_abstract_must_be_followed_by_def));
break;
}
}
} }
int has_static = FALSE; int has_static = FALSE;