From 56294105769cf34f7284e815571a6dbc52044ddf Mon Sep 17 00:00:00 2001
From: Paul Gilbert <dreammaster@scummvm.org>
Date: Sun, 12 Jul 2015 23:09:38 -0400
Subject: [PATCH] SHERLOCK: Fix compiler warnings

---
 engines/sherlock/image_file.cpp          |  3 +++
 engines/sherlock/tattoo/tattoo_darts.cpp | 20 +++++++-------------
 engines/sherlock/tattoo/tattoo_darts.h   |  2 +-
 3 files changed, 11 insertions(+), 14 deletions(-)

diff --git a/engines/sherlock/image_file.cpp b/engines/sherlock/image_file.cpp
index c202ab3dca8a..81087dae8b72 100644
--- a/engines/sherlock/image_file.cpp
+++ b/engines/sherlock/image_file.cpp
@@ -729,6 +729,9 @@ void ImageFile3DO::load3DOCelRoomData(Common::SeekableReadStream &stream) {
 			push_back(imageFrame);
 		}
 	}
+
+	// Suppress compiler warning
+	warning("ccbPPMP0 = %d, ccbPPMP1 = %d", ccbPPMP0, ccbPPMP1);
 }
 
 static uint16 imagefile3DO_cel_bitsMask[17] = {
diff --git a/engines/sherlock/tattoo/tattoo_darts.cpp b/engines/sherlock/tattoo/tattoo_darts.cpp
index 917b80460285..842320e27046 100644
--- a/engines/sherlock/tattoo/tattoo_darts.cpp
+++ b/engines/sherlock/tattoo/tattoo_darts.cpp
@@ -128,7 +128,6 @@ void Darts::playDarts(GameType gameType) {
 			} else {
 				// check for cricket game over
 				bool allClosed = true;
-				bool otherAllClosed = true;
 				int nOtherScore;
 
 				for (int y = 0; y < 7; y++) {
@@ -136,11 +135,6 @@ void Darts::playDarts(GameType gameType) {
 						allClosed = false;
 				}
 
-				for (int y = 0; y < 7; ++y) {
-					if (_cricketScore[playerNum ^ 1][y] < 3)
-						otherAllClosed = false;
-				}
-
 				if (allClosed) {
 					nOtherScore = (playerNum == 0) ? _score2 : _score1;
 					if (score >= nOtherScore)
@@ -879,8 +873,8 @@ int Darts::throwDart(int dartNum, int computer) {
 	drawDartsLeft(dartNum, computer);
 
 	if (!computer) {
-		screen.print(Common::Point(_dartInfo.left, _dartInfo.top + _spacing), 0, FIXED(HitAKey));
-		screen.print(Common::Point(_dartInfo.left, _dartInfo.top + _spacing * 2), 0, FIXED(ToStart));
+		screen.print(Common::Point(_dartInfo.left, _dartInfo.top + _spacing), 0, "%s", FIXED(HitAKey));
+		screen.print(Common::Point(_dartInfo.left, _dartInfo.top + _spacing * 2), 0, "%s", FIXED(ToStart));
 	}
 
 	if (!computer) {
@@ -938,13 +932,13 @@ void Darts::doCricketScoreHits(int player, int scoreIndex, int numHits) {
 	}
 }
 
-void Darts::updateCricketScore(int player, int dartHit, int multiplier) {
-	if (dartHit < 15)
+void Darts::updateCricketScore(int player, int dartVal, int multiplier) {
+	if (dartVal < 15)
 		return;
 
-	if (dartHit <= 20)
-		doCricketScoreHits(player, 20 - dartHit, multiplier);
-	else if (dartHit == 25)
+	if (dartVal <= 20)
+		doCricketScoreHits(player, 20 - dartVal, multiplier);
+	else if (dartVal == 25)
 		doCricketScoreHits(player, 6, multiplier);
 }
 
diff --git a/engines/sherlock/tattoo/tattoo_darts.h b/engines/sherlock/tattoo/tattoo_darts.h
index dbbb1f408c29..f65ec19d1089 100644
--- a/engines/sherlock/tattoo/tattoo_darts.h
+++ b/engines/sherlock/tattoo/tattoo_darts.h
@@ -150,7 +150,7 @@ class Darts {
 	/**
 	 * Updates the score based upon what the dart hit
 	 */
-	void updateCricketScore(int player, int dartHit, int multiplier);
+	void updateCricketScore(int player, int dartVal, int multiplier);
 
 	/**
 	 * Draw the darts left