From 12a63021d01abd7395be3614767648d0b39f42a3 Mon Sep 17 00:00:00 2001 From: Pedro Gimeno Date: Thu, 4 Apr 2019 18:53:55 +0200 Subject: Fix regression in automatic_face_movement_max_rotation_per_sec Values <= 0 should make the yaw change instant. This worked in 0.4.16 but was broken in 089f59458286. Per bug report by oil_boi_minetest on IRC. --- src/client/content_cao.cpp | 12 +++++++++--- 1 file changed, 9 insertions(+), 3 deletions(-) (limited to 'src/client') diff --git a/src/client/content_cao.cpp b/src/client/content_cao.cpp index 8643b5824..22f62e6c3 100644 --- a/src/client/content_cao.cpp +++ b/src/client/content_cao.cpp @@ -1000,10 +1000,16 @@ void GenericCAO::step(float dtime, ClientEnvironment *env) float target_yaw = atan2(m_velocity.Z, m_velocity.X) * 180 / M_PI + m_prop.automatic_face_movement_dir_offset; - float max_rotation_delta = - dtime * m_prop.automatic_face_movement_max_rotation_per_sec; + float max_rotation_per_sec = + m_prop.automatic_face_movement_max_rotation_per_sec; + if (max_rotation_per_sec > 0) { + float max_rotation_delta = dtime * max_rotation_per_sec; + + wrappedApproachShortest(m_rotation.Y, target_yaw, max_rotation_delta, 360.f); + } else + // Negative values of ...max_rotation_per_sec mean disabled. + m_rotation.Y = target_yaw; - wrappedApproachShortest(m_rotation.Y, target_yaw, max_rotation_delta, 360.f); rot_translator.val_current = m_rotation; updateNodePos(); -- cgit v1.2.3