From 1f96dc550c554e5481496b729434e0b36fa2eaa7 Mon Sep 17 00:00:00 2001
From: Max Horn
Date: Mon, 12 Dec 2011 16:17:15 +0100
Subject: TSAGE: Remove unused method

---
 engines/tsage/core.h | 1 -
 1 file changed, 1 deletion(-)

(limited to 'engines')

diff --git a/engines/tsage/core.h b/engines/tsage/core.h
index cbd3d9f77c..060ffee121 100644
--- a/engines/tsage/core.h
+++ b/engines/tsage/core.h
@@ -421,7 +421,6 @@ public:
 	virtual void destroy() {}
 	virtual bool startAction(CursorType action, Event &event);
 	virtual void doAction(int action);
-	virtual bool performAction(CursorType action, Event &event) { return startAction(action, event); }
 
 	bool contains(const Common::Point &pt);
 	void setBounds(const Rect &newBounds) { _bounds = newBounds; }
-- 
cgit v1.2.3


From 30909b1ae455a7cb615720d5a65616925c3172a3 Mon Sep 17 00:00:00 2001
From: Max Horn
Date: Mon, 12 Dec 2011 16:17:30 +0100
Subject: TSAGE: Fix warning about hidden remove() method from parent class

---
 engines/tsage/blue_force/blueforce_logic.h | 3 +++
 1 file changed, 3 insertions(+)

(limited to 'engines')

diff --git a/engines/tsage/blue_force/blueforce_logic.h b/engines/tsage/blue_force/blueforce_logic.h
index d0d0e0ee40..f5a3938f2b 100644
--- a/engines/tsage/blue_force/blueforce_logic.h
+++ b/engines/tsage/blue_force/blueforce_logic.h
@@ -65,6 +65,9 @@ public:
 
 	void add(EventHandler *obj);
 	void remove(EventHandler *obj);
+	// The following line prevents compiler warnings about hiding the remove()
+	// method from the parent class.
+	virtual void remove() { EventHandler::remove(); }
 };
 
 class Timer: public EventHandler {
-- 
cgit v1.2.3


From f485450758ffa37c789beb6bd038782dec8d97ad Mon Sep 17 00:00:00 2001
From: Max Horn
Date: Mon, 12 Dec 2011 16:22:55 +0100
Subject: HUGO: Fix warning about hidden pause() method from parent class

---
 engines/hugo/sound.h | 6 ++++++
 1 file changed, 6 insertions(+)

(limited to 'engines')

diff --git a/engines/hugo/sound.h b/engines/hugo/sound.h
index 1e504fbdea..cb6d4e3168 100644
--- a/engines/hugo/sound.h
+++ b/engines/hugo/sound.h
@@ -43,6 +43,12 @@ public:
 	void pause(bool p);
 	void play(uint8 *stream, uint16 size);
 
+	// The following line prevents compiler warnings about hiding the pause()
+	// method from the parent class.
+	// FIXME: Maybe the pause(bool p) method should be removed and the
+	// pause/resume methods of the parent class be used instead?
+	virtual void pause() { Audio::MidiPlayer::pause(); }
+
 	uint32 getBaseTempo();
 
 	// Overload Audio::MidiPlayer method
-- 
cgit v1.2.3


From 6310cce74e224818dab3d714afd08a6cf6789874 Mon Sep 17 00:00:00 2001
From: Max Horn
Date: Mon, 12 Dec 2011 17:17:45 +0100
Subject: AGI: Remove two unused pure virtual methods

This avoid warnings, as these methods were being hidden by methods in subclasses
with differing signatures.
---
 engines/agi/agi.h | 2 --
 1 file changed, 2 deletions(-)

(limited to 'engines')

diff --git a/engines/agi/agi.h b/engines/agi/agi.h
index 6bb3beec78..e9923aba2e 100644
--- a/engines/agi/agi.h
+++ b/engines/agi/agi.h
@@ -809,8 +809,6 @@ public:
 	virtual void replayImageStackCall(uint8 type, int16 p1, int16 p2, int16 p3,
 		int16 p4, int16 p5, int16 p6, int16 p7) = 0;
 	virtual void releaseImageStack() = 0;
-	virtual	int saveGame(const Common::String &fileName, const Common::String &saveName) = 0;
-	virtual int loadGame(const Common::String &fileName, bool checkId = true) = 0;
 
 	int _soundemu;
 
-- 
cgit v1.2.3


From 00e6ab43f8db3b79486f324dfb11febbd4ad8c6b Mon Sep 17 00:00:00 2001
From: Max Horn
Date: Mon, 12 Dec 2011 17:24:06 +0100
Subject: AGI: Add FIXME to cmdCallV1

---
 engines/agi/op_cmd.cpp | 3 +++
 1 file changed, 3 insertions(+)

(limited to 'engines')

diff --git a/engines/agi/op_cmd.cpp b/engines/agi/op_cmd.cpp
index 41d9cc3ac9..7e04328a67 100644
--- a/engines/agi/op_cmd.cpp
+++ b/engines/agi/op_cmd.cpp
@@ -1702,6 +1702,9 @@ void cmdSetItemView(AgiGame *state, uint8 *p) {
 
 void cmdCallV1(AgiGame *state, uint8 *p) {
 	state->_vm->agiLoadResource(rLOGIC, p0);
+	// FIXME: The following instruction looks incomplete.
+	// Maybe something is meant to be assigned to, or read from,
+	// the logic_list entry?
 	state->logic_list[++state->max_logics];
 	_v[13] = 1;
 }
-- 
cgit v1.2.3