Planet
navi homePPSaboutscreenshotsdownloaddevelopmentforum

Ignore:
Timestamp:
Jan 21, 2016, 1:59:04 PM (9 years ago)
Author:
muemart
Message:

Fix some clang-tidy warnings.
Also, Serialise.h was doing some C-style casts that ended up being const casts. I moved those const casts as close to the source as possible and changed the loadAndIncrease functions to not do that.

Location:
code/trunk/src/orxonox
Files:
16 edited

Legend:

Unmodified
Added
Removed

  • code/trunk/src/orxonox/chat/ChatInputHandler.cc

    r11071 r11083  
    214214  {
    215215    /* sanity checks */
    216     if( !tocolor )
    217       orxout(internal_warning) << "Empty ListBoxTextItem given to "
    218         "ChatInputhandler::sub_setcolor()." << endl;
     216    if (!tocolor)
     217    {
     218        orxout(internal_warning) << "Empty ListBoxTextItem given to "
     219            "ChatInputhandler::sub_setcolor()." << endl;
     220        return;
     221    }
    219222
    220223    /* "hash" the name */

  • code/trunk/src/orxonox/controllers/ActionpointController.cc

    r11071 r11083  
    7474    void ActionpointController::tick(float dt)
    7575    {
    76         if (!this || !this->getControllableEntity() || !this->isActive())
     76        if (!this->getControllableEntity() || !this->isActive())
    7777            return;
    7878
    … …  
    8686        }
    8787
    88         if (!this || !this->getControllableEntity())
    89             return;
    9088        //fly
    9189        if (this->bHasTargetPosition_)
    … …  
    9896        }
    9997       
    100 
    101         if (!this || !this->getControllableEntity())
    102             return;
    10398        //don't fire rocket each tick
    10499        if (timeout_ <= 0)
    … …  
    111106        }
    112107
    113         if (!this || !this->getControllableEntity())
    114             return;
    115108        //sometimes dodge, sometimes attack
    116109        if (this->ticks_ % 80 <= 10)
    … …  
    123116        }
    124117
    125         if (!this || !this->getControllableEntity())
    126             return;
    127118        //fire if you can
    128119        if (this->bShooting_)
    … …  
    140131    void ActionpointController::action()
    141132    {
    142         if (!this || !this->getControllableEntity() || !this->isActive())
     133        if (!this->getControllableEntity() || !this->isActive())
    143134            return;
    144135
    … …  
    152143            this->startAttackingEnemiesThatAreClose();
    153144        }
    154         if (!this || !this->getControllableEntity())
    155             return;
    156145
    157146        //No action -> pop one from stack
    … …  
    161150            if (this->parsedActionpoints_.empty() && this->loopActionpoints_.empty() && this->bDefaultFightAll_)
    162151            {
    163                 if (!this || !this->getControllableEntity())
    164                     return;
    165152                Point p = { Action::FIGHTALL, "", Vector3::ZERO, false };
    166153                this->parsedActionpoints_.push_back (p);
    167154            }
    168             if (!this || !this->getControllableEntity())
    169                 return;
    170155            this->executeActionpoint();
    171             if (!this || !this->getControllableEntity())
    172                 return;
    173156            this->bTakenOver_ = false;
    174157        }
    175         if (!this || !this->getControllableEntity())
    176             return;
    177158
    178159        //Action fightall -> fight till nobody alive
    … …  
    185166                if (newTarget)
    186167                {
    187                     if (!this || !this->getControllableEntity())
    188                         return;
    189168                    this->setAction (Action::FIGHTALL, newTarget);
    190169                }
    191170                else
    192171                {
    193                     if (!this || !this->getControllableEntity())
    194                         return;
    195172                    this->nextActionpoint();
    196                     if (!this || !this->getControllableEntity())
    197                         return;
    198173                    this->executeActionpoint();
    199174   
    … …  
    207182            {
    208183                //----find a target----
    209                 ControllableEntity* newTarget = this->closestTarget(); 
    210                 if (!this || !this->getControllableEntity())
    211                     return; 
     184                ControllableEntity* newTarget = this->closestTarget();
    212185                if (newTarget &&
    213186                        CommonController::distance (this->getControllableEntity(), newTarget) < this->attackRange_)
    214187                {
    215                     if (!this || !this->getControllableEntity())
    216                         return;
    217188                    this->setAction (Action::FIGHT, newTarget);
    218189                }
    219190                else
    220191                {
    221                     if (!this || !this->getControllableEntity())
    222                         return;
    223192                    this->nextActionpoint();
    224                     if (!this || !this->getControllableEntity())
    225                         return;
    226193                    this->executeActionpoint();
    227194                }
    … …  
    235202                {
    236203                    ControllableEntity* newTarget = this->closestTarget();
    237                     if (!this || !this->getControllableEntity())
    238                         return;
    239204                    if (newTarget &&
    240205                        CommonController::distance (this->getControllableEntity(), newTarget) < this->attackRange_)
    241206                    {
    242                         if (!this || !this->getControllableEntity())
    243                             return;
    244207                        this->setAction (Action::FIGHT, newTarget);
    245208                    }
    246209                    else
    247210                    {
    248                         if (!this || !this->getControllableEntity())
    249                             return;
    250211                        this->nextActionpoint();
    251                         if (!this || !this->getControllableEntity())
    252                             return;
    253212                        this->executeActionpoint();
    254213                    }
    … …  
    260219            if (this->squaredDistanceToTarget() <= this->squaredaccuracy_)
    261220            {
    262                 if (!this || !this->getControllableEntity())
    263                     return;
    264221                this->nextActionpoint();   
    265                 if (!this || !this->getControllableEntity())
    266                     return;
    267222                this->executeActionpoint();
    268223            }
    … …  
    272227            if (!this->getProtect())
    273228            {
    274                 if (!this || !this->getControllableEntity())
    275                     return;
    276229                this->nextActionpoint();
    277                 if (!this || !this->getControllableEntity())
    278                     return;
    279230                this->executeActionpoint();
    280231            }
    281             if (!this || !this->getControllableEntity())
    282                 return;
    283232            this->stayNearProtect();
    284233        }
    285234        else if (this->action_ == Action::ATTACK)
    286235        {   
    287             if (!this || !this->getControllableEntity())
    288                 return;
    289236            if (!this->hasTarget())
    290237            {
    291                 if (!this || !this->getControllableEntity())
    292                     return;
    293238                this->nextActionpoint();
    294                 if (!this || !this->getControllableEntity())
    295                     return;
    296239                this->executeActionpoint();
    297240            }
    … …  
    408351    void ActionpointController::setAction (Action action, ControllableEntity* target)
    409352    {
    410         if (!this || !this->getControllableEntity())
     353        if (!this->getControllableEntity())
    411354            return;
    412355        this->action_ = action;
    … …  
    425368    void ActionpointController::setAction (Action action, const Vector3& target)
    426369    {
    427         if (!this || !this->getControllableEntity())
     370        if (!this->getControllableEntity())
    428371            return;
    429372        this->action_ = action;
    … …  
    436379    void ActionpointController::setAction (Action action, const Vector3& target,  const Quaternion& orient )
    437380    {
    438         if (!this || !this->getControllableEntity())
     381        if (!this->getControllableEntity())
    439382            return;
    440383        this->action_ = action;
    … …  
    454397    void ActionpointController::executeActionpoint()
    455398    {
    456         if (!this || !this->getControllableEntity())
     399        if (!this->getControllableEntity())
    457400            return;
    458401
    … …  
    473416        else
    474417        {
    475             if (!this || !this->getControllableEntity())
    476                 return;
    477 
    478418            this->setTarget(nullptr);
    479419            this->setTargetPosition(this->getControllableEntity()->getWorldPosition());
    … …  
    481421            return;
    482422        }
    483         if (!this || !this->getControllableEntity())
    484             return;
    485423        if (!this->bLoop_ && this->parsedActionpoints_.back().inLoop)
    486424        {
    487425            //MOVES all points that are in loop to a loop vector
    488426            this->fillLoop();
    489             if (!this || !this->getControllableEntity())
    490                 return;
    491427            this->bLoop_ = true;
    492428            executeActionpoint();
    493429            return;
    494430        }
    495         if (!this || !this->getControllableEntity())
    496             return;
    497431        this->setAction (p.action);
    498         if (!this || !this->getControllableEntity())
    499             return;
    500432
    501433        switch (this->action_)
    … …  
    508440                for (Pawn* pawn : ObjectList<Pawn>())
    509441                {
    510                     if (!this || !this->getControllableEntity())
    511                         return;
    512442                    if (CommonController::getName(pawn) == targetName)
    513443                    {
    … …  
    520450            {
    521451                this->setTargetPosition( p.position );
    522                 if (!this || !this->getControllableEntity())
    523                     return;
    524452                if (this->squaredDistanceToTarget() <= this->squaredaccuracy_)
    525453                {
    526                     if (!this || !this->getControllableEntity())
    527                         return;
    528454                    this->nextActionpoint();
    529                     if (!this || !this->getControllableEntity())
    530                         return;
    531455                    this->executeActionpoint();
    532456                }
    … …  
    535459            case Action::PROTECT:
    536460            {
    537                 if (!this || !this->getControllableEntity())
    538                     return;
    539461
    540462                std::string protectName = p.name;
    … …  
    582504                    if (CommonController::getName(pawn) == targetName)
    583505                    {
    584                         if (!this || !this->getControllableEntity())
    585                             return;
    586506                        this->setTarget (static_cast<ControllableEntity*>(pawn));
    587507                    }
    … …  
    590510                {
    591511                    this->nextActionpoint();
    592                     if (!this || !this->getControllableEntity())
    593                         return;
    594512                    this->executeActionpoint();
    595513                }
    … …  
    604522    void ActionpointController::stayNearProtect()
    605523    {
    606         if (!this || !this->getControllableEntity())
     524        if (!this->getControllableEntity())
    607525            return;
    608526
    … …  
    626544    void ActionpointController::nextActionpoint()
    627545    {
    628         if (!this || !this->getControllableEntity())
    629             return;
    630546        if (this->bLoop_)
    631547        {
    … …  
    653569    void ActionpointController::moveBackToTop()
    654570    {
    655         if (!this || !this->getControllableEntity())
     571        if (!this->getControllableEntity())
    656572            return;
    657573
    … …  
    684600    void ActionpointController::takeActionpoints (const std::vector<Point>& vector, const std::vector<Point>& loop, bool b)
    685601    {
    686         if (!this || !this->getControllableEntity())
     602        if (!this->getControllableEntity())
    687603            return;
    688604        this->parsedActionpoints_ = vector;
    689         if (!this || !this->getControllableEntity())
    690             return;
    691605        this->loopActionpoints_ = loop;
    692606        this->bLoop_ = b;
    … …  
    701615    Pawn* ActionpointController::closestTarget()
    702616    {
    703         if (!this || !this->getControllableEntity())
     617        if (!this->getControllableEntity())
    704618            return nullptr;
    705619
    … …  
    709623        for (Pawn* pawn : ObjectList<Pawn>())
    710624        {
    711             if (!this || !this->getControllableEntity())
    712                 return nullptr;
    713625            if ( CommonController::sameTeam (this->getControllableEntity(), static_cast<ControllableEntity*>(pawn), gt) )
    714626                continue;
    … …  
    730642    void ActionpointController::startAttackingEnemiesThatAreClose()
    731643    {
    732         if (!this || !this->getControllableEntity())
     644        if (!this->getControllableEntity())
    733645            return;
    734646       
    735647        if (!this->target_ || (this->target_ && CommonController::distance (this->getControllableEntity(), this->target_) > this->attackRange_))
    736648        {
    737             if (!this || !this->getControllableEntity())
    738                 return;
    739649            Pawn* newTarget = this->closestTarget();
    740650            if ( newTarget &&
    … …  
    742652                    <= this->attackRange_ )
    743653            {
    744                 if (!this || !this->getControllableEntity())
    745                     return;
    746654                this->setTarget(newTarget);
    747655                if (this->bLoop_ && !this->bPatrolling_)

  • code/trunk/src/orxonox/controllers/DivisionController.cc

    r11071 r11083  
    5757    void DivisionController::action()
    5858    {   
    59         if (!this || !this->getControllableEntity() || !this->isActive())
     59        if (!this->getControllableEntity() || !this->isActive())
    6060            return;
    6161       
    6262        ActionpointController::action();
    63         if (!this || !this->getControllableEntity())
    64             return;
    6563        if (!(this->parsedActionpoints_.empty() && this->loopActionpoints_.empty()))
    6664        {

  • code/trunk/src/orxonox/controllers/FightingController.cc

    r11071 r11083  
    5858    void FightingController::lookAtTarget(float dt)
    5959    {
    60         if (!this || !this->getControllableEntity())
     60        if (!this->getControllableEntity())
    6161            return;
    6262        ControllableEntity* entity = this->getControllableEntity();
    … …  
    124124        Vector3 diffUnit = diffVector/diffLength;
    125125
    126         if (!this || !this->getControllableEntity())
    127             return;
    128 
    129126        //too far? well, come closer then
    130127        if (diffLength > this->attackRange_)
    … …  
    142139            this->bKeepFormation_ = false;
    143140
    144             if (!this || !this->getControllableEntity())
    145                 return;
    146141            if (!this->bDodge_)
    147142            {
    … …  
    217212    float FightingController::squaredDistanceToTarget()  const
    218213    {
    219         if (!this || !this->getControllableEntity())
     214        if (!this->getControllableEntity())
    220215            return 0;
    221216        if (!this->target_ || !this->getControllableEntity())
    … …  
    237232    Pawn* FightingController::closestTarget() const
    238233    {
    239         if (!this || !this->getControllableEntity())
     234        if (!this->getControllableEntity())
    240235            return nullptr;
    241236

  • code/trunk/src/orxonox/controllers/ScriptController.cc

    r11071 r11083  
    227227          {
    228228            // Specify the axis
    229             Vector3* a;
     229            Vector3 a;
    230230              switch ((int) currentEvent.d) {
    231231                case 3:
    232                   a = new Vector3(this->currentEvent.v1.x + this->currentEvent.e*cos(2*math::pi*dl),
     232                  a = Vector3(this->currentEvent.v1.x + this->currentEvent.e*cos(2*math::pi*dl),
    233233                                  this->currentEvent.v1.y + this->currentEvent.e*sin(2*math::pi*dl),
    234234                                  this->currentEvent.v1.z);
    235235                break;
    236236                case 2:
    237                   a = new Vector3(this->currentEvent.v1.x + this->currentEvent.e*sin(2*math::pi*dl),
     237                  a = Vector3(this->currentEvent.v1.x + this->currentEvent.e*sin(2*math::pi*dl),
    238238                                  this->currentEvent.v1.y,
    239239                                  this->currentEvent.v1.z + this->currentEvent.e*cos(2*math::pi*dl));
    240240                break;
    241241                case 1:
    242                   a = new Vector3(this->currentEvent.v1.x,
     242                  a = Vector3(this->currentEvent.v1.x,
    243243                                  this->currentEvent.v1.y + this->currentEvent.e*sin(2*math::pi*dl),
    244244                                  this->currentEvent.v1.z + this->currentEvent.e*cos(2*math::pi*dl));
    … …  
    246246              }
    247247
    248             this->entity_->setPosition(*a);
     248            this->entity_->setPosition(a);
    249249
    250250            /* Look at the specified position */

  • code/trunk/src/orxonox/controllers/SectionController.cc

    r11071 r11083  
    5959    void SectionController::action()
    6060    {
    61         if (!this || !this->getControllableEntity() || !this->isActive())
     61        if (!this->getControllableEntity() || !this->isActive())
    6262            return;
    6363
    … …  
    6666        {
    6767            ActionpointController* newDivisionLeader = findNewDivisionLeader();
    68             if (!this || !this->getControllableEntity())
    69                 return;
    7068
    7169            this->myDivisionLeader_ = newDivisionLeader;
    … …  
    7876        {
    7977            ActionpointController::action();
    80             if (!this || !this->getControllableEntity())
    81                 return;
    8278            if (!(this->parsedActionpoints_.empty() && this->loopActionpoints_.empty()))
    8379            {
    … …  
    9894            else if (!this->myDivisionLeader_->bKeepFormation_)
    9995            {
    100                 if (!this || !this->getControllableEntity())
    101                     return;
    10296
    10397                if (!this->hasTarget())
    … …  
    121115        if (action == Action::FIGHT || action == Action::FIGHTALL || action == Action::ATTACK)
    122116        {
    123             Pawn* target;
     117            Pawn* target = nullptr;
    124118            //----if he has a target----
    125119            if (this->myDivisionLeader_->hasTarget())

  • code/trunk/src/orxonox/controllers/WingmanController.cc

    r11071 r11083  
    5858    void WingmanController::action()
    5959    {
    60         if (!this || !this->getControllableEntity() || !this->isActive())
     60        if (!this->getControllableEntity() || !this->isActive())
    6161            return;
    6262        //----If no leader, find one----
    … …  
    6464        {
    6565            ActionpointController* newLeader = (findNewLeader());
    66             if (!this || !this->getControllableEntity())
    67                 return;
    6866
    6967            this->myLeader_ = newLeader;
    … …  
    9290            else if (!this->myLeader_->bKeepFormation_)
    9391            {
    94                 if (!this || !this->getControllableEntity())
    95                     return;
    9692
    9793                if (!this->hasTarget())

  • code/trunk/src/orxonox/gametypes/LastManStanding.cc

    r11071 r11083  
    8888                if (it->first->getClientID()== NETWORK_PEER_ID_UNKNOWN)
    8989                    return true;
    90                 const std::string& message = ""; // resets Camper-Warning-message
     90                const std::string message = ""; // resets Camper-Warning-message
    9191                this->gtinfo_->sendFadingMessage(message,it->first->getClientID());
    9292            }
    … …  
    195195            if (it->first->getClientID()== NETWORK_PEER_ID_UNKNOWN)
    196196                return;
    197             const std::string& message = ""; // resets Camper-Warning-message
     197            const std::string message = ""; // resets Camper-Warning-message
    198198            this->gtinfo_->sendFadingMessage(message,it->first->getClientID());
    199199        }
    … …  
    262262                        if (mapEntry.first->getClientID()== NETWORK_PEER_ID_UNKNOWN)
    263263                            return;
    264                         const std::string& message = ""; // resets Camper-Warning-message
     264                        const std::string message = ""; // resets Camper-Warning-message
    265265                        this->gtinfo_->sendFadingMessage(message,mapEntry.first->getClientID());
    266266                    }
    … …  
    270270                    if (mapEntry.first->getClientID()== NETWORK_PEER_ID_UNKNOWN)
    271271                        continue;
    272                     const std::string& message = "Camper Warning! Don't forget to shoot.";
     272                    const std::string message = "Camper Warning! Don't forget to shoot.";
    273273                    this->gtinfo_->sendFadingMessage(message,mapEntry.first->getClientID());
    274274                }

  • code/trunk/src/orxonox/gametypes/LastTeamStanding.cc

    r11071 r11083  
    136136                if (it->first->getClientID()== NETWORK_PEER_ID_UNKNOWN)
    137137                    return true;
    138                 const std::string& message = ""; // resets Camper-Warning-message
     138                const std::string message = ""; // resets Camper-Warning-message
    139139                this->gtinfo_->sendFadingMessage(message,it->first->getClientID());
    140140            }
    … …  
    170170            if (it->first->getClientID()== NETWORK_PEER_ID_UNKNOWN)
    171171                return;
    172             const std::string& message = ""; // resets Camper-Warning-message
     172            const std::string message = ""; // resets Camper-Warning-message
    173173            this->gtinfo_->sendFadingMessage(message,it->first->getClientID());
    174174        }
    … …  
    209209                        if (mapEntry.first->getClientID() == NETWORK_PEER_ID_UNKNOWN)
    210210                            return;
    211                         const std::string& message = ""; // resets Camper-Warning-message
     211                        const std::string message = ""; // resets Camper-Warning-message
    212212                        this->gtinfo_->sendFadingMessage(message, mapEntry.first->getClientID());
    213213                    }
    … …  
    217217                  if (mapEntry.first->getClientID()== NETWORK_PEER_ID_UNKNOWN)
    218218                        continue;
    219                     const std::string& message = "Camper Warning! Don't forget to shoot.";
     219                    const std::string message = "Camper Warning! Don't forget to shoot.";
    220220                    this->gtinfo_->sendFadingMessage(message, mapEntry.first->getClientID());
    221221                }

  • code/trunk/src/orxonox/graphics/LensFlare.cc

    r11080 r11083  
    3434#include "LensFlare.h"
    3535
     36#include "core/CoreIncludes.h"
    3637#include "core/XMLPort.h"
    3738#include "graphics/Billboard.h"
    3839#include "CameraManager.h"
     40#include "Camera.h"
    3941#include "RenderQueueListener.h"
     42#include "Scene.h"
    4043
    4144#include <OgreSphere.h>
    4245#include <OgreRenderWindow.h>
     46#include <OgreCamera.h>
    4347
    4448namespace orxonox

  • code/trunk/src/orxonox/graphics/LensFlare.h

    r11080 r11083  
    4141#include "core/GraphicsManager.h"
    4242#include "util/Math.h"
     43#include "tools/interfaces/Tickable.h"
    4344#include "worldentities/StaticEntity.h"
    4445#include "graphics/Billboard.h"

  • code/trunk/src/orxonox/weaponsystem/Munition.cc

    r11071 r11083  
    544544        if (bUseReloadTime && munition->reloadTime_ > 0 && munition->deployment_ != MunitionDeployment::Stack)
    545545        {
    546             const ExecutorPtr& executor = createExecutor(createFunctor(&Magazine::loaded, this));
     546            const ExecutorPtr executor = createExecutor(createFunctor(&Magazine::loaded, this));
    547547            executor->setDefaultValues(munition);
    548548

  • code/trunk/src/orxonox/worldentities/StaticEntity.cc

    r11071 r11083  
    5454    void StaticEntity::registerVariables()
    5555    {
    56         registerVariable(this->getPosition(),    VariableDirection::ToClient, new NetworkCallback<StaticEntity>(this, &StaticEntity::positionChanged));
    57         registerVariable(this->getOrientation(), VariableDirection::ToClient, new NetworkCallback<StaticEntity>(this, &StaticEntity::orientationChanged));
     56        // Ugly const casts, but are valid because position and orientation are not actually const
     57        registerVariable(const_cast<Vector3&>(this->getPosition()),    VariableDirection::ToClient, new NetworkCallback<StaticEntity>(this, &StaticEntity::positionChanged));
     58        registerVariable(const_cast<Quaternion&>(this->getOrientation()), VariableDirection::ToClient, new NetworkCallback<StaticEntity>(this, &StaticEntity::orientationChanged));
    5859    }
    5960

  • code/trunk/src/orxonox/worldentities/WorldEntity.cc

    r11071 r11083  
    190190        registerVariable(this->bVisible_,       VariableDirection::ToClient, new NetworkCallback<WorldEntity>(this, &WorldEntity::changedVisibility));
    191191
    192         registerVariable(this->getScale3D(),    VariableDirection::ToClient, new NetworkCallback<WorldEntity>(this, &WorldEntity::scaleChanged));
     192        // Ugly const cast, but is valid because the scale is not actually const
     193        registerVariable(const_cast<Vector3&>(this->getScale3D()),    VariableDirection::ToClient, new NetworkCallback<WorldEntity>(this, &WorldEntity::scaleChanged));
    193194
    194195        // Physics stuff

  • code/trunk/src/orxonox/worldentities/pawns/FpsPlayer.h

    r11071 r11083  
    4646            virtual ~FpsPlayer();
    4747
    48             virtual void XMLPort(Element& xmlelement, XMLPort::Mode mode);
    49             virtual void tick(float dt);
     48            virtual void XMLPort(Element& xmlelement, XMLPort::Mode mode) override;
     49            virtual void tick(float dt) override;
    5050            void registerVariables();
    5151            void setConfigValues();
    5252
    53             virtual void moveFrontBack(const Vector2& value);
    54             virtual void moveRightLeft(const Vector2& value);
    55             virtual void moveUpDown(const Vector2& value);
     53            virtual void moveFrontBack(const Vector2& value) override;
     54            virtual void moveRightLeft(const Vector2& value) override;
     55            virtual void moveUpDown(const Vector2& value) override;
    5656
    57             virtual void rotateYaw(const Vector2& value);
    58             virtual void rotatePitch(const Vector2& value);
    59             virtual void rotateRoll(const Vector2& value);
     57            virtual void rotateYaw(const Vector2& value) override;
     58            virtual void rotatePitch(const Vector2& value) override;
     59            virtual void rotateRoll(const Vector2& value) override;
    6060
    6161
    … …  
    6565                { return this->meshSrc_; }
    6666
    67             void boost(bool bBoost); //acctually jump
     67            void boost(bool bBoost) override; //actually jump
    6868
    6969            virtual void fire();
    … …  
    7171            virtual bool collidesAgainst(WorldEntity* otherObject, const btCollisionShape* ownCollisionShape, btManifoldPoint& contactPoint) override;
    7272
    73             virtual void addedWeaponPack(WeaponPack* wPack);
     73            virtual void addedWeaponPack(WeaponPack* wPack) override;
    7474
    7575        protected:
    76             virtual void setPlayer(PlayerInfo* player);
    77             virtual void startLocalHumanControl();
     76            virtual void setPlayer(PlayerInfo* player) override;
     77            virtual void startLocalHumanControl() override;
    7878            bool bInvertYAxis_;
    7979
    … …  
    8989
    9090        private:
    91             virtual bool isCollisionTypeLegal(WorldEntity::CollisionType type) const;
     91            virtual bool isCollisionTypeLegal(WorldEntity::CollisionType type) const override;
    9292            float speed_;
    9393

  • code/trunk/src/orxonox/worldentities/pawns/SpaceShip.cc

    r11071 r11083  
    235235
    236236        // This function call adds a lift to the ship when it is rotating to make it's movement more "realistic" and enhance the feeling.
    237         if (this->getLocalVelocity().z < 0 && abs(this->getLocalVelocity().z) < stallSpeed_)
    238             this->moveRightLeft(-lift_ / 5.0f * value * sqrt(abs(this->getLocalVelocity().z)));
     237        if (this->getLocalVelocity().z < 0 && std::abs(this->getLocalVelocity().z) < stallSpeed_)
     238            this->moveRightLeft(-lift_ / 5.0f * value * sqrt(std::abs(this->getLocalVelocity().z)));
    239239    }
    240240
    … …  
    257257
    258258        // This function call adds a lift to the ship when it is pitching to make it's movement more "realistic" and enhance the feeling.
    259         if (this->getLocalVelocity().z < 0 && abs(this->getLocalVelocity().z) < stallSpeed_)
    260             this->moveUpDown(lift_ / 5.0f * pitch * sqrt(abs(this->getLocalVelocity().z)));
     259        if (this->getLocalVelocity().z < 0 && std::abs(this->getLocalVelocity().z) < stallSpeed_)
     260            this->moveUpDown(lift_ / 5.0f * pitch * sqrt(std::abs(this->getLocalVelocity().z)));
    261261    }
    262262
    … …  
    505505            this->shakeDt_ += dt;
    506506
    507             float frequency = this->shakeFrequency_ * (square(abs(this->getLocalVelocity().z)));
     507            float frequency = this->shakeFrequency_ * (square(std::abs(this->getLocalVelocity().z)));
    508508
    509509            if (this->shakeDt_ >= 1.0f/frequency)
Note: See TracChangeset for help on using the changeset viewer.