summaryrefslogtreecommitdiffstats
path: root/src/Entities/Entity.cpp
diff options
context:
space:
mode:
authorarchshift <admin@archshift.com>2014-08-03 07:35:29 +0200
committerarchshift <admin@archshift.com>2014-08-03 07:35:29 +0200
commit9ecce2366e81ae66c77312220e378ff2f1b02a24 (patch)
tree49d909ee4e16da9cc8c146aa968052c2480c16a7 /src/Entities/Entity.cpp
parentRemoved unused cPlayer::FoodPoison function (diff)
downloadcuberite-9ecce2366e81ae66c77312220e378ff2f1b02a24.tar
cuberite-9ecce2366e81ae66c77312220e378ff2f1b02a24.tar.gz
cuberite-9ecce2366e81ae66c77312220e378ff2f1b02a24.tar.bz2
cuberite-9ecce2366e81ae66c77312220e378ff2f1b02a24.tar.lz
cuberite-9ecce2366e81ae66c77312220e378ff2f1b02a24.tar.xz
cuberite-9ecce2366e81ae66c77312220e378ff2f1b02a24.tar.zst
cuberite-9ecce2366e81ae66c77312220e378ff2f1b02a24.zip
Diffstat (limited to '')
-rw-r--r--src/Entities/Entity.cpp28
1 files changed, 9 insertions, 19 deletions
diff --git a/src/Entities/Entity.cpp b/src/Entities/Entity.cpp
index c2727e6a1..042e3b868 100644
--- a/src/Entities/Entity.cpp
+++ b/src/Entities/Entity.cpp
@@ -543,10 +543,7 @@ void cEntity::KilledBy(TakeDamageInfo & a_TDI)
void cEntity::Heal(int a_HitPoints)
{
m_Health += a_HitPoints;
- if (m_Health > m_MaxHealth)
- {
- m_Health = m_MaxHealth;
- }
+ m_Health = std::min(m_Health, m_MaxHealth);
}
@@ -555,7 +552,7 @@ void cEntity::Heal(int a_HitPoints)
void cEntity::SetHealth(int a_Health)
{
- m_Health = std::max(0, std::min(m_MaxHealth, a_Health));
+ m_Health = Clamp(a_Health, 0, m_MaxHealth);
}
@@ -1264,10 +1261,10 @@ void cEntity::HandleAir(void)
SetSpeedY(1); // Float in the water
}
- // Either reduce air level or damage player
- if (m_AirLevel < 1)
+ if (m_AirLevel <= 0)
{
- if (m_AirTickTimer < 1)
+ // Either reduce air level or damage player
+ if (m_AirTickTimer <= 0)
{
// Damage player
TakeDamage(dtDrowning, NULL, 1, 1, 0);
@@ -1552,17 +1549,10 @@ void cEntity::SetHeight(double a_Height)
void cEntity::SetMass(double a_Mass)
{
- if (a_Mass > 0)
- {
- m_Mass = a_Mass;
- }
- else
- {
- // Make sure that mass is not zero. 1g is the default because we
- // have to choose a number. It's perfectly legal to have a mass
- // less than 1g as long as is NOT equal or less than zero.
- m_Mass = 0.001;
- }
+ // Make sure that mass is not zero. 1g is the default because we
+ // have to choose a number. It's perfectly legal to have a mass
+ // less than 1g as long as is NOT equal or less than zero.
+ m_Mass = std::max(a_Mass, 0.001);
}