1
0
mirror of https://github.com/godotengine/godot.git synced 2025-12-01 16:38:31 +00:00

Use RequiredParam and RequiredResult in a handful of places in order to test

This commit is contained in:
David Snopek
2025-10-08 16:37:43 -05:00
parent d95d49ee12
commit 090a4540b7
10 changed files with 68 additions and 68 deletions

View File

@@ -165,7 +165,7 @@ void ResourceLoader::_bind_methods() {
////// ResourceSaver ////// ////// ResourceSaver //////
Error ResourceSaver::save(const Ref<Resource> &p_resource, const String &p_path, BitField<SaverFlags> p_flags) { Error ResourceSaver::save(RequiredParam<Resource> p_resource, const String &p_path, BitField<SaverFlags> p_flags) {
return ::ResourceSaver::save(p_resource, p_path, p_flags); return ::ResourceSaver::save(p_resource, p_path, p_flags);
} }

View File

@@ -109,7 +109,7 @@ public:
static ResourceSaver *get_singleton() { return singleton; } static ResourceSaver *get_singleton() { return singleton; }
Error save(const Ref<Resource> &p_resource, const String &p_path, BitField<SaverFlags> p_flags); Error save(RequiredParam<Resource> p_resource, const String &p_path, BitField<SaverFlags> p_flags);
Error set_uid(const String &p_path, ResourceUID::ID p_uid); Error set_uid(const String &p_path, ResourceUID::ID p_uid);
Vector<String> get_recognized_extensions(const Ref<Resource> &p_resource); Vector<String> get_recognized_extensions(const Ref<Resource> &p_resource);
void add_resource_format_saver(Ref<ResourceFormatSaver> p_format_saver, bool p_at_front); void add_resource_format_saver(Ref<ResourceFormatSaver> p_format_saver, bool p_at_front);

View File

@@ -97,8 +97,8 @@ void ResourceFormatSaver::_bind_methods() {
GDVIRTUAL_BIND(_recognize_path, "resource", "path"); GDVIRTUAL_BIND(_recognize_path, "resource", "path");
} }
Error ResourceSaver::save(const Ref<Resource> &p_resource, const String &p_path, uint32_t p_flags) { Error ResourceSaver::save(RequiredParam<Resource> rp_resource, const String &p_path, uint32_t p_flags) {
ERR_FAIL_COND_V_MSG(p_resource.is_null(), ERR_INVALID_PARAMETER, vformat("Can't save empty resource to path '%s'.", p_path)); EXTRACT_PARAM_OR_FAIL_V_MSG(p_resource, rp_resource, ERR_INVALID_PARAMETER, vformat("Can't save empty resource to path '%s'.", p_path));
String path = p_path; String path = p_path;
if (path.is_empty()) { if (path.is_empty()) {
path = p_resource->get_path(); path = p_resource->get_path();

View File

@@ -83,7 +83,7 @@ public:
FLAG_REPLACE_SUBRESOURCE_PATHS = 64, FLAG_REPLACE_SUBRESOURCE_PATHS = 64,
}; };
static Error save(const Ref<Resource> &p_resource, const String &p_path = "", uint32_t p_flags = (uint32_t)FLAG_NONE); static Error save(RequiredParam<Resource> p_resource, const String &p_path = "", uint32_t p_flags = (uint32_t)FLAG_NONE);
static void get_recognized_extensions(const Ref<Resource> &p_resource, List<String> *p_extensions); static void get_recognized_extensions(const Ref<Resource> &p_resource, List<String> *p_extensions);
static void add_resource_format_saver(Ref<ResourceFormatSaver> p_format_saver, bool p_at_front = false); static void add_resource_format_saver(Ref<ResourceFormatSaver> p_format_saver, bool p_at_front = false);
static void remove_resource_format_saver(Ref<ResourceFormatSaver> p_format_saver); static void remove_resource_format_saver(Ref<ResourceFormatSaver> p_format_saver);

View File

@@ -95,7 +95,7 @@ void Tween::_stop_internal(bool p_reset) {
} }
} }
Ref<PropertyTweener> Tween::tween_property(const Object *p_target, const NodePath &p_property, Variant p_to, double p_duration) { RequiredResult<PropertyTweener> Tween::tween_property(const Object *p_target, const NodePath &p_property, Variant p_to, double p_duration) {
ERR_FAIL_NULL_V(p_target, nullptr); ERR_FAIL_NULL_V(p_target, nullptr);
CHECK_VALID(); CHECK_VALID();
@@ -118,7 +118,7 @@ Ref<PropertyTweener> Tween::tween_property(const Object *p_target, const NodePat
return tweener; return tweener;
} }
Ref<IntervalTweener> Tween::tween_interval(double p_time) { RequiredResult<IntervalTweener> Tween::tween_interval(double p_time) {
CHECK_VALID(); CHECK_VALID();
Ref<IntervalTweener> tweener; Ref<IntervalTweener> tweener;
@@ -127,7 +127,7 @@ Ref<IntervalTweener> Tween::tween_interval(double p_time) {
return tweener; return tweener;
} }
Ref<CallbackTweener> Tween::tween_callback(const Callable &p_callback) { RequiredResult<CallbackTweener> Tween::tween_callback(const Callable &p_callback) {
CHECK_VALID(); CHECK_VALID();
Ref<CallbackTweener> tweener; Ref<CallbackTweener> tweener;
@@ -136,7 +136,7 @@ Ref<CallbackTweener> Tween::tween_callback(const Callable &p_callback) {
return tweener; return tweener;
} }
Ref<MethodTweener> Tween::tween_method(const Callable &p_callback, const Variant p_from, Variant p_to, double p_duration) { RequiredResult<MethodTweener> Tween::tween_method(const Callable &p_callback, const Variant p_from, Variant p_to, double p_duration) {
CHECK_VALID(); CHECK_VALID();
if (!Animation::validate_type_match(p_from, p_to)) { if (!Animation::validate_type_match(p_from, p_to)) {
@@ -149,7 +149,7 @@ Ref<MethodTweener> Tween::tween_method(const Callable &p_callback, const Variant
return tweener; return tweener;
} }
Ref<SubtweenTweener> Tween::tween_subtween(const Ref<Tween> &p_subtween) { RequiredResult<SubtweenTweener> Tween::tween_subtween(const Ref<Tween> &p_subtween) {
CHECK_VALID(); CHECK_VALID();
// Ensure that the subtween being added is not null. // Ensure that the subtween being added is not null.
@@ -221,7 +221,7 @@ void Tween::clear() {
tweeners.clear(); tweeners.clear();
} }
Ref<Tween> Tween::bind_node(const Node *p_node) { RequiredResult<Tween> Tween::bind_node(const Node *p_node) {
ERR_FAIL_NULL_V(p_node, this); ERR_FAIL_NULL_V(p_node, this);
bound_node = p_node->get_instance_id(); bound_node = p_node->get_instance_id();
@@ -229,7 +229,7 @@ Ref<Tween> Tween::bind_node(const Node *p_node) {
return this; return this;
} }
Ref<Tween> Tween::set_process_mode(TweenProcessMode p_mode) { RequiredResult<Tween> Tween::set_process_mode(TweenProcessMode p_mode) {
process_mode = p_mode; process_mode = p_mode;
return this; return this;
} }
@@ -238,7 +238,7 @@ Tween::TweenProcessMode Tween::get_process_mode() const {
return process_mode; return process_mode;
} }
Ref<Tween> Tween::set_pause_mode(TweenPauseMode p_mode) { RequiredResult<Tween> Tween::set_pause_mode(TweenPauseMode p_mode) {
pause_mode = p_mode; pause_mode = p_mode;
return this; return this;
} }
@@ -247,7 +247,7 @@ Tween::TweenPauseMode Tween::get_pause_mode() const {
return pause_mode; return pause_mode;
} }
Ref<Tween> Tween::set_ignore_time_scale(bool p_ignore) { RequiredResult<Tween> Tween::set_ignore_time_scale(bool p_ignore) {
ignore_time_scale = p_ignore; ignore_time_scale = p_ignore;
return this; return this;
} }
@@ -256,13 +256,13 @@ bool Tween::is_ignoring_time_scale() const {
return ignore_time_scale; return ignore_time_scale;
} }
Ref<Tween> Tween::set_parallel(bool p_parallel) { RequiredResult<Tween> Tween::set_parallel(bool p_parallel) {
default_parallel = p_parallel; default_parallel = p_parallel;
parallel_enabled = p_parallel; parallel_enabled = p_parallel;
return this; return this;
} }
Ref<Tween> Tween::set_loops(int p_loops) { RequiredResult<Tween> Tween::set_loops(int p_loops) {
loops = p_loops; loops = p_loops;
return this; return this;
} }
@@ -275,12 +275,12 @@ int Tween::get_loops_left() const {
} }
} }
Ref<Tween> Tween::set_speed_scale(float p_speed) { RequiredResult<Tween> Tween::set_speed_scale(float p_speed) {
speed_scale = p_speed; speed_scale = p_speed;
return this; return this;
} }
Ref<Tween> Tween::set_trans(TransitionType p_trans) { RequiredResult<Tween> Tween::set_trans(TransitionType p_trans) {
default_transition = p_trans; default_transition = p_trans;
return this; return this;
} }
@@ -289,7 +289,7 @@ Tween::TransitionType Tween::get_trans() const {
return default_transition; return default_transition;
} }
Ref<Tween> Tween::set_ease(EaseType p_ease) { RequiredResult<Tween> Tween::set_ease(EaseType p_ease) {
default_ease = p_ease; default_ease = p_ease;
return this; return this;
} }
@@ -298,12 +298,12 @@ Tween::EaseType Tween::get_ease() const {
return default_ease; return default_ease;
} }
Ref<Tween> Tween::parallel() { RequiredResult<Tween> Tween::parallel() {
parallel_enabled = true; parallel_enabled = true;
return this; return this;
} }
Ref<Tween> Tween::chain() { RequiredResult<Tween> Tween::chain() {
parallel_enabled = false; parallel_enabled = false;
return this; return this;
} }
@@ -554,7 +554,7 @@ double PropertyTweener::_get_custom_interpolated_value(const Variant &p_value) {
return result; return result;
} }
Ref<PropertyTweener> PropertyTweener::from(const Variant &p_value) { RequiredResult<PropertyTweener> PropertyTweener::from(const Variant &p_value) {
Ref<Tween> tween = _get_tween(); Ref<Tween> tween = _get_tween();
ERR_FAIL_COND_V(tween.is_null(), nullptr); ERR_FAIL_COND_V(tween.is_null(), nullptr);
@@ -568,32 +568,32 @@ Ref<PropertyTweener> PropertyTweener::from(const Variant &p_value) {
return this; return this;
} }
Ref<PropertyTweener> PropertyTweener::from_current() { RequiredResult<PropertyTweener> PropertyTweener::from_current() {
do_continue = false; do_continue = false;
return this; return this;
} }
Ref<PropertyTweener> PropertyTweener::as_relative() { RequiredResult<PropertyTweener> PropertyTweener::as_relative() {
relative = true; relative = true;
return this; return this;
} }
Ref<PropertyTweener> PropertyTweener::set_trans(Tween::TransitionType p_trans) { RequiredResult<PropertyTweener> PropertyTweener::set_trans(Tween::TransitionType p_trans) {
trans_type = p_trans; trans_type = p_trans;
return this; return this;
} }
Ref<PropertyTweener> PropertyTweener::set_ease(Tween::EaseType p_ease) { RequiredResult<PropertyTweener> PropertyTweener::set_ease(Tween::EaseType p_ease) {
ease_type = p_ease; ease_type = p_ease;
return this; return this;
} }
Ref<PropertyTweener> PropertyTweener::set_custom_interpolator(const Callable &p_method) { RequiredResult<PropertyTweener> PropertyTweener::set_custom_interpolator(const Callable &p_method) {
custom_method = p_method; custom_method = p_method;
return this; return this;
} }
Ref<PropertyTweener> PropertyTweener::set_delay(double p_delay) { RequiredResult<PropertyTweener> PropertyTweener::set_delay(double p_delay) {
delay = p_delay; delay = p_delay;
return this; return this;
} }
@@ -731,7 +731,7 @@ IntervalTweener::IntervalTweener() {
ERR_FAIL_MSG("IntervalTweener can't be created directly. Use the tween_interval() method in Tween."); ERR_FAIL_MSG("IntervalTweener can't be created directly. Use the tween_interval() method in Tween.");
} }
Ref<CallbackTweener> CallbackTweener::set_delay(double p_delay) { RequiredResult<CallbackTweener> CallbackTweener::set_delay(double p_delay) {
delay = p_delay; delay = p_delay;
return this; return this;
} }
@@ -781,17 +781,17 @@ CallbackTweener::CallbackTweener() {
ERR_FAIL_MSG("CallbackTweener can't be created directly. Use the tween_callback() method in Tween."); ERR_FAIL_MSG("CallbackTweener can't be created directly. Use the tween_callback() method in Tween.");
} }
Ref<MethodTweener> MethodTweener::set_delay(double p_delay) { RequiredResult<MethodTweener> MethodTweener::set_delay(double p_delay) {
delay = p_delay; delay = p_delay;
return this; return this;
} }
Ref<MethodTweener> MethodTweener::set_trans(Tween::TransitionType p_trans) { RequiredResult<MethodTweener> MethodTweener::set_trans(Tween::TransitionType p_trans) {
trans_type = p_trans; trans_type = p_trans;
return this; return this;
} }
Ref<MethodTweener> MethodTweener::set_ease(Tween::EaseType p_ease) { RequiredResult<MethodTweener> MethodTweener::set_ease(Tween::EaseType p_ease) {
ease_type = p_ease; ease_type = p_ease;
return this; return this;
} }
@@ -912,7 +912,7 @@ bool SubtweenTweener::step(double &r_delta) {
return true; return true;
} }
Ref<SubtweenTweener> SubtweenTweener::set_delay(double p_delay) { RequiredResult<SubtweenTweener> SubtweenTweener::set_delay(double p_delay) {
delay = p_delay; delay = p_delay;
return this; return this;
} }

