Ticket #1945: add-missed-profiler-stop-calls-in-ai.patch

trunk/binaries/data/mods/public/simulation/ai/qbot-wc/attack_plan.js

@@ -519 +519 @@
                 // HACK (TODO replace) : if we have no trainable template... Then we'll simply remove the buildOrder, effectively removing the unit from the plan.
                 if (template === undefined) {
                     // TODO: this is a complete hack.
-                    if (this.needsShip && this.buildOrder[0][4] == "TransportShip")
+                    if (this.needsShip && this.buildOrder[0][4] == "TransportShip") {
+                        Engine.ProfileStop();
+                        Engine.ProfileStop();
                         return 0;
+                    }
                     delete this.unitStat[this.buildOrder[0][4]];    // deleting the associated unitstat.
                     this.buildOrder.splice(0,1);
                 } else {
@@ -569 +572 @@
             }
         }
         */
+        Engine.ProfileStop();
+        Engine.ProfileStop();
         // can happen for now
         if (this.buildOrder.length === 0) {
             debug ("Ending plan: no build orders");
             return 0;   // will abort the plan, should return something else
         }
-        Engine.ProfileStop();
-        Engine.ProfileStop();
         return 1;
     }
     this.unitCollection.forEach(function (entity) { entity.setMetadata(PlayerID, "role","attack"); });
@@ -729 +732 @@
     var bool_attacked = false;
     // raids don't care about attacks much
     
-    if (this.unitCollection.length === 0)
+    if (this.unitCollection.length === 0) {
+        Engine.ProfileStop();
         return 0;
+    }
     
     this.position = this.unitCollection.getCentrePosition();
     
@@ -842 +847 @@
         this.position = this.unitCollectionNoWarship.getCentrePosition();
 
         // probably not too good.
-        if (!this.position)
+        if (!this.position) {
+            Engine.ProfileStop();
             return undefined;   // should spawn an error.
-        
+        }
+
         // basically haven't moved an inch: very likely stuck)
         if (SquareVectorDistance(this.position, this.position5TurnsAgo) < 10 && this.path.length > 0 && gameState.ai.playedTurn % 5 === 0) {
             // check for stuck siege units
@@ -882 +889 @@
             } else if (nexttoWalls) {
                 // abort plan.
                 debug ("Attack Plan " +this.type +" " +this.name +" has met walls and gives up.");
+                Engine.ProfileStop();
                 return 0;
             }
         }
@@ -911 +919 @@
                     // okay we must load our units.
                     // check if we have some kind of ships.
                     var ships = this.unitCollection.filter(Filters.byClass("Warship"));
-                    if (ships.length === 0)
+                    if (ships.length === 0) {
+                        Engine.ProfileStop();
                         return 0; // abort
+                    }
                 
                     debug ("switch to boarding");
                     this.state = "boarding";
@@ -936 +946 @@
                     this.unitCollection.filter(Filters.byClass("Warship")).move(this.path[0][0][0], this.path[0][0][1]);
                 } else {
                     debug ("Attack Plan " +this.type +" " +this.name +" has arrived to destination, but it's still on the ship…");
+                    Engine.ProfileStop();
                     return 0; // abort
                 }
             } else if (this.path[0][1] === true)
@@ -949 +960 @@
         this.position = this.unitCollectionNoWarship.getCentrePosition();
 
         var ships = this.unitCollection.filter(Filters.byClass("Warship"));
-        if (ships.length === 0)
+        if (ships.length === 0) {
+            Engine.ProfileStop();
             return 0; // abort
+        }
         
         var globalPos = this.unitCollectionNoWarship.getCentrePosition();
         var shipPos = ships.getCentrePosition();
@@ -993 +1006 @@
     } else if (this.state === "unboarding") {
         
         var ships = this.unitCollection.filter(Filters.byClass("Warship"));
-        if (ships.length === 0)
+        if (ships.length === 0) {
+            Engine.ProfileStop();
             return 0; // abort
-        
+        }
+
         this.position = ships.getCentrePosition();
         
         // the procedure is pretty simple: we move the ships to the next point and try to unload until all units are over.
@@ -1264 +1279 @@
                     count++;
                     if (count > 1000){
                         debug("No target with a valid position found");
+                        Engine.ProfileStop();
                         return false;
                     }
                 }

trunk/binaries/data/mods/public/simulation/ai/qbot-wc/economy.js

@@ -1184 +1184 @@
         });
 
         Engine.ProfileStop();
+        Engine.ProfileStop();
         return;
     }