From 00f2ba1a53ca465e6ec82ea808967253a1771da7 Mon Sep 17 00:00:00 2001 From: abcdefg30 Date: Tue, 16 May 2023 19:22:32 +0200 Subject: [PATCH] DefaultSpriteSequence: Remove redundant modulo operations frame is already bounded by length.Value --- OpenRA.Mods.Common/Graphics/DefaultSpriteSequence.cs | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/OpenRA.Mods.Common/Graphics/DefaultSpriteSequence.cs b/OpenRA.Mods.Common/Graphics/DefaultSpriteSequence.cs index a21fa326c1..e691ceda53 100644 --- a/OpenRA.Mods.Common/Graphics/DefaultSpriteSequence.cs +++ b/OpenRA.Mods.Common/Graphics/DefaultSpriteSequence.cs @@ -310,8 +310,8 @@ namespace OpenRA.Mods.Common.Graphics var facingInner = reverseFacings ? (facings - facing) % facings : facing; for (var frame = 0; frame < length.Value; frame++) { - var i = transpose ? frame % length.Value * facings + facingInner : - facingInner * stride + frame % length.Value; + var i = transpose ? frame * facings + facingInner : + facingInner * stride + frame; usedFrames.Add(frames?[i] ?? start + i); }