Bug 507: Reset stuff upon player death
authorFlorian Forster <supertux@octo.it>
Sat, 30 Jan 2010 10:43:02 +0000 (10:43 +0000)
committerFlorian Forster <supertux@octo.it>
Sat, 30 Jan 2010 10:43:02 +0000 (10:43 +0000)
If Tux dies while he is subject to increased gravity (jump_early_apex), he
retains this attribute during the dying sequence, which looks weird.

I will attach a patch that resets the safe_timer and gravity and, for
consistency, resets the invincibility upon death instead of checking !dying
when creating sparkles.

Resolves #507. Thanks to Matt McCutchen for this patch.

SVN-Revision: 6290

src/object/player.cpp

index 77ef593..06941c6 100644 (file)
@@ -390,7 +390,7 @@ Player::update(float elapsed_time)
   ice_this_frame = false;
 
   // when invincible, spawn particles
-  if (invincible_timer.started() && !dying)
+  if (invincible_timer.started())
   {
     if (systemRandom.rand(0, 2) == 0) {
       float px = systemRandom.randf(bbox.p1.x+0, bbox.p2.x-0);
@@ -1291,6 +1291,9 @@ Player::kill(bool completely)
       GameSession::current()->set_reset_point("", Vector());
     }
     physic.enable_gravity(true);
+    physic.set_gravity_modifier(1.0f); // Undo jump_early_apex
+    safe_timer.stop();
+    invincible_timer.stop();
     physic.set_acceleration(0, 0);
     physic.set_velocity(0, -700);
     set_bonus(NO_BONUS, true);