View File

@@ -143,11 +143,11 @@ protected:
virtual String _to_string() override; virtual String _to_string() override;
public: public:
Ref<PropertyTweener> tween_property(const Object *p_target, const NodePath &p_property, Variant p_to, double p_duration); RequiredResult<PropertyTweener> tween_property(const Object *p_target, const NodePath &p_property, Variant p_to, double p_duration);
Ref<IntervalTweener> tween_interval(double p_time); RequiredResult<IntervalTweener> tween_interval(double p_time);
Ref<CallbackTweener> tween_callback(const Callable &p_callback); RequiredResult<CallbackTweener> tween_callback(const Callable &p_callback);
Ref<MethodTweener> tween_method(const Callable &p_callback, const Variant p_from, Variant p_to, double p_duration); RequiredResult<MethodTweener> tween_method(const Callable &p_callback, const Variant p_from, Variant p_to, double p_duration);
Ref<SubtweenTweener> tween_subtween(const Ref<Tween> &p_subtween); RequiredResult<SubtweenTweener> tween_subtween(const Ref<Tween> &p_subtween);
void append(Ref<Tweener> p_tweener); void append(Ref<Tweener> p_tweener);
bool custom_step(double p_delta); bool custom_step(double p_delta);
@@ -160,25 +160,25 @@ public:
bool is_valid(); bool is_valid();
void clear(); void clear();
Ref<Tween> bind_node(const Node *p_node); RequiredResult<Tween> bind_node(const Node *p_node);
Ref<Tween> set_process_mode(TweenProcessMode p_mode); RequiredResult<Tween> set_process_mode(TweenProcessMode p_mode);
TweenProcessMode get_process_mode() const; TweenProcessMode get_process_mode() const;
Ref<Tween> set_pause_mode(TweenPauseMode p_mode); RequiredResult<Tween> set_pause_mode(TweenPauseMode p_mode);
TweenPauseMode get_pause_mode() const; TweenPauseMode get_pause_mode() const;
Ref<Tween> set_ignore_time_scale(bool p_ignore = true); RequiredResult<Tween> set_ignore_time_scale(bool p_ignore = true);
bool is_ignoring_time_scale() const; bool is_ignoring_time_scale() const;
Ref<Tween> set_parallel(bool p_parallel); RequiredResult<Tween> set_parallel(bool p_parallel);
Ref<Tween> set_loops(int p_loops); RequiredResult<Tween> set_loops(int p_loops);
int get_loops_left() const; int get_loops_left() const;
Ref<Tween> set_speed_scale(float p_speed); RequiredResult<Tween> set_speed_scale(float p_speed);
Ref<Tween> set_trans(TransitionType p_trans); RequiredResult<Tween> set_trans(TransitionType p_trans);
TransitionType get_trans() const; TransitionType get_trans() const;
Ref<Tween> set_ease(EaseType p_ease); RequiredResult<Tween> set_ease(EaseType p_ease);
EaseType get_ease() const; EaseType get_ease() const;
Ref<Tween> parallel(); RequiredResult<Tween> parallel();
Ref<Tween> chain(); RequiredResult<Tween> chain();
static real_t run_equation(TransitionType p_trans_type, EaseType p_ease_type, real_t t, real_t b, real_t c, real_t d); static real_t run_equation(TransitionType p_trans_type, EaseType p_ease_type, real_t t, real_t b, real_t c, real_t d);
static Variant interpolate_variant(const Variant &p_initial_val, const Variant &p_delta_val, double p_time, double p_duration, Tween::TransitionType p_trans, Tween::EaseType p_ease); static Variant interpolate_variant(const Variant &p_initial_val, const Variant &p_delta_val, double p_time, double p_duration, Tween::TransitionType p_trans, Tween::EaseType p_ease);
@@ -203,13 +203,13 @@ class PropertyTweener : public Tweener {
double _get_custom_interpolated_value(const Variant &p_value); double _get_custom_interpolated_value(const Variant &p_value);
public: public:
Ref<PropertyTweener> from(const Variant &p_value); RequiredResult<PropertyTweener> from(const Variant &p_value);
Ref<PropertyTweener> from_current(); RequiredResult<PropertyTweener> from_current();
Ref<PropertyTweener> as_relative(); RequiredResult<PropertyTweener> as_relative();
Ref<PropertyTweener> set_trans(Tween::TransitionType p_trans); RequiredResult<PropertyTweener> set_trans(Tween::TransitionType p_trans);
Ref<PropertyTweener> set_ease(Tween::EaseType p_ease); RequiredResult<PropertyTweener> set_ease(Tween::EaseType p_ease);
Ref<PropertyTweener> set_custom_interpolator(const Callable &p_method); RequiredResult<PropertyTweener> set_custom_interpolator(const Callable &p_method);
Ref<PropertyTweener> set_delay(double p_delay); RequiredResult<PropertyTweener> set_delay(double p_delay);
void set_tween(const Ref<Tween> &p_tween) override; void set_tween(const Ref<Tween> &p_tween) override;
void start() override; void start() override;
@@ -259,7 +259,7 @@ class CallbackTweener : public Tweener {
GDCLASS(CallbackTweener, Tweener); GDCLASS(CallbackTweener, Tweener);
public: public:
Ref<CallbackTweener> set_delay(double p_delay); RequiredResult<CallbackTweener> set_delay(double p_delay);
bool step(double &r_delta) override; bool step(double &r_delta) override;
@@ -280,9 +280,9 @@ class MethodTweener : public Tweener {
GDCLASS(MethodTweener, Tweener); GDCLASS(MethodTweener, Tweener);
public: public:
Ref<MethodTweener> set_trans(Tween::TransitionType p_trans); RequiredResult<MethodTweener> set_trans(Tween::TransitionType p_trans);
Ref<MethodTweener> set_ease(Tween::EaseType p_ease); RequiredResult<MethodTweener> set_ease(Tween::EaseType p_ease);
Ref<MethodTweener> set_delay(double p_delay); RequiredResult<MethodTweener> set_delay(double p_delay);
void set_tween(const Ref<Tween> &p_tween) override; void set_tween(const Ref<Tween> &p_tween) override;
bool step(double &r_delta) override; bool step(double &r_delta) override;
@@ -315,7 +315,7 @@ public:
void start() override; void start() override;
bool step(double &r_delta) override; bool step(double &r_delta) override;
Ref<SubtweenTweener> set_delay(double p_delay); RequiredResult<SubtweenTweener> set_delay(double p_delay);
SubtweenTweener(const Ref<Tween> &p_subtween); SubtweenTweener(const Ref<Tween> &p_subtween);
SubtweenTweener(); SubtweenTweener();

View File

@@ -1696,11 +1696,11 @@ void Node::_add_child_nocheck(Node *p_child, const StringName &p_name, InternalM
emit_signal(SNAME("child_order_changed")); emit_signal(SNAME("child_order_changed"));
} }
void Node::add_child(Node *p_child, bool p_force_readable_name, InternalMode p_internal) { void Node::add_child(RequiredParam<Node> rp_child, bool p_force_readable_name, InternalMode p_internal) {
ERR_FAIL_COND_MSG(data.tree && !Thread::is_main_thread(), "Adding children to a node inside the SceneTree is only allowed from the main thread. Use call_deferred(\"add_child\",node)."); ERR_FAIL_COND_MSG(data.tree && !Thread::is_main_thread(), "Adding children to a node inside the SceneTree is only allowed from the main thread. Use call_deferred(\"add_child\",node).");
ERR_THREAD_GUARD ERR_THREAD_GUARD
ERR_FAIL_NULL(p_child); EXTRACT_PARAM_OR_FAIL(p_child, rp_child);
ERR_FAIL_COND_MSG(p_child == this, vformat("Can't add child '%s' to itself.", p_child->get_name())); // adding to itself! ERR_FAIL_COND_MSG(p_child == this, vformat("Can't add child '%s' to itself.", p_child->get_name())); // adding to itself!
ERR_FAIL_COND_MSG(p_child->data.parent, vformat("Can't add child '%s' to '%s', already has a parent '%s'.", p_child->get_name(), get_name(), p_child->data.parent->get_name())); //Fail if node has a parent ERR_FAIL_COND_MSG(p_child->data.parent, vformat("Can't add child '%s' to '%s', already has a parent '%s'.", p_child->get_name(), get_name(), p_child->data.parent->get_name())); //Fail if node has a parent
#ifdef DEBUG_ENABLED #ifdef DEBUG_ENABLED
@@ -2635,7 +2635,7 @@ void Node::_propagate_replace_owner(Node *p_owner, Node *p_by_owner) {
data.blocked--; data.blocked--;
} }
Ref<Tween> Node::create_tween() { RequiredResult<Tween> Node::create_tween() {
ERR_THREAD_GUARD_V(Ref<Tween>()); ERR_THREAD_GUARD_V(Ref<Tween>());
SceneTree *tree = data.tree; SceneTree *tree = data.tree;

View File

@@ -512,7 +512,7 @@ public:
InternalMode get_internal_mode() const; InternalMode get_internal_mode() const;
void add_child(Node *p_child, bool p_force_readable_name = false, InternalMode p_internal = INTERNAL_MODE_DISABLED); void add_child(RequiredParam<Node> rp_child, bool p_force_readable_name = false, InternalMode p_internal = INTERNAL_MODE_DISABLED);
void add_sibling(Node *p_sibling, bool p_force_readable_name = false); void add_sibling(Node *p_sibling, bool p_force_readable_name = false);
void remove_child(Node *p_child); void remove_child(Node *p_child);
@@ -608,7 +608,7 @@ public:
} }
} }
Ref<Tween> create_tween(); RequiredResult<Tween> create_tween();
void print_tree(); void print_tree();
void print_tree_pretty(); void print_tree_pretty();

View File

@@ -1751,7 +1751,7 @@ Ref<SceneTreeTimer> SceneTree::create_timer(double p_delay_sec, bool p_process_a
return stt; return stt;
} }
Ref<Tween> SceneTree::create_tween() { RequiredResult<Tween> SceneTree::create_tween() {
_THREAD_SAFE_METHOD_ _THREAD_SAFE_METHOD_
Ref<Tween> tween; Ref<Tween> tween;
tween.instantiate(this); tween.instantiate(this);

View File

@@ -429,7 +429,7 @@ public:
void unload_current_scene(); void unload_current_scene();
Ref<SceneTreeTimer> create_timer(double p_delay_sec, bool p_process_always = true, bool p_process_in_physics = false, bool p_ignore_time_scale = false); Ref<SceneTreeTimer> create_timer(double p_delay_sec, bool p_process_always = true, bool p_process_in_physics = false, bool p_ignore_time_scale = false);
Ref<Tween> create_tween(); RequiredResult<Tween> create_tween();
void remove_tween(const Ref<Tween> &p_tween); void remove_tween(const Ref<Tween> &p_tween);
TypedArray<Tween> get_processed_tweens(); TypedArray<Tween> get_processed_tweens();