godotengine
/
godot
mirror of https://github.com/godotengine/godot
1
0
Fork 0
Code

Core: Isolate `Ref` forward declare logic

This commit is contained in:
Thaddeus Crews 2025-01-09 10:02:27 -06:00
parent 24d74510e5
commit e5d203f452
No known key found for this signature in database
GPG Key ID: 62181B86FE9E5D84
2 changed files with 9 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
core/io/logger.cpp
View File

@ -35,10 +35,9 @@
#include "core/os/time.h" #include "core/os/time.h"
#include "modules/modules_enabled.gen.h" // For regex. #include "modules/modules_enabled.gen.h" // For regex.
#ifdef MODULE_REGEX_ENABLED #ifdef MODULE_REGEX_ENABLED
#include "modules/regex/regex.h" #include "modules/regex/regex.h"
#else
class RegEx : public RefCounted {};
#endif // MODULE_REGEX_ENABLED #endif // MODULE_REGEX_ENABLED
#if defined(MINGW_ENABLED) || defined(_MSC_VER) #if defined(MINGW_ENABLED) || defined(_MSC_VER)

11
core/object/ref_counted.h
View File

@ -181,10 +181,15 @@ public:
// do a lot of referencing on references and stuff // do a lot of referencing on references and stuff
// mutexes will avoid more crashes? // mutexes will avoid more crashes?
if (reference && reference->unreference()) { if (reference) {
memdelete(reference); // NOTE: `reinterpret_cast` is "safe" here, as the only way to achieve a non-null
// reference is after `ref_pointer` validates the type. This allows for passing
// forward-declared types without cascading dependency-chains.
if (reinterpret_cast<RefCounted *>(reference)->unreference()) {
memdelete(reinterpret_cast<RefCounted *>(reference));
}
reference = nullptr;
} }
reference = nullptr;
} }
template <typename... VarArgs> template <typename... VarArgs>