handleCommand_Breath mustn't update breath is player is dead. A dead …

…player doesn't breath. Add Player::isDead function to check player is dead
minetest · Mar 12, 2015 · afb19f3 · afb19f3
1 parent 3067d78
commit afb19f3
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 0 deletions.
diff --git a/src/network/packethandlers/server.cpp b/src/network/packethandlers/server.cpp
@@ -916,6 +916,15 @@ void Server::handleCommand_Breath(NetworkPacket* pkt)
 		return;
 	}
 
+	/*
+	 * If player is dead, we don't need to update the breath
+	 * He is dead !
+	 */
+	if (player->isDead()) {
+		return;
+	}
+
+
 	PlayerSAO *playersao = player->getPlayerSAO();
 	if (playersao == NULL) {
 		errorstream << "Server::ProcessData(): Cancelling: "

diff --git a/src/player.h b/src/player.h
@@ -238,6 +238,9 @@ class Player
 			inventory.setModified(x);
 	}
 
+	// Use a function, if isDead can be defined by other conditions
+	bool isDead() { return hp == 0; }
+
 	bool touching_ground;
 	// This oscillates so that the player jumps a bit above the surface
 	bool in_liquid;