godotengine
/
godot
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

MessageQueue::flush now always destroys parameters of a spent message 

Previously, destructors of Variant parameters were not called if the
target of the message was not found.
This commit is contained in:
Ibrahn Sahir 2019-05-19 14:05:15 +01:00
parent 33897d9b58
commit 5626a1ec20
1 changed files with 7 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

12
core/message_queue.cpp
View File

@ -302,10 +302,6 @@ void MessageQueue::flush() {
_call_function(target, message->target, args, message->args, message->type & FLAG_SHOW_ERROR); _call_function(target, message->target, args, message->args, message->type & FLAG_SHOW_ERROR);
for (int i = 0; i < message->args; i++) {
args[i].~Variant();
}
} break; } break;
case TYPE_NOTIFICATION: { case TYPE_NOTIFICATION: {
@ -319,11 +315,17 @@ void MessageQueue::flush() {
// messages don't expect a return value // messages don't expect a return value
target->set(message->target, *arg); target->set(message->target, *arg);
arg->~Variant();
} break; } break;
} }
} }
if ((message->type & FLAG_MASK) != TYPE_NOTIFICATION) {
Variant *args = (Variant *)(message + 1);
for (int i = 0; i < message->args; i++) {
args[i].~Variant();
}
}
message->~Message(); message->~Message();
_THREAD_SAFE_LOCK_ _THREAD_SAFE_LOCK_