From d89da56f471735b50a5eafbf9ee9eeb0c190f4eb Mon Sep 17 00:00:00 2001
From: Yotam Barnoy
Date: Thu, 10 Jun 2010 14:01:30 +0000
Subject: PSP: swapped order of checks in renderAll. It's a little cheaper this
 way.

svn-id: r49574
---
 backends/platform/psp/display_manager.cpp | 7 ++++---
 1 file changed, 4 insertions(+), 3 deletions(-)

(limited to 'backends/platform/psp/display_manager.cpp')

diff --git a/backends/platform/psp/display_manager.cpp b/backends/platform/psp/display_manager.cpp
index 5c9f8f92f0..15e0188ebf 100644
--- a/backends/platform/psp/display_manager.cpp
+++ b/backends/platform/psp/display_manager.cpp
@@ -322,9 +322,7 @@ bool DisplayManager::renderAll() {
 	}	
 #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 @@ bool DisplayManager::renderAll() {
 		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",
-- 
cgit v1.2.3