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

Merge pull request #11821 from UnknownProgrammer/patch-1

Browse Source 
Fix for System Overflow exception
This commit is contained in:
reaperrr
2016-09-18 13:12:59 +02:00
committed by GitHub
parent f1bdcca91d 65fe88daef
commit b820f421c0
1 changed files with 4 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
OpenRA.Game/WPos.cs
View File

@@ -65,8 +65,10 @@ namespace OpenRA
// Add an additional quadratic variation to height
// Uses decimal to avoid integer overflow
var offset = (int)((decimal)(b - a).Length * pitch.Tan() * mul * (div - mul) / (1024 * div * div));
return new WPos(ret.X, ret.Y, ret.Z + offset);
var offset = (decimal)(b - a).Length * pitch.Tan() * mul * (div - mul) / (1024 * div * div);
var clampedOffset = (int)(offset + (decimal)ret.Z).Clamp<decimal>((decimal)int.MinValue, (decimal)int.MaxValue);
return new WPos(ret.X, ret.Y, clampedOffset);
}
public override int GetHashCode() { return X.GetHashCode() ^ Y.GetHashCode() ^ Z.GetHashCode(); }