[Sc2-cvs] SF.net SVN: sc2:[3720] trunk/sc2/src/uqm/flash.c

[<Mee...@us...>]

Revision: 3720
          http://sc2.svn.sourceforge.net/sc2/?rev=3720&view=rev
Author:   Meep-Eep
Date:     2011-12-29 21:29:16 +0000 (Thu, 29 Dec 2011)
Log Message:
-----------
Change 'if' chain to switch.
Now why didn't I use a switch in the first place?

Modified Paths:
--------------
    trunk/sc2/src/uqm/flash.c

Modified: trunk/sc2/src/uqm/flash.c
===================================================================
--- trunk/sc2/src/uqm/flash.c	2011-12-29 21:18:08 UTC (rev 3719)
+++ trunk/sc2/src/uqm/flash.c	2011-12-29 21:29:16 UTC (rev 3720)
@@ -275,38 +275,37 @@
 	
 	now = GetTimeCounter ();
 
-	if (context->state == FlashState_fadeIn)
+	switch (context->state)
 	{
-		if (now >= context->lastStateTime + context->fadeInTime)
-		{
+		case FlashState_fadeIn:
+			if (now >= context->lastStateTime + context->fadeInTime)
+			{
+				Flash_nextState (context);
+				context->lastStateTime = now;
+			}
+			context->lastFrameTime = now;
+			break;
+		case FlashState_on:
+			if (now < context->lastStateTime + context->onTime)
+				return;
 			Flash_nextState (context);
 			context->lastStateTime = now;
-		}
-		context->lastFrameTime = now;
-	}
-	else if (context->state == FlashState_on)
-	{
-		if (now < context->lastStateTime + context->onTime)
-			return;
-		Flash_nextState (context);
-		context->lastStateTime = now;
-	}
-	else if (context->state == FlashState_fadeOut)
-	{
-		if (now >= context->lastStateTime + context->fadeOutTime)
-		{
+			break;
+		case FlashState_fadeOut:
+			if (now >= context->lastStateTime + context->fadeOutTime)
+			{
+				Flash_nextState (context);
+				context->lastStateTime = now;
+			}
+			context->lastFrameTime = now;
+			break;
+		case FlashState_off:
+			if (now < context->lastStateTime + context->offTime)
+				return;
 			Flash_nextState (context);
 			context->lastStateTime = now;
-		}
-		context->lastFrameTime = now;
+			break;
 	}
-	else /* context->state == FlashState_off */
-	{
-		if (now < context->lastStateTime + context->offTime)
-			return;
-		Flash_nextState (context);
-		context->lastStateTime = now;
-	}
 
 	Flash_drawCurrentFrame (context);
 }

This was sent by the SourceForge.net collaborative development platform, the world's largest Open Source development site.