diff options
author | Yuri Rubinsky <chaosus89@gmail.com> | 2023-01-20 09:48:43 +0300 |
---|---|---|
committer | GitHub <noreply@github.com> | 2023-01-20 09:48:43 +0300 |
commit | 2f2bf8489040b94928f915b6193bdd2903d3d4a3 (patch) | |
tree | 6644f817e33a551ff60d48708c610a79672193fe /servers/rendering/shader_preprocessor.cpp | |
parent | b2e375824bd84f6664d484144256bcf598895875 (diff) | |
parent | 6f57aae0ab6234afe0e6ab5f2566ae706adddc4e (diff) |
Merge pull request #71698 from bitsawer/fix_preprocessor_leak
Diffstat (limited to 'servers/rendering/shader_preprocessor.cpp')
-rw-r--r-- | servers/rendering/shader_preprocessor.cpp | 14 |
1 files changed, 6 insertions, 8 deletions
diff --git a/servers/rendering/shader_preprocessor.cpp b/servers/rendering/shader_preprocessor.cpp index 40c8acffe5..ccbf5defa2 100644 --- a/servers/rendering/shader_preprocessor.cpp +++ b/servers/rendering/shader_preprocessor.cpp @@ -1081,21 +1081,17 @@ ShaderPreprocessor::Define *ShaderPreprocessor::create_define(const String &p_bo return define; } -void ShaderPreprocessor::clear() { - if (state_owner && state != nullptr) { +void ShaderPreprocessor::clear_state() { + if (state != nullptr) { for (const RBMap<String, Define *>::Element *E = state->defines.front(); E; E = E->next()) { memdelete(E->get()); } - - memdelete(state); + state->defines.clear(); } - state_owner = false; state = nullptr; } Error ShaderPreprocessor::preprocess(State *p_state, const String &p_code, String &r_result) { - clear(); - output.clear(); state = p_state; @@ -1242,6 +1238,9 @@ Error ShaderPreprocessor::preprocess(const String &p_code, const String &p_filen } } } + + clear_state(); + return err; } @@ -1273,5 +1272,4 @@ ShaderPreprocessor::ShaderPreprocessor() { } ShaderPreprocessor::~ShaderPreprocessor() { - clear(); } |