Browse Source

QOC

Code cleaning of unnecessary else'
Remove unnecessary dynamic_casts
marauder2k7 3 years ago
parent
commit
57aceb60fb

+ 1 - 1
Engine/source/T3D/fx/explosion.cpp

@@ -1382,7 +1382,7 @@ bool Explosion::explode()
       resetWorldBox();
       resetWorldBox();
    }
    }
 
 
-   SFXProfile* sound_prof = dynamic_cast<SFXProfile*>(mDataBlock->getSFXProfile());
+   SFXProfile* sound_prof = mDataBlock->getSFXProfile();
    if (sound_prof)
    if (sound_prof)
    {
    {
       soundProfile_clone = sound_prof->cloneAndPerformSubstitutions(ss_object, ss_index);
       soundProfile_clone = sound_prof->cloneAndPerformSubstitutions(ss_object, ss_index);

+ 2 - 2
Engine/source/T3D/fx/explosion.h

@@ -76,8 +76,8 @@ class ExplosionData : public GameBaseData {
    SFXProfile* getSFXProfile() {
    SFXProfile* getSFXProfile() {
       if (mSoundAsset.notNull())
       if (mSoundAsset.notNull())
          return mSoundAsset->getSfxProfile();
          return mSoundAsset->getSfxProfile();
-      else
-         return NULL;
+
+      return NULL;
    }
    }
 
 
    ParticleEmitterData* particleEmitter;
    ParticleEmitterData* particleEmitter;

+ 2 - 0
Engine/source/T3D/fx/lightning.cpp

@@ -241,7 +241,9 @@ LightningData::LightningData()
    INIT_SOUNDASSET(StrikeSound);
    INIT_SOUNDASSET(StrikeSound);
 
 
    for (S32 i = 0; i < MaxThunders; i++)
    for (S32 i = 0; i < MaxThunders; i++)
+   {
       INIT_SOUNDASSET_ARRAY(ThunderSound, MaxThunders);
       INIT_SOUNDASSET_ARRAY(ThunderSound, MaxThunders);
+   }
 
 
    for (S32 i = 0; i < MaxTextures; i++)
    for (S32 i = 0; i < MaxTextures; i++)
    {
    {

+ 4 - 4
Engine/source/T3D/fx/lightning.h

@@ -97,15 +97,15 @@ class LightningData : public GameBaseData
    {
    {
       if (mThunderSoundAsset[id] != NULL)
       if (mThunderSoundAsset[id] != NULL)
          return mThunderSoundAsset[id]->getSfxProfile();
          return mThunderSoundAsset[id]->getSfxProfile();
-      else
-         return NULL;
+
+      return NULL;
    }
    }
 
 
    SFXProfile* getSFXProfile() {
    SFXProfile* getSFXProfile() {
       if (mStrikeSoundAsset.notNull())
       if (mStrikeSoundAsset.notNull())
          return mStrikeSoundAsset->getSfxProfile();
          return mStrikeSoundAsset->getSfxProfile();
-      else
-         return NULL;
+
+      return NULL;
    }
    }
 
 
 };
 };

+ 2 - 2
Engine/source/T3D/fx/precipitation.h

@@ -75,8 +75,8 @@ class PrecipitationData : public GameBaseData
       SFXProfile* getSFXProfile() {
       SFXProfile* getSFXProfile() {
          if (mSoundAsset.notNull())
          if (mSoundAsset.notNull())
             return mSoundAsset->getSfxProfile();
             return mSoundAsset->getSfxProfile();
-         else
-            return NULL;
+
+         return NULL;
       }
       }
 };
 };
 
 

+ 3 - 1
Engine/source/T3D/fx/splash.cpp

@@ -97,7 +97,9 @@ SplashData::SplashData()
 
 
    U32 i;
    U32 i;
    for (i = 0; i < NUM_TEX; i++)
    for (i = 0; i < NUM_TEX; i++)
+   {
       INIT_IMAGEASSET_ARRAY(Texture, i);
       INIT_IMAGEASSET_ARRAY(Texture, i);
+   }
 
 
    for( i=0; i<NUM_TIME_KEYS; i++ )
    for( i=0; i<NUM_TIME_KEYS; i++ )
       times[i] = 1.0;
       times[i] = 1.0;
@@ -679,7 +681,7 @@ void Splash::spawnExplosion()
 
 
    /// could just play the explosion one, but explosion could be weapon specific,
    /// could just play the explosion one, but explosion could be weapon specific,
    /// splash sound could be liquid specific. food for thought.
    /// splash sound could be liquid specific. food for thought.
-   SFXProfile* sound_prof = dynamic_cast<SFXProfile*>(mDataBlock->getSFXProfile());
+   SFXProfile* sound_prof = mDataBlock->getSFXProfile();
    if (sound_prof)
    if (sound_prof)
    {
    {
       SFX->playOnce(sound_prof, &getTransform());
       SFX->playOnce(sound_prof, &getTransform());

+ 2 - 2
Engine/source/T3D/fx/splash.h

@@ -102,8 +102,8 @@ public:
    SFXProfile* getSFXProfile() {
    SFXProfile* getSFXProfile() {
       if (mSoundAsset.notNull())
       if (mSoundAsset.notNull())
          return mSoundAsset->getSfxProfile();
          return mSoundAsset->getSfxProfile();
-      else
-         return NULL;
+
+      return NULL;
    }
    }
 
 
    ParticleEmitterData*    emitterList[NUM_EMITTERS];
    ParticleEmitterData*    emitterList[NUM_EMITTERS];

+ 3 - 2
Engine/source/T3D/projectile.h

@@ -118,11 +118,12 @@ public:
 
 
    DECLARE_SOUNDASSET(ProjectileData, ProjectileSound);
    DECLARE_SOUNDASSET(ProjectileData, ProjectileSound);
    DECLARE_SOUNDASSET_SETGET(ProjectileData, ProjectileSound);
    DECLARE_SOUNDASSET_SETGET(ProjectileData, ProjectileSound);
+
    SFXProfile* getSFXProfile() {
    SFXProfile* getSFXProfile() {
       if (mProjectileSoundAsset.notNull())
       if (mProjectileSoundAsset.notNull())
          return mProjectileSoundAsset->getSfxProfile();
          return mProjectileSoundAsset->getSfxProfile();
-      else
-         return NULL;
+
+      return NULL;
    }
    }
    
    
    LightDescription *lightDesc;
    LightDescription *lightDesc;

+ 4 - 4
Engine/source/T3D/rigidShape.h

@@ -70,8 +70,8 @@ class RigidShapeData : public ShapeBaseData
    {
    {
       if (mBodySoundsAsset[id] != NULL)
       if (mBodySoundsAsset[id] != NULL)
          return mBodySoundsAsset[id]->getSfxProfile();
          return mBodySoundsAsset[id]->getSfxProfile();
-      else
-         return NULL;
+
+      return NULL;
    }
    }
 
 
    enum RigidShapeConsts
    enum RigidShapeConsts
@@ -98,8 +98,8 @@ class RigidShapeData : public ShapeBaseData
    {
    {
       if (mWaterSoundsAsset[id] != NULL)
       if (mWaterSoundsAsset[id] != NULL)
          return mWaterSoundsAsset[id]->getSfxProfile();
          return mWaterSoundsAsset[id]->getSfxProfile();
-      else
-         return NULL;
+
+      return NULL;
    }
    }
 
 
    F32 exitSplashSoundVel;
    F32 exitSplashSoundVel;

+ 2 - 2
Engine/source/T3D/vehicles/flyingVehicle.h

@@ -51,8 +51,8 @@ struct FlyingVehicleData: public VehicleData {
    {
    {
       if (mFlyingSoundsAsset[id] != NULL)
       if (mFlyingSoundsAsset[id] != NULL)
          return mFlyingSoundsAsset[id]->getSfxProfile();
          return mFlyingSoundsAsset[id]->getSfxProfile();
-      else
-         return NULL;
+
+      return NULL;
    }
    }
 
 
    enum Jets {
    enum Jets {

+ 2 - 2
Engine/source/T3D/vehicles/hoverVehicle.h

@@ -52,8 +52,8 @@ class HoverVehicleData : public VehicleData
    {
    {
       if (mHoverSoundsAsset[id] != NULL)
       if (mHoverSoundsAsset[id] != NULL)
          return mHoverSoundsAsset[id]->getSfxProfile();
          return mHoverSoundsAsset[id]->getSfxProfile();
-      else
-         return NULL;
+
+      return NULL;
    }
    }
 
 
    enum Jets {
    enum Jets {

+ 4 - 4
Engine/source/T3D/vehicles/vehicle.h

@@ -56,8 +56,8 @@ struct VehicleData : public RigidShapeData
    {
    {
       if (mVehicleBodySoundsAsset[id] != NULL)
       if (mVehicleBodySoundsAsset[id] != NULL)
          return mVehicleBodySoundsAsset[id]->getSfxProfile();
          return mVehicleBodySoundsAsset[id]->getSfxProfile();
-      else
-         return NULL;
+
+      return NULL;
    }
    }
 
 
 
 
@@ -88,8 +88,8 @@ struct VehicleData : public RigidShapeData
   {
   {
      if (mVehicleWaterSoundsAsset[id] != NULL)
      if (mVehicleWaterSoundsAsset[id] != NULL)
         return mVehicleWaterSoundsAsset[id]->getSfxProfile();
         return mVehicleWaterSoundsAsset[id]->getSfxProfile();
-     else
-        return NULL;
+
+     return NULL;
   }
   }
 
 
    F32 exitSplashSoundVel;
    F32 exitSplashSoundVel;

+ 2 - 2
Engine/source/T3D/vehicles/wheeledVehicle.h

@@ -125,8 +125,8 @@ struct WheeledVehicleData: public VehicleData
    {
    {
       if (mWheeledVehicleSoundsAsset[id] != NULL)
       if (mWheeledVehicleSoundsAsset[id] != NULL)
          return mWheeledVehicleSoundsAsset[id]->getSfxProfile();
          return mWheeledVehicleSoundsAsset[id]->getSfxProfile();
-      else
-         return NULL;
+
+      return NULL;
    }
    }