Some code style cleanups in AutoAttack class

1 files changed, 18 insertions, 6 deletions

diff --git a/src/game-server/autoattack.cpp b/src/game-server/autoattack.cpp
index b8c8a165..d433b421 100644
--- a/src/game-server/autoattack.cpp
+++ b/src/game-server/autoattack.cpp
@@ -20,9 +20,9 @@
 
 #include "autoattack.h"
 
-void AutoAttacks::add(AutoAttack n)
+void AutoAttacks::add(const AutoAttack &autoAttack)
 {
-    mAutoAttacks.push_back(n);
+    mAutoAttacks.push_back(autoAttack);
     // Slow, but safe.
     mAutoAttacks.sort();
 }
@@ -34,26 +34,38 @@ void AutoAttacks::clear()
 
 void AutoAttacks::stop()
 {
-    for (std::list<AutoAttack>::iterator it = mAutoAttacks.begin(); it != mAutoAttacks.end(); ++it)
+    for (std::list<AutoAttack>::iterator it = mAutoAttacks.begin();
+         it != mAutoAttacks.end(); ++it)
+    {
         it->halt();
+    }
     mActive = false;
 }
 
 void AutoAttacks::start()
 {
-    for (std::list<AutoAttack>::iterator it = mAutoAttacks.begin(); it != mAutoAttacks.end(); ++it)
+    for (std::list<AutoAttack>::iterator it = mAutoAttacks.begin();
+         it != mAutoAttacks.end(); ++it)
+    {
         it->softReset();
+    }
     mActive = true;
 }
 
 void AutoAttacks::tick(std::list<AutoAttack> *ret)
 {
-    for (std::list<AutoAttack>::iterator it = mAutoAttacks.begin(); it != mAutoAttacks.end(); ++it)
+    for (std::list<AutoAttack>::iterator it = mAutoAttacks.begin();
+         it != mAutoAttacks.end(); ++it)
+    {
         if (it->tick()) {
             if (mActive)
                 it->reset();
             else
                 it->halt();
-        } else if (ret && it->isReady())
+        }
+        else if (ret && it->isReady())
+        {
             ret->push_back(*it);
+        }
+    }
 }