asgardius/virtualx-engine
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

GDScript: Properly catch error when missing index in subscript

Browse source
This commit is contained in:
George Marques 2021-09-17 14:31:51 -03:00
parent b8fdeb6467
commit 651319de11
No known key found for this signature in database
GPG key ID: 046BD46A3201E43D
3 changed files with 9 additions and 0 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
modules/gdscript/gdscript_parser.cpp
View file

@ -2592,6 +2592,10 @@ GDScriptParser::ExpressionNode *GDScriptParser::parse_subscript(ExpressionNode *
subscript->base = p_previous_operand;
subscript->index = parse_expression(false);
if (subscript->index == nullptr) {
push_error(R"(Expected expression after "[".)");
}
pop_multiline();
consume(GDScriptTokenizer::Token::BRACKET_CLOSE, R"(Expected "]" after subscription index.)");

3
modules/gdscript/tests/scripts/parser/errors/subscript_without_index.gd Normal file
View file

@ -0,0 +1,3 @@
func test():
var array = [1, 2, 3]
array[] = 4

2
modules/gdscript/tests/scripts/parser/errors/subscript_without_index.out Normal file
View file

@ -0,0 +1,2 @@
GDTEST_PARSER_ERROR
Expected expression after "[".