epic/OpenRA
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Use OccupiesSpace to avoid Mobile look-up in Move

Browse Source 
While individual trait look-ups may be cheap,
if a large army that is currently standing still gets
its first move-including order, the look-ups of dozens
or even hundreds of actors may happen on the same tick.

Therefore this may help reducing that first-order lag spike,
at least a little bit.
This commit is contained in:
reaperrr
2020-05-29 14:19:37 +02:00
committed by Matthias Mailänder
parent 96b06c75d1
commit 4bf614c5cd
1 changed files with 10 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files

15
OpenRA.Mods.Common/Activities/Move/Move.cs
View File

@@ -52,7 +52,8 @@ namespace OpenRA.Mods.Common.Activities
// Ignores lane bias and nearby units // Ignores lane bias and nearby units
public Move(Actor self, CPos destination, Color? targetLineColor = null) public Move(Actor self, CPos destination, Color? targetLineColor = null)
{ {
mobile = self.Trait<Mobile>(); // PERF: Because we can be sure that OccupiesSpace is Mobile here, we can save some performance by avoiding querying for the trait.
mobile = (Mobile)self.OccupiesSpace;
getPath = check => getPath = check =>
{ {
@@ -72,7 +73,8 @@ namespace OpenRA.Mods.Common.Activities
public Move(Actor self, CPos destination, WDist nearEnough, Actor ignoreActor = null, bool evaluateNearestMovableCell = false, public Move(Actor self, CPos destination, WDist nearEnough, Actor ignoreActor = null, bool evaluateNearestMovableCell = false,
Color? targetLineColor = null) Color? targetLineColor = null)
{ {
mobile = self.Trait<Mobile>(); // PERF: Because we can be sure that OccupiesSpace is Mobile here, we can save some performance by avoiding querying for the trait.
mobile = (Mobile)self.OccupiesSpace;
getPath = check => getPath = check =>
{ {
@@ -93,7 +95,8 @@ namespace OpenRA.Mods.Common.Activities
public Move(Actor self, CPos destination, SubCell subCell, WDist nearEnough, Color? targetLineColor = null) public Move(Actor self, CPos destination, SubCell subCell, WDist nearEnough, Color? targetLineColor = null)
{ {
mobile = self.Trait<Mobile>(); // PERF: Because we can be sure that OccupiesSpace is Mobile here, we can save some performance by avoiding querying for the trait.
mobile = (Mobile)self.OccupiesSpace;
getPath = check => mobile.Pathfinder.FindUnitPathToRange( getPath = check => mobile.Pathfinder.FindUnitPathToRange(
mobile.FromCell, subCell, self.World.Map.CenterOfSubCell(destination, subCell), nearEnough, self, check); mobile.FromCell, subCell, self.World.Map.CenterOfSubCell(destination, subCell), nearEnough, self, check);
@@ -105,7 +108,8 @@ namespace OpenRA.Mods.Common.Activities
public Move(Actor self, Target target, WDist range, Color? targetLineColor = null) public Move(Actor self, Target target, WDist range, Color? targetLineColor = null)
{ {
mobile = self.Trait<Mobile>(); // PERF: Because we can be sure that OccupiesSpace is Mobile here, we can save some performance by avoiding querying for the trait.
mobile = (Mobile)self.OccupiesSpace;
getPath = check => getPath = check =>
{ {
@@ -123,7 +127,8 @@ namespace OpenRA.Mods.Common.Activities
public Move(Actor self, Func<BlockedByActor, List<CPos>> getPath, Color? targetLineColor = null) public Move(Actor self, Func<BlockedByActor, List<CPos>> getPath, Color? targetLineColor = null)
{ {
mobile = self.Trait<Mobile>(); // PERF: Because we can be sure that OccupiesSpace is Mobile here, we can save some performance by avoiding querying for the trait.
mobile = (Mobile)self.OccupiesSpace;
this.getPath = getPath; this.getPath = getPath;