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

Revert "Ensure non-emitting particles not processed on entering tree" 

This reverts commit 24b7f08892.

This causes a particles regression in tps-demo
This commit is contained in:
Hein-Pieter van Braam-Stewart 2019-04-21 00:59:13 +02:00
parent 4fefc0896d
commit 4aa7760c74
2 changed files with 7 additions and 10 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
scene/2d/cpu_particles_2d.cpp
View File

@ -37,8 +37,7 @@
void CPUParticles2D::set_emitting(bool p_emitting) { void CPUParticles2D::set_emitting(bool p_emitting) {
emitting = p_emitting; emitting = p_emitting;
if (emitting) set_process_internal(true);
set_process_internal(true);
} }
void CPUParticles2D::set_amount(int p_amount) { void CPUParticles2D::set_amount(int p_amount) {
@ -966,7 +965,7 @@ void CPUParticles2D::_update_render_thread() {
void CPUParticles2D::_notification(int p_what) { void CPUParticles2D::_notification(int p_what) {
if (p_what == NOTIFICATION_ENTER_TREE) { if (p_what == NOTIFICATION_ENTER_TREE) {
set_process_internal(emitting); _set_redraw(true);
} }
if (p_what == NOTIFICATION_EXIT_TREE) { if (p_what == NOTIFICATION_EXIT_TREE) {
@ -1002,6 +1001,7 @@ void CPUParticles2D::_notification(int p_what) {
float delta = get_process_delta_time(); float delta = get_process_delta_time();
if (emitting) { if (emitting) {
_set_redraw(true);
inactive_time = 0; inactive_time = 0;
} else { } else {
inactive_time += delta; inactive_time += delta;
@ -1017,7 +1017,6 @@ void CPUParticles2D::_notification(int p_what) {
return; return;
} }
} }
_set_redraw(true);
if (time == 0 && pre_process_time > 0.0) { if (time == 0 && pre_process_time > 0.0) {

10
scene/3d/cpu_particles.cpp
View File

@ -47,8 +47,7 @@ PoolVector<Face3> CPUParticles::get_faces(uint32_t p_usage_flags) const {
void CPUParticles::set_emitting(bool p_emitting) { void CPUParticles::set_emitting(bool p_emitting) {
emitting = p_emitting; emitting = p_emitting;
if (emitting) set_process_internal(true);
set_process_internal(true);
} }
void CPUParticles::set_amount(int p_amount) { void CPUParticles::set_amount(int p_amount) {
@ -1003,11 +1002,9 @@ void CPUParticles::_set_redraw(bool p_redraw) {
if (redraw) { if (redraw) {
VS::get_singleton()->connect("frame_pre_draw", this, "_update_render_thread"); VS::get_singleton()->connect("frame_pre_draw", this, "_update_render_thread");
VS::get_singleton()->instance_geometry_set_flag(get_instance(), VS::INSTANCE_FLAG_DRAW_NEXT_FRAME_IF_VISIBLE, true); VS::get_singleton()->instance_geometry_set_flag(get_instance(), VS::INSTANCE_FLAG_DRAW_NEXT_FRAME_IF_VISIBLE, true);
VS::get_singleton()->multimesh_set_visible_instances(multimesh, -1);
} else { } else {
VS::get_singleton()->disconnect("frame_pre_draw", this, "_update_render_thread"); VS::get_singleton()->disconnect("frame_pre_draw", this, "_update_render_thread");
VS::get_singleton()->instance_geometry_set_flag(get_instance(), VS::INSTANCE_FLAG_DRAW_NEXT_FRAME_IF_VISIBLE, false); VS::get_singleton()->instance_geometry_set_flag(get_instance(), VS::INSTANCE_FLAG_DRAW_NEXT_FRAME_IF_VISIBLE, false);
VS::get_singleton()->multimesh_set_visible_instances(multimesh, 0);
} }
#ifndef NO_THREADS #ifndef NO_THREADS
update_mutex->unlock(); update_mutex->unlock();
@ -1032,7 +1029,7 @@ void CPUParticles::_update_render_thread() {
void CPUParticles::_notification(int p_what) { void CPUParticles::_notification(int p_what) {
if (p_what == NOTIFICATION_ENTER_TREE) { if (p_what == NOTIFICATION_ENTER_TREE) {
set_process_internal(emitting); _set_redraw(true);
} }
if (p_what == NOTIFICATION_EXIT_TREE) { if (p_what == NOTIFICATION_EXIT_TREE) {
@ -1051,6 +1048,8 @@ void CPUParticles::_notification(int p_what) {
float delta = get_process_delta_time(); float delta = get_process_delta_time();
if (emitting) { if (emitting) {
_set_redraw(true);
inactive_time = 0; inactive_time = 0;
} else { } else {
inactive_time += delta; inactive_time += delta;
@ -1066,7 +1065,6 @@ void CPUParticles::_notification(int p_what) {
return; return;
} }
} }
_set_redraw(true);
bool processed = false; bool processed = false;