Added convenient and efficient Timer class

The Timer is efficient because it does not depend on incrementing a
counter to keep track of time, nor does it call SDL_GetTicks every time
its state is checked (this happens once per frame instead).

Along with global functions Time::absoluteTimeMs() and
Time::deltaTimeMs(), this replaces previous globals tick_time, cur_time
and get_elapsed_time().

For now, there is still a fixed 100 times per second logic call rate,
but the new Time::deltaTimeMs() function should allow getting rid of
this.

1 files changed, 3 insertions, 7 deletions

diff --git a/src/being.cpp b/src/being.cpp
index 95f7c0d5..88104d74 100644
--- a/src/being.cpp
+++ b/src/being.cpp
@@ -643,7 +643,7 @@ void Being::setAction(Action action, int attackId)
     }
 
     if (currentAction != SpriteAction::MOVE)
-        mActionTime = tick_time;
+        mActionTimer.set();
 }
 
 void Being::lookAt(const Vector &destPos)
@@ -865,13 +865,9 @@ void Being::logic()
 
     // Remove it after 1.5 secs if the dead animation isn't long enough,
     // or simply play it until it's finished.
-    if (!isAlive() && Net::getGameHandler()->removeDeadBeings() &&
-        get_elapsed_time(mActionTime) > std::max(getDuration(), 1500))
-    {
-
-        if (getType() != PLAYER)
+    if (!isAlive() && Net::getGameHandler()->removeDeadBeings() && getType() != PLAYER)
+        if (mActionTimer.elapsed() > std::max(getDuration(), 1500))
             actorSpriteManager->scheduleDelete(this);
-    }
 }
 
 void Being::drawSpeech(int offsetX, int offsetY)