[Scummvm-cvs-logs] SF.net SVN: scummvm:[49574] scummvm/trunk/backends/platform/psp/ display_manager.cpp

[Bluddy at users.sourceforge.net]

Revision: 49574
          http://scummvm.svn.sourceforge.net/scummvm/?rev=49574&view=rev
Author:   Bluddy
Date:     2010-06-10 14:01:30 +0000 (Thu, 10 Jun 2010)

Log Message:
-----------
PSP: swapped order of checks in renderAll. It's a little cheaper this way.

Modified Paths:
--------------
    scummvm/trunk/backends/platform/psp/display_manager.cpp

Modified: scummvm/trunk/backends/platform/psp/display_manager.cpp
===================================================================
--- scummvm/trunk/backends/platform/psp/display_manager.cpp	2010-06-10 13:43:38 UTC (rev 49573)
+++ scummvm/trunk/backends/platform/psp/display_manager.cpp	2010-06-10 14:01:30 UTC (rev 49574)
@@ -322,9 +322,7 @@
 	}	
 #endif /* USE_DISPLAY_CALLBACK */
 	
-	if (!isTimeToUpdate()) 
-		return false;	// didn't render
-
+	// This is cheaper than checking time, so we do it first
 	if (!_screen->isDirty() &&
 	        (!_overlay->isDirty()) &&
 	        (!_cursor->isDirty()) &&
@@ -333,6 +331,9 @@
 		return true;	// nothing to render
 	}
 
+	if (!isTimeToUpdate()) 
+		return false;	// didn't render
+
 	PSP_DEBUG_PRINT("screen[%s], overlay[%s], cursor[%s], keyboard[%s]\n",
 	                _screen->isDirty() ? "true" : "false",
 	                _overlay->isDirty() ? "true" : "false",


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