Remove unnecessary returns

[xonotic/xonotic-data.pk3dir.git] / qcsrc / server / mutators / mutator / gamemode_assault.qc

diff --git a/qcsrc/server/mutators/mutator/gamemode_assault.qc b/qcsrc/server/mutators/mutator/gamemode_assault.qc
index a377d0368d607a6fa5040d6a0366396a335303dc..7c32b077ab87d70ae69cf83983b7eb562383fe81 100644 (file)
--- a/qcsrc/server/mutators/mutator/gamemode_assault.qc
+++ b/qcsrc/server/mutators/mutator/gamemode_assault.qc
@@ -232,11 +232,6 @@ void assault_roundstart_use_this(entity this)
 {
        assault_roundstart_use(this, NULL, NULL);
 }
-void assault_roundstart_use_self()
-{
-    SELFPARAM();
-    assault_roundstart_use(this, NULL, NULL);
-}
 
 void assault_wall_think(entity this)
 {
@@ -258,12 +253,12 @@ void assault_wall_think(entity this)
 // reset objectives, toggle spawnpoints, reset triggers, ...
 void vehicles_clearreturn(entity veh);
 void vehicles_spawn(entity this);
-void assault_new_round()
-{SELFPARAM();
+void assault_new_round(entity this)
+{
        //bprint("ASSAULT: new round\n");
 
        // Eject players from vehicles
-    FOREACH_CLIENT(IS_PLAYER(it) && it.vehicle, WITHSELF(it, vehicles_exit(VHEF_RELEASE)));
+    FOREACH_CLIENT(IS_PLAYER(it) && it.vehicle, vehicles_exit(it.vehicle, VHEF_RELEASE));
 
     FOREACH_ENTITY_FLAGS(vehicle_flags, VHF_ISVEHICLE, LAMBDA(
         vehicles_clearreturn(it);
@@ -271,7 +266,7 @@ void assault_new_round()
     ));
 
        // up round counter
-       self.winning = self.winning + 1;
+       this.winning = this.winning + 1;
 
        // swap attacker/defender roles
        if(assault_attacker_team == NUM_TEAM_1)
@@ -293,7 +288,7 @@ void assault_new_round()
 // they win. Otherwise the defending team wins once the timelimit passes.
 int WinningCondition_Assault()
 {
-       WinningConditionHelper(); // set worldstatus
+       WinningConditionHelper(NULL); // set worldstatus
 
        int status = WINNING_NO;
        // as the timelimit has not yet passed just assume the defending team will win
@@ -323,7 +318,7 @@ int WinningCondition_Assault()
                        }
                        else
                        {
-                               WITHSELF(ent, assault_new_round());
+                               assault_new_round(ent);
                        }
                }
        }
@@ -424,7 +419,7 @@ spawnfunc(target_assault_roundstart)
        assault_attacker_team = NUM_TEAM_1;
        this.classname = "target_assault_roundstart";
        this.use = assault_roundstart_use;
-       this.reset2 = assault_roundstart_use_self;
+       this.reset2 = assault_roundstart_use_this;
        InitializeEntity(this, assault_roundstart_use_this, INITPRIO_FINDTARGET);
 }
 
@@ -620,8 +615,6 @@ MUTATOR_HOOKFUNCTION(as, PlayerSpawn)
                Send_Notification(NOTIF_ONE, player, MSG_CENTER, CENTER_ASSAULT_ATTACKING);
        else
                Send_Notification(NOTIF_ONE, player, MSG_CENTER, CENTER_ASSAULT_DEFENDING);
-
-       return false;
 }
 
 MUTATOR_HOOKFUNCTION(as, TurretSpawn)
@@ -630,20 +623,20 @@ MUTATOR_HOOKFUNCTION(as, TurretSpawn)
 
        if(!turret.team || turret.team == MAX_SHOT_DISTANCE)
                turret.team = 5; // this gets reversed when match starts?
-
-       return false;
 }
 
 MUTATOR_HOOKFUNCTION(as, VehicleSpawn)
-{SELFPARAM();
-       self.nextthink = time + 0.5;
+{
+       entity veh = M_ARGV(0, entity);
 
-       return false;
+       veh.nextthink = time + 0.5;
 }
 
 MUTATOR_HOOKFUNCTION(as, HavocBot_ChooseRole)
-{SELFPARAM();
-       havocbot_ast_reset_role(self);
+{
+       entity bot = M_ARGV(0, entity);
+
+       havocbot_ast_reset_role(bot);
        return true;
 }
 
@@ -671,7 +664,6 @@ MUTATOR_HOOKFUNCTION(as, ReadLevelCvars)
 {
        // no assault warmups
        warmup_stage = 0;
-       return false;
 }
 
 MUTATOR_HOOKFUNCTION(as, OnEntityPreSpawn)
@@ -686,8 +678,6 @@ MUTATOR_HOOKFUNCTION(as, OnEntityPreSpawn)
                case "info_player_team4":
                        return true;
        }
-
-       return false;
 }
 
 // scoreboard setup