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

change PlayThen and PlayRepeating definitions to not allocate Action objects like crazy

Browse Source
This commit is contained in:
Chris Forbes
2011-03-05 21:52:34 +13:00
parent 269347ef01
commit ea022a0e06
1 changed files with 12 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

16
OpenRA.Game/Graphics/Animation.cs
View File

@@ -48,12 +48,21 @@ namespace OpenRA.Graphics
public void Play( string sequenceName ) public void Play( string sequenceName )
{ {
PlayThen(sequenceName, () => { }); PlayThen(sequenceName, null);
} }
public void PlayRepeating( string sequenceName ) public void PlayRepeating( string sequenceName )
{ {
PlayThen( sequenceName, () => PlayRepeating( CurrentSequence.Name ) ); backwards = false;
tickAlways = false;
CurrentSequence = SequenceProvider.GetSequence(name, sequenceName);
frame = 0;
tickFunc = () =>
{
++frame;
if (frame >= CurrentSequence.Length)
frame = 0;
};
} }
public bool ReplaceAnim(string sequenceName) public bool ReplaceAnim(string sequenceName)
@@ -68,7 +77,6 @@ namespace OpenRA.Graphics
public void PlayThen( string sequenceName, Action after ) public void PlayThen( string sequenceName, Action after )
{ {
after = after ?? ( () => { } );
backwards = false; backwards = false;
tickAlways = false; tickAlways = false;
CurrentSequence = SequenceProvider.GetSequence( name, sequenceName ); CurrentSequence = SequenceProvider.GetSequence( name, sequenceName );
@@ -80,7 +88,7 @@ namespace OpenRA.Graphics
{ {
frame = CurrentSequence.Length - 1; frame = CurrentSequence.Length - 1;
tickFunc = () => { }; tickFunc = () => { };
after(); if (after != null) after();
} }
}; };
} }