From 65fe88daefc57f0d54c02c4a4e0290de3286060b Mon Sep 17 00:00:00 2001 From: UnknownProgrammer Date: Sat, 3 Sep 2016 15:27:04 +0200 Subject: [PATCH] used clamp instead of Math.Min/Math.Max construct clarified naming restored comments removed blank line --- OpenRA.Game/WPos.cs | 9 +++++---- 1 file changed, 5 insertions(+), 4 deletions(-) diff --git a/OpenRA.Game/WPos.cs b/OpenRA.Game/WPos.cs index c29c336240..6ea08e0b78 100644 --- a/OpenRA.Game/WPos.cs +++ b/OpenRA.Game/WPos.cs @@ -63,11 +63,12 @@ namespace OpenRA if (pitch.Angle == 0) return ret; - var tempoffset = (decimal)(b - a).Length * pitch.Tan() * mul * (div - mul) / (1024 * div * div); + // Add an additional quadratic variation to height + // Uses decimal to avoid integer overflow + var offset = (decimal)(b - a).Length * pitch.Tan() * mul * (div - mul) / (1024 * div * div); + var clampedOffset = (int)(offset + (decimal)ret.Z).Clamp((decimal)int.MinValue, (decimal)int.MaxValue); - var offset = (int)Math.Min(Math.Max((decimal)int.MinValue, tempoffset + (decimal)ret.Z), (decimal)int.MaxValue); - - return new WPos(ret.X, ret.Y, offset); + return new WPos(ret.X, ret.Y, clampedOffset); } public override int GetHashCode() { return X.GetHashCode() ^ Y.GetHashCode() ^ Z.GetHashCode(); }