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

GDScript: Fix type certainty for result of ternary operator

Browse source
This commit is contained in:
Dmitrii Maganov 2023-02-01 14:52:18 +02:00
parent 0a9e6e478e
commit 1a810ff45e
5 changed files with 23 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
modules/gdscript/gdscript_analyzer.cpp
View file

@ -3988,7 +3988,6 @@ void GDScriptAnalyzer::reduce_ternary_op(GDScriptParser::TernaryOpNode *p_ternar
if (!is_type_compatible(true_type, false_type)) {
result = false_type;
if (!is_type_compatible(false_type, true_type)) {
result.type_source = GDScriptParser::DataType::UNDETECTED;
result.kind = GDScriptParser::DataType::VARIANT;
#ifdef DEBUG_ENABLED
parser->push_warning(p_ternary_op, GDScriptWarning::INCOMPATIBLE_TERNARY);
@ -3996,6 +3995,7 @@ void GDScriptAnalyzer::reduce_ternary_op(GDScriptParser::TernaryOpNode *p_ternar
}
}
}
result.type_source = true_type.is_hard_type() && false_type.is_hard_type() ? GDScriptParser::DataType::ANNOTATED_INFERRED : GDScriptParser::DataType::INFERRED;
p_ternary_op->set_datatype(result);
}

6
modules/gdscript/tests/scripts/analyzer/errors/ternary_weak_infer.gd Normal file
View file

@ -0,0 +1,6 @@
func test():
var left_hard_int := 1
var right_weak_int = 2
var result_hm_int := left_hard_int if true else right_weak_int
print('not ok')

2
modules/gdscript/tests/scripts/analyzer/errors/ternary_weak_infer.out Normal file
View file

@ -0,0 +1,2 @@
GDTEST_ANALYZER_ERROR
Cannot infer the type of "result_hm_int" variable because the value doesn't have a set type.

12
modules/gdscript/tests/scripts/analyzer/features/ternary_hard_infer.gd Normal file
View file

@ -0,0 +1,12 @@
func test():
var left_hard_int := 1
var right_hard_int := 2
var result_hard_int := left_hard_int if true else right_hard_int
assert(result_hard_int == 1)
var left_hard_variant := 1 as Variant
var right_hard_variant := 2.0 as Variant
var result_hard_variant := left_hard_variant if true else right_hard_variant
assert(result_hard_variant == 1)
print('ok')

2
modules/gdscript/tests/scripts/analyzer/features/ternary_hard_infer.out Normal file
View file

@ -0,0 +1,2 @@
GDTEST_OK
ok