diff options
| author | Rémi Verschelde <remi@verschelde.fr> | 2022-03-21 21:21:49 +0100 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2022-03-21 21:21:49 +0100 |
| commit | 188fa0c846bc98766a3c79cfa4106777632686be (patch) | |
| tree | d272a8260c4a64bbdbc1b8a8592be08ba492fa65 /modules | |
| parent | 5e249d047fe46faaaf53c9ecf36738b13a231ca0 (diff) | |
| parent | 8a6ab899ec5c3f323766c529a5c506e4429c1f17 (diff) | |
Merge pull request #59194 from Chaosus/gds_export_flags_limit_error
Add an error emitting when the `@export_flags` arg count is exceeded
Diffstat (limited to 'modules')
| -rw-r--r-- | modules/gdscript/gdscript_parser.cpp | 9 |
1 files changed, 9 insertions, 0 deletions
diff --git a/modules/gdscript/gdscript_parser.cpp b/modules/gdscript/gdscript_parser.cpp index 725b62f6d6..812da6caf2 100644 --- a/modules/gdscript/gdscript_parser.cpp +++ b/modules/gdscript/gdscript_parser.cpp @@ -3480,6 +3480,15 @@ template <PropertyHint t_hint, Variant::Type t_type> bool GDScriptParser::export_annotations(const AnnotationNode *p_annotation, Node *p_node) { ERR_FAIL_COND_V_MSG(p_node->type != Node::VARIABLE, false, vformat(R"("%s" annotation can only be applied to variables.)", p_annotation->name)); + { + const int max_flags = 32; + + if (t_hint == PropertyHint::PROPERTY_HINT_FLAGS && p_annotation->resolved_arguments.size() > max_flags) { + push_error(vformat(R"(The argument count limit for "@export_flags" is exceeded (%d/%d).)", p_annotation->resolved_arguments.size(), max_flags), p_annotation); + return false; + } + } + VariableNode *variable = static_cast<VariableNode *>(p_node); if (variable->exported) { push_error(vformat(R"(Annotation "%s" cannot be used with another "@export" annotation.)", p_annotation->name), p_annotation); |