(svn r3187) Simplify overly complicated ifs, especially if (foo) return false; else return true; is confusing

1 files changed, 3 insertions, 4 deletions

diff --git a/gfx.c b/gfx.c
index c8c6d6845..32ecca3ae 100644
--- a/gfx.c
+++ b/gfx.c
@@ -1970,10 +1970,9 @@ void SetAnimatedMouseCursor(const CursorID *table)
 
 bool ChangeResInGame(int w, int h)
 {
-	if ((_screen.width != w || _screen.height != h) && !_video_driver->change_resolution(w, h))
-		return false;
-
-	return true;
+	return
+		(_screen.width == w && _screen.height == h) ||
+		_video_driver->change_resolution(w, h);
 }
 
 void ToggleFullScreen(bool fs) {_video_driver->toggle_fullscreen(fs);}