Don't allow negative values for OS.delay_usec()/OS.delay_msec()

This closes #46190.
This commit is contained in:
Hugo Locurcio 2021-02-18 21:59:09 +01:00
parent 247b7e2448
commit 76f1f9b3c5
No known key found for this signature in database
GPG Key ID: 39E8F8BE30B0A49C
3 changed files with 14 additions and 6 deletions

View File

@ -519,11 +519,19 @@ double _OS::get_unix_time() const {
return OS::get_singleton()->get_unix_time(); return OS::get_singleton()->get_unix_time();
} }
void _OS::delay_usec(uint32_t p_usec) const { /** This method uses a signed argument for better error reporting as it's used from the scripting API. */
void _OS::delay_usec(int p_usec) const {
ERR_FAIL_COND_MSG(
p_usec < 0,
vformat("Can't sleep for %d microseconds. The delay provided must be greater than or equal to 0 microseconds.", p_usec));
OS::get_singleton()->delay_usec(p_usec); OS::get_singleton()->delay_usec(p_usec);
} }
void _OS::delay_msec(uint32_t p_msec) const { /** This method uses a signed argument for better error reporting as it's used from the scripting API. */
void _OS::delay_msec(int p_msec) const {
ERR_FAIL_COND_MSG(
p_msec < 0,
vformat("Can't sleep for %d milliseconds. The delay provided must be greater than or equal to 0 milliseconds.", p_msec));
OS::get_singleton()->delay_usec(int64_t(p_msec) * 1000); OS::get_singleton()->delay_usec(int64_t(p_msec) * 1000);
} }

View File

@ -211,8 +211,8 @@ public:
uint64_t get_static_memory_usage() const; uint64_t get_static_memory_usage() const;
uint64_t get_static_memory_peak_usage() const; uint64_t get_static_memory_peak_usage() const;
void delay_usec(uint32_t p_usec) const; void delay_usec(int p_usec) const;
void delay_msec(uint32_t p_msec) const; void delay_msec(int p_msec) const;
uint32_t get_ticks_msec() const; uint32_t get_ticks_msec() const;
uint64_t get_ticks_usec() const; uint64_t get_ticks_usec() const;

View File

@ -54,7 +54,7 @@
<argument index="0" name="msec" type="int"> <argument index="0" name="msec" type="int">
</argument> </argument>
<description> <description>
Delay execution of the current thread by [code]msec[/code] milliseconds. Delay execution of the current thread by [code]msec[/code] milliseconds. [code]usec[/code] must be greater than or equal to [code]0[/code]. Otherwise, [method delay_msec] will do nothing and will print an error message.
</description> </description>
</method> </method>
<method name="delay_usec" qualifiers="const"> <method name="delay_usec" qualifiers="const">
@ -63,7 +63,7 @@
<argument index="0" name="usec" type="int"> <argument index="0" name="usec" type="int">
</argument> </argument>
<description> <description>
Delay execution of the current thread by [code]usec[/code] microseconds. Delay execution of the current thread by [code]usec[/code] microseconds. [code]usec[/code] must be greater than or equal to [code]0[/code]. Otherwise, [method delay_usec] will do nothing and will print an error message.
</description> </description>
</method> </method>
<method name="dump_memory_to_file"> <method name="dump_memory_to_file">