Merge pull request #49287 from reduz/expose-variant-internal-binders

Make some Variant internal functions public.
godotengine · Jun 18, 2021 · a9c53fa · a9c53fa
2 parents 9b0800c + b5f1e88
commit a9c53fa
Show file tree

Hide file tree

Showing 8 changed files with 2,233 additions and 2,150 deletions.
diff --git a/core/variant/variant_construct.cpp b/core/variant/variant_construct.cpp
diff --git a/core/variant/variant_construct.h b/core/variant/variant_construct.h
diff --git a/core/variant/variant_op.cpp b/core/variant/variant_op.cpp
diff --git a/core/variant/variant_op.h b/core/variant/variant_op.h
diff --git a/core/variant/variant_setget.cpp b/core/variant/variant_setget.cpp
diff --git a/core/variant/variant_setget.h b/core/variant/variant_setget.h
diff --git a/modules/gdscript/gdscript_analyzer.cpp b/modules/gdscript/gdscript_analyzer.cpp
@@ -3447,6 +3447,7 @@ GDScriptParser::DataType GDScriptAnalyzer::get_operation_type(Variant::Operator
 	}
 
 	r_valid = true;
+	result.type_source = GDScriptParser::DataType::ANNOTATED_INFERRED;
 
 	result.kind = GDScriptParser::DataType::BUILTIN;
 	result.builtin_type = Variant::get_operator_return_type(p_operation, a_type, b_type);

diff --git a/modules/gdscript/gdscript_byte_codegen.cpp b/modules/gdscript/gdscript_byte_codegen.cpp
@@ -554,6 +554,14 @@ void GDScriptByteCodeGenerator::write_unary_operator(const Address &p_target, Va
 
 void GDScriptByteCodeGenerator::write_binary_operator(const Address &p_target, Variant::Operator p_operator, const Address &p_left_operand, const Address &p_right_operand) {
 	if (HAS_BUILTIN_TYPE(p_left_operand) && HAS_BUILTIN_TYPE(p_right_operand)) {
+		if (p_target.mode == Address::TEMPORARY) {
+			Variant::Type result_type = Variant::get_operator_return_type(p_operator, p_left_operand.type.builtin_type, p_right_operand.type.builtin_type);
+			Variant::Type temp_type = temporaries[p_target.address].type;
+			if (result_type != temp_type) {
+				write_type_adjust(p_target, result_type);
+			}
+		}
+
 		// Gather specific operator.
 		Variant::ValidatedOperatorEvaluator op_func = Variant::get_validated_operator_evaluator(p_operator, p_left_operand.type.builtin_type, p_right_operand.type.builtin_type);