From e3f81c267b5ef08838734baa31e84bec9ee7fb3d Mon Sep 17 00:00:00 2001
From: Paul Gilbert
Date: Mon, 16 Nov 2015 18:51:33 -0500
Subject: SHERLOCK: RT: Fix Coverity warning in original savegame dialog

---
 engines/sherlock/tattoo/widget_files.cpp | 20 +++++++++-----------
 1 file changed, 9 insertions(+), 11 deletions(-)

(limited to 'engines')

diff --git a/engines/sherlock/tattoo/widget_files.cpp b/engines/sherlock/tattoo/widget_files.cpp
index 0f5e6793c4..ca5e1bff93 100644
--- a/engines/sherlock/tattoo/widget_files.cpp
+++ b/engines/sherlock/tattoo/widget_files.cpp
@@ -143,19 +143,17 @@ void WidgetFiles::render(FilesRenderMode mode) {
 	int yp = _surface.fontHeight() + 14;
 	
 	for (int idx = _savegameIndex; idx < (_savegameIndex + FILES_LINES_COUNT); ++idx) {
-		if (OP_NAMES || idx == _selector || idx == _oldSelector) {
-			if (idx == _selector && mode != RENDER_ALL)
-				color = COMMAND_HIGHLIGHTED;
-			else
-				color = INFO_TOP;
+		if (idx == _selector && mode != RENDER_ALL)
+			color = COMMAND_HIGHLIGHTED;
+		else
+			color = INFO_TOP;
 
-			if (mode == RENDER_NAMES_AND_SCROLLBAR)
-				_surface.fillRect(Common::Rect(4, yp, _surface.w() - BUTTON_SIZE - 9, yp + _surface.fontHeight()), TRANSPARENCY);
+		if (mode == RENDER_NAMES_AND_SCROLLBAR)
+			_surface.fillRect(Common::Rect(4, yp, _surface.w() - BUTTON_SIZE - 9, yp + _surface.fontHeight()), TRANSPARENCY);
 			
-			Common::String numStr = Common::String::format("%d.", idx + 1);
-			_surface.writeString(numStr, Common::Point(_surface.widestChar(), yp), color);
-			_surface.writeString(_savegames[idx], Common::Point(xp, yp), color);
-		}
+		Common::String numStr = Common::String::format("%d.", idx + 1);
+		_surface.writeString(numStr, Common::Point(_surface.widestChar(), yp), color);
+		_surface.writeString(_savegames[idx], Common::Point(xp, yp), color);
 
 		yp += _surface.fontHeight() + 1;
 	}
-- 
cgit v1.2.3