gaming/godot
gaming/godot
1
0
Fork 0
mirror of https://github.com/godotengine/godot synced 2024-10-14 10:33:49 +00:00
Code Issues Packages Projects Releases Wiki Activity

GDScript: Add some checks for @tool and @icon

Browse source
This commit is contained in:
Danil Alexeev 2023-04-14 20:45:49 +03:00
parent 27253f3eb2
commit dbc3c82043
No known key found for this signature in database
GPG key ID: 124453E157DA8DC7
5 changed files with 30 additions and 0 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

16
modules/gdscript/gdscript_parser.cpp
View file

@ -3686,6 +3686,12 @@ bool GDScriptParser::validate_annotation_arguments(AnnotationNode *p_annotation)
}
bool GDScriptParser::tool_annotation(const AnnotationNode *p_annotation, Node *p_node) {
#ifdef DEBUG_ENABLED
if (this->_is_tool) {
push_error(R"("@tool" annotation can only be used once.)", p_annotation);
return false;
}
#endif // DEBUG_ENABLED
this->_is_tool = true;
return true;
}
@ -3694,6 +3700,16 @@ bool GDScriptParser::icon_annotation(const AnnotationNode *p_annotation, Node *p
ERR_FAIL_COND_V_MSG(p_node->type != Node::CLASS, false, R"("@icon" annotation can only be applied to classes.)");
ERR_FAIL_COND_V(p_annotation->resolved_arguments.is_empty(), false);
ClassNode *p_class = static_cast<ClassNode *>(p_node);
#ifdef DEBUG_ENABLED
if (!p_class->icon_path.is_empty()) {
push_error(R"("@icon" annotation can only be used once.)", p_annotation);
return false;
}
if (String(p_annotation->resolved_arguments[0]).is_empty()) {
push_error(R"("@icon" annotation argument must contain the path to the icon.)", p_annotation->arguments[0]);
return false;
}
#endif // DEBUG_ENABLED
p_class->icon_path = p_annotation->resolved_arguments[0];
return true;
}

5
modules/gdscript/tests/scripts/parser/errors/duplicate_icon.gd Normal file
View file

@ -0,0 +1,5 @@
@icon("res://1.png")
@icon("res://1.png")
func test():
pass

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

@ -0,0 +1,2 @@
GDTEST_PARSER_ERROR
"@icon" annotation can only be used once.

5
modules/gdscript/tests/scripts/parser/errors/duplicate_tool.gd Normal file
View file

@ -0,0 +1,5 @@
@tool
@tool
func test():
pass

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

@ -0,0 +1,2 @@
GDTEST_ANALYZER_ERROR
"@tool" annotation can only be used once.