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

Fix FindActorsOnLine overscan

Browse Source
This commit is contained in:
reaperrr
2017-08-24 12:45:51 +02:00
committed by Chris Forbes
parent a7c2f3e7e3
commit b1c63a4752
1 changed files with 5 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

7
OpenRA.Mods.Common/WorldExtensions.cs
View File

@@ -32,8 +32,11 @@ namespace OpenRA.Mods.Common
// Then we iterate over this list, and find all actors for which their health radius is at least within lineWidth of the line. // Then we iterate over this list, and find all actors for which their health radius is at least within lineWidth of the line.
// For actors without a health radius, we simply check their center point. // For actors without a health radius, we simply check their center point.
// The square in which we select all actors must be large enough to encompass the entire line's width. // The square in which we select all actors must be large enough to encompass the entire line's width.
var xDir = Math.Sign(lineEnd.X - lineStart.X); // xDir and yDir must never be 0, otherwise the overscan will be 0 in the respective direction.
var yDir = Math.Sign(lineEnd.Y - lineStart.Y); var xDiff = lineEnd.X - lineStart.X;
var yDiff = lineEnd.Y - lineStart.Y;
var xDir = xDiff < 0 ? -1 : 1;
var yDir = yDiff < 0 ? -1 : 1;
var dir = new WVec(xDir, yDir, 0); var dir = new WVec(xDir, yDir, 0);
var overselect = dir * (1024 + lineWidth.Length + targetExtraSearchRadius.Length); var overselect = dir * (1024 + lineWidth.Length + targetExtraSearchRadius.Length);