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

DefaultSpriteSequence: Remove redundant modulo operations

Browse Source 
frame is already bounded by length.Value
This commit is contained in:
abcdefg30
2023-05-16 19:22:32 +02:00
committed by Gustas
parent 991e0a4c9a
commit 00f2ba1a53
1 changed files with 2 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
OpenRA.Mods.Common/Graphics/DefaultSpriteSequence.cs
View File

@@ -310,8 +310,8 @@ namespace OpenRA.Mods.Common.Graphics
var facingInner = reverseFacings ? (facings - facing) % facings : facing; var facingInner = reverseFacings ? (facings - facing) % facings : facing;
for (var frame = 0; frame < length.Value; frame++) for (var frame = 0; frame < length.Value; frame++)
{ {
var i = transpose ? frame % length.Value * facings + facingInner : var i = transpose ? frame * facings + facingInner :
facingInner * stride + frame % length.Value; facingInner * stride + frame;
usedFrames.Add(frames?[i] ?? start + i); usedFrames.Add(frames?[i] ?? start + i);
} }