[Scummvm-cvs-logs] SF.net SVN: scummvm:[34906] scummvm/trunk

fingolfin at users.sourceforge.net fingolfin at users.sourceforge.net
Wed Nov 5 18:24:56 CET 2008


Revision: 34906
          http://scummvm.svn.sourceforge.net/scummvm/?rev=34906&view=rev
Author:   fingolfin
Date:     2008-11-05 17:24:56 +0000 (Wed, 05 Nov 2008)

Log Message:
-----------
Turned enum PluginError into Common::Error, which in the future is to be used in more places. Help with this is highly welcome

Modified Paths:
--------------
    scummvm/trunk/backends/midi/alsa.cpp
    scummvm/trunk/backends/midi/camd.cpp
    scummvm/trunk/backends/midi/coreaudio.cpp
    scummvm/trunk/backends/midi/coremidi.cpp
    scummvm/trunk/backends/midi/dmedia.cpp
    scummvm/trunk/backends/midi/quicktime.cpp
    scummvm/trunk/backends/midi/seq.cpp
    scummvm/trunk/backends/midi/stmidi.cpp
    scummvm/trunk/backends/midi/timidity.cpp
    scummvm/trunk/backends/midi/windows.cpp
    scummvm/trunk/backends/midi/ypa1.cpp
    scummvm/trunk/backends/midi/zodiac.cpp
    scummvm/trunk/base/main.cpp
    scummvm/trunk/common/advancedDetector.cpp
    scummvm/trunk/common/advancedDetector.h
    scummvm/trunk/common/error.h
    scummvm/trunk/engines/metaengine.h
    scummvm/trunk/engines/queen/queen.cpp
    scummvm/trunk/engines/scumm/detection.cpp
    scummvm/trunk/engines/sky/sky.cpp
    scummvm/trunk/engines/sword1/sword1.cpp
    scummvm/trunk/engines/sword2/sword2.cpp
    scummvm/trunk/sound/musicplugin.h
    scummvm/trunk/sound/null.cpp
    scummvm/trunk/sound/softsynth/adlib.cpp
    scummvm/trunk/sound/softsynth/fluidsynth.cpp
    scummvm/trunk/sound/softsynth/mt32.cpp
    scummvm/trunk/sound/softsynth/ym2612.cpp

Modified: scummvm/trunk/backends/midi/alsa.cpp
===================================================================
--- scummvm/trunk/backends/midi/alsa.cpp	2008-11-05 15:41:12 UTC (rev 34905)
+++ scummvm/trunk/backends/midi/alsa.cpp	2008-11-05 17:24:56 UTC (rev 34906)
@@ -269,7 +269,7 @@
 	}
 
 	MusicDevices getDevices() const;
-	PluginError createInstance(Audio::Mixer *mixer, MidiDriver **mididriver) const;
+	Common::Error createInstance(Audio::Mixer *mixer, MidiDriver **mididriver) const;
 };
 
 #define perm_ok(pinfo,bits) ((snd_seq_port_info_get_capability(pinfo) & (bits)) == (bits))
@@ -315,10 +315,10 @@
 	return devices;
 }
 
-PluginError AlsaMusicPlugin::createInstance(Audio::Mixer *mixer, MidiDriver **mididriver) const {
+Common::Error AlsaMusicPlugin::createInstance(Audio::Mixer *mixer, MidiDriver **mididriver) const {
 	*mididriver = new MidiDriver_ALSA();
 
-	return kNoError;
+	return Common::kNoError;
 }
 
 MidiDriver *MidiDriver_ALSA_create(Audio::Mixer *mixer) {

Modified: scummvm/trunk/backends/midi/camd.cpp
===================================================================
--- scummvm/trunk/backends/midi/camd.cpp	2008-11-05 15:41:12 UTC (rev 34905)
+++ scummvm/trunk/backends/midi/camd.cpp	2008-11-05 17:24:56 UTC (rev 34906)
@@ -177,7 +177,7 @@
 	}
 
 	MusicDevices getDevices() const;
-	PluginError createInstance(Audio::Mixer *mixer, MidiDriver **mididriver) const;
+	Common::Error createInstance(Audio::Mixer *mixer, MidiDriver **mididriver) const;
 };
 
 MusicDevices CamdMusicPlugin::getDevices() const {
@@ -188,10 +188,10 @@
 	return devices;
 }
 
-PluginError CamdMusicPlugin::createInstance(Audio::Mixer *mixer, MidiDriver **mididriver) const {
+Common::Error CamdMusicPlugin::createInstance(Audio::Mixer *mixer, MidiDriver **mididriver) const {
 	*mididriver = new MidiDriver_CAMD();
 
-	return kNoError;
+	return Common::kNoError;
 }
 
 MidiDriver *MidiDriver_CAMD_create(Audio::Mixer *mixer) {

Modified: scummvm/trunk/backends/midi/coreaudio.cpp
===================================================================
--- scummvm/trunk/backends/midi/coreaudio.cpp	2008-11-05 15:41:12 UTC (rev 34905)
+++ scummvm/trunk/backends/midi/coreaudio.cpp	2008-11-05 17:24:56 UTC (rev 34906)
@@ -218,7 +218,7 @@
 	}
 
 	MusicDevices getDevices() const;
-	PluginError createInstance(Audio::Mixer *mixer, MidiDriver **mididriver) const;
+	Common::Error createInstance(Audio::Mixer *mixer, MidiDriver **mididriver) const;
 };
 
 MusicDevices CoreAudioMusicPlugin::getDevices() const {
@@ -229,10 +229,10 @@
 	return devices;
 }
 
-PluginError CoreAudioMusicPlugin::createInstance(Audio::Mixer *mixer, MidiDriver **mididriver) const {
+Common::Error CoreAudioMusicPlugin::createInstance(Audio::Mixer *mixer, MidiDriver **mididriver) const {
 	*mididriver = new MidiDriver_CORE();
 
-	return kNoError;
+	return Common::kNoError;
 }
 
 MidiDriver *MidiDriver_CORE_create(Audio::Mixer *mixer) {

Modified: scummvm/trunk/backends/midi/coremidi.cpp
===================================================================
--- scummvm/trunk/backends/midi/coremidi.cpp	2008-11-05 15:41:12 UTC (rev 34905)
+++ scummvm/trunk/backends/midi/coremidi.cpp	2008-11-05 17:24:56 UTC (rev 34906)
@@ -190,7 +190,7 @@
 	}
 
 	MusicDevices getDevices() const;
-	PluginError createInstance(Audio::Mixer *mixer, MidiDriver **mididriver) const;
+	Common::Error createInstance(Audio::Mixer *mixer, MidiDriver **mididriver) const;
 };
 
 MusicDevices CoreMIDIMusicPlugin::getDevices() const {
@@ -201,10 +201,10 @@
 	return devices;
 }
 
-PluginError CoreMIDIMusicPlugin::createInstance(Audio::Mixer *mixer, MidiDriver **mididriver) const {
+Common::Error CoreMIDIMusicPlugin::createInstance(Audio::Mixer *mixer, MidiDriver **mididriver) const {
 	*mididriver = new MidiDriver_CoreMIDI();
 
-	return kNoError;
+	return Common::kNoError;
 }
 
 MidiDriver *MidiDriver_CoreMIDI_create(Audio::Mixer *mixer) {

Modified: scummvm/trunk/backends/midi/dmedia.cpp
===================================================================
--- scummvm/trunk/backends/midi/dmedia.cpp	2008-11-05 15:41:12 UTC (rev 34905)
+++ scummvm/trunk/backends/midi/dmedia.cpp	2008-11-05 17:24:56 UTC (rev 34906)
@@ -189,7 +189,7 @@
 	}
 
 	MusicDevices getDevices() const;
-	PluginError createInstance(Audio::Mixer *mixer, MidiDriver **mididriver) const;
+	Common::Error createInstance(Audio::Mixer *mixer, MidiDriver **mididriver) const;
 };
 
 MusicDevices DMediaMusicPlugin::getDevices() const {
@@ -200,10 +200,10 @@
 	return devices;
 }
 
-PluginError DMediaMusicPlugin::createInstance(Audio::Mixer *mixer, MidiDriver **mididriver) const {
+Common::Error DMediaMusicPlugin::createInstance(Audio::Mixer *mixer, MidiDriver **mididriver) const {
 	*mididriver = new MidiDriver_DMEDIA();
 
-	return kNoError;
+	return Common::kNoError;
 }
 
 MidiDriver *MidiDriver_DMEDIA_create(Audio::Mixer *mixer) {

Modified: scummvm/trunk/backends/midi/quicktime.cpp
===================================================================
--- scummvm/trunk/backends/midi/quicktime.cpp	2008-11-05 15:41:12 UTC (rev 34905)
+++ scummvm/trunk/backends/midi/quicktime.cpp	2008-11-05 17:24:56 UTC (rev 34906)
@@ -274,7 +274,7 @@
 	}
 
 	MusicDevices getDevices() const;
-	PluginError createInstance(Audio::Mixer *mixer, MidiDriver **mididriver) const;
+	Common::Error createInstance(Audio::Mixer *mixer, MidiDriver **mididriver) const;
 };
 
 MusicDevices QuickTimeMusicPlugin::getDevices() const {
@@ -285,10 +285,10 @@
 	return devices;
 }
 
-PluginError QuickTimeMusicPlugin::createInstance(Audio::Mixer *mixer, MidiDriver **mididriver) const {
+Common::Error QuickTimeMusicPlugin::createInstance(Audio::Mixer *mixer, MidiDriver **mididriver) const {
 	*mididriver = new MidiDriver_QT();
 
-	return kNoError;
+	return Common::kNoError;
 }
 
 MidiDriver *MidiDriver_QT_create(Audio::Mixer *mixer) {

Modified: scummvm/trunk/backends/midi/seq.cpp
===================================================================
--- scummvm/trunk/backends/midi/seq.cpp	2008-11-05 15:41:12 UTC (rev 34905)
+++ scummvm/trunk/backends/midi/seq.cpp	2008-11-05 17:24:56 UTC (rev 34906)
@@ -184,7 +184,7 @@
 	}
 
 	MusicDevices getDevices() const;
-	PluginError createInstance(Audio::Mixer *mixer, MidiDriver **mididriver) const;
+	Common::Error createInstance(Audio::Mixer *mixer, MidiDriver **mididriver) const;
 };
 
 MusicDevices SeqMusicPlugin::getDevices() const {
@@ -195,10 +195,10 @@
 	return devices;
 }
 
-PluginError SeqMusicPlugin::createInstance(Audio::Mixer *mixer, MidiDriver **mididriver) const {
+Common::Error SeqMusicPlugin::createInstance(Audio::Mixer *mixer, MidiDriver **mididriver) const {
 	*mididriver = new MidiDriver_SEQ();
 
-	return kNoError;
+	return Common::kNoError;
 }
 
 MidiDriver *MidiDriver_SEQ_create(Audio::Mixer *mixer) {

Modified: scummvm/trunk/backends/midi/stmidi.cpp
===================================================================
--- scummvm/trunk/backends/midi/stmidi.cpp	2008-11-05 15:41:12 UTC (rev 34905)
+++ scummvm/trunk/backends/midi/stmidi.cpp	2008-11-05 17:24:56 UTC (rev 34906)
@@ -119,7 +119,7 @@
         }
 
         MusicDevices getDevices() const;
-        PluginError createInstance(Audio::Mixer *mixer, MidiDriver **mididriver)
+        Common::Error createInstance(Audio::Mixer *mixer, MidiDriver **mididriver)
  const;
 };
 
@@ -131,10 +131,10 @@
         return devices;
 }
 
-PluginError StMidiMusicPlugin::createInstance(Audio::Mixer *mixer, MidiDriver **mididriver) const {
+Common::Error StMidiMusicPlugin::createInstance(Audio::Mixer *mixer, MidiDriver **mididriver) const {
         *mididriver = new MidiDriver_STMIDI();
 
-        return kNoError;
+        return Common::kNoError;
 }
 
 MidiDriver *MidiDriver_STMIDI_create(Audio::Mixer *mixer) {

Modified: scummvm/trunk/backends/midi/timidity.cpp
===================================================================
--- scummvm/trunk/backends/midi/timidity.cpp	2008-11-05 15:41:12 UTC (rev 34905)
+++ scummvm/trunk/backends/midi/timidity.cpp	2008-11-05 17:24:56 UTC (rev 34906)
@@ -525,7 +525,7 @@
 	}
 
 	MusicDevices getDevices() const;
-	PluginError createInstance(Audio::Mixer *mixer, MidiDriver **mididriver) const;
+	Common::Error createInstance(Audio::Mixer *mixer, MidiDriver **mididriver) const;
 };
 
 MusicDevices TimidityMusicPlugin::getDevices() const {
@@ -534,10 +534,10 @@
 	return devices;
 }
 
-PluginError TimidityMusicPlugin::createInstance(Audio::Mixer *mixer, MidiDriver **mididriver) const {
+Common::Error TimidityMusicPlugin::createInstance(Audio::Mixer *mixer, MidiDriver **mididriver) const {
 	*mididriver = new MidiDriver_TIMIDITY();
 
-	return kNoError;
+	return Common::kNoError;
 }
 
 MidiDriver *MidiDriver_TIMIDITY_create(Audio::Mixer *mixer) {

Modified: scummvm/trunk/backends/midi/windows.cpp
===================================================================
--- scummvm/trunk/backends/midi/windows.cpp	2008-11-05 15:41:12 UTC (rev 34905)
+++ scummvm/trunk/backends/midi/windows.cpp	2008-11-05 17:24:56 UTC (rev 34906)
@@ -158,7 +158,7 @@
 	}
 
 	MusicDevices getDevices() const;
-	PluginError createInstance(Audio::Mixer *mixer, MidiDriver **mididriver) const;
+	Common::Error createInstance(Audio::Mixer *mixer, MidiDriver **mididriver) const;
 };
 
 MusicDevices WindowsMusicPlugin::getDevices() const {
@@ -169,10 +169,10 @@
 	return devices;
 }
 
-PluginError WindowsMusicPlugin::createInstance(Audio::Mixer *mixer, MidiDriver **mididriver) const {
+Common::Error WindowsMusicPlugin::createInstance(Audio::Mixer *mixer, MidiDriver **mididriver) const {
 	*mididriver = new MidiDriver_WIN();
 
-	return kNoError;
+	return Common::kNoError;
 }
 
 MidiDriver *MidiDriver_WIN_create(Audio::Mixer *mixer) {

Modified: scummvm/trunk/backends/midi/ypa1.cpp
===================================================================
--- scummvm/trunk/backends/midi/ypa1.cpp	2008-11-05 15:41:12 UTC (rev 34905)
+++ scummvm/trunk/backends/midi/ypa1.cpp	2008-11-05 17:24:56 UTC (rev 34906)
@@ -117,7 +117,7 @@
 	}
 
 	MusicDevices getDevices() const;
-	PluginError createInstance(Audio::Mixer *mixer, MidiDriver **mididriver) const;
+	Common::Error createInstance(Audio::Mixer *mixer, MidiDriver **mididriver) const;
 };
 
 MusicDevices YamahaPa1MusicPlugin::getDevices() const {
@@ -128,10 +128,10 @@
 	return devices;
 }
 
-PluginError YamahaPa1MusicPlugin::createInstance(Audio::Mixer *mixer, MidiDriver **mididriver) const {
+Common::Error YamahaPa1MusicPlugin::createInstance(Audio::Mixer *mixer, MidiDriver **mididriver) const {
 	*mididriver = new MidiDriver_YamahaPa1();
 
-	return kNoError;
+	return Common::kNoError;
 }
 
 MidiDriver *MidiDriver_YamahaPa1_create(Audio::Mixer *mixer) {

Modified: scummvm/trunk/backends/midi/zodiac.cpp
===================================================================
--- scummvm/trunk/backends/midi/zodiac.cpp	2008-11-05 15:41:12 UTC (rev 34905)
+++ scummvm/trunk/backends/midi/zodiac.cpp	2008-11-05 17:24:56 UTC (rev 34906)
@@ -135,7 +135,7 @@
 	}
 
 	MusicDevices getDevices() const;
-	PluginError createInstance(Audio::Mixer *mixer, MidiDriver **mididriver) const;
+	Common::Error createInstance(Audio::Mixer *mixer, MidiDriver **mididriver) const;
 };
 
 MusicDevices ZodiacMusicPlugin::getDevices() const {
@@ -146,10 +146,10 @@
 	return devices;
 }
 
-PluginError ZodiacMusicPlugin::createInstance(Audio::Mixer *mixer, MidiDriver **mididriver) const {
+Common::Error ZodiacMusicPlugin::createInstance(Audio::Mixer *mixer, MidiDriver **mididriver) const {
 	*mididriver = new MidiDriver_Zodiac();
 
-	return kNoError;
+	return Common::kNoError;
 }
 
 MidiDriver *MidiDriver_Zodiac_create(Audio::Mixer *mixer) {

Modified: scummvm/trunk/base/main.cpp
===================================================================
--- scummvm/trunk/base/main.cpp	2008-11-05 15:41:12 UTC (rev 34905)
+++ scummvm/trunk/base/main.cpp	2008-11-05 17:24:56 UTC (rev 34906)
@@ -132,18 +132,18 @@
 
 	// Create the game engine
 	Engine *engine = 0;
-	PluginError err = (*plugin)->createInstance(&system, &engine);
-	if (!engine || err != kNoError) {
+	Common::Error err = (*plugin)->createInstance(&system, &engine);
+	if (!engine || err != Common::kNoError) {
 		// TODO: Show an error dialog or so?
 		// TODO: Also take 'err' into consideration...
 		//GUI::MessageDialog alert("ScummVM could not find any game in the specified directory!");
 		//alert.runModal();
 		const char *errMsg = 0;
 		switch (err) {
-		case kInvalidPathError:
+		case Common::kInvalidPathError:
 			errMsg = "Invalid game path";
 			break;
-		case kNoGameDataFoundError:
+		case Common::kNoGameDataFoundError:
 			errMsg = "Unable to locate game data";
 			break;
 		default:

Modified: scummvm/trunk/common/advancedDetector.cpp
===================================================================
--- scummvm/trunk/common/advancedDetector.cpp	2008-11-05 15:41:12 UTC (rev 34905)
+++ scummvm/trunk/common/advancedDetector.cpp	2008-11-05 17:24:56 UTC (rev 34906)
@@ -220,7 +220,7 @@
 	return detectedGames;
 }
 
-PluginError AdvancedMetaEngine::createInstance(OSystem *syst, Engine **engine) const {
+Common::Error AdvancedMetaEngine::createInstance(OSystem *syst, Engine **engine) const {
 	assert(engine);
 	upgradeTargetIfNecessary(params);
 

Modified: scummvm/trunk/common/advancedDetector.h
===================================================================
--- scummvm/trunk/common/advancedDetector.h	2008-11-05 15:41:12 UTC (rev 34905)
+++ scummvm/trunk/common/advancedDetector.h	2008-11-05 17:24:56 UTC (rev 34906)
@@ -201,7 +201,7 @@
 	virtual GameList getSupportedGames() const;
 	virtual GameDescriptor findGame(const char *gameid) const;
 	virtual GameList detectGames(const FSList &fslist) const;
-	virtual PluginError createInstance(OSystem *syst, Engine **engine) const;
+	virtual Common::Error createInstance(OSystem *syst, Engine **engine) const;
 
 	// To be provided by subclasses
 	virtual bool createInstance(OSystem *syst, Engine **engine, const Common::ADGameDescription *desc) const = 0;

Modified: scummvm/trunk/common/error.h
===================================================================
--- scummvm/trunk/common/error.h	2008-11-05 15:41:12 UTC (rev 34905)
+++ scummvm/trunk/common/error.h	2008-11-05 17:24:56 UTC (rev 34906)
@@ -32,6 +32,8 @@
 
 /**
  * Errors used in the SaveFileManager class.
+ *
+ * @todo	Merge this partially into Common::Error. We only need a small subset of these errors, though.
  */
 enum SFMError {
 	SFM_NO_ERROR,			//Default state, indicates no error has been recorded
@@ -44,17 +46,26 @@
 	SFM_DIR_ROFS			//mkdir()::EROFS: The parent directory resides on a read-only file system
 };
 
+namespace Common {
+
 /**
  * Error codes which may be reported by plugins under various circumstances.
- * @todo Turn this into a global 'ErrorCode' enum used by all of ScummVM ?
+ *
+ * @todo Clarify the names, and add doxygen comments to each error.
+ * @todo Add more error values, e.g. for load/save errors. Use those in SaveFileManager,
+ *       (Meta)Engine save/load API, Engine::init() and Engine::go(), ...
+ * @todo Maybe add an API which keeps track of an error message,
+ *       similiar to SDL_SetError/SDL_GetError/SDL_ClearError?
  */
-enum PluginError {
-	kNoError = 0,	// No error occured
-	kInvalidPathError,
-	kNoGameDataFoundError,
-	kUnsupportedGameidError,
+enum Error {
+	kNoError = 0,				//!< No error occured
+	kInvalidPathError,			//!< Engine initialization: Invalid game path was passed
+	kNoGameDataFoundError,		//!< Engine initialization: No game data was found in the specified location
+	kUnsupportedGameidError,	//!< Engine initialization: Gameid not supported by this (Meta)Engine
 
-	kUnknownError		// Catch-all error, used if no other error code matches
+	kUnknownError				//!< Catch-all error, used if no other error code matches
 };
 
+} // End of namespace Common
+
 #endif //COMMON_ERROR_H

Modified: scummvm/trunk/engines/metaengine.h
===================================================================
--- scummvm/trunk/engines/metaengine.h	2008-11-05 15:41:12 UTC (rev 34905)
+++ scummvm/trunk/engines/metaengine.h	2008-11-05 17:24:56 UTC (rev 34906)
@@ -75,9 +75,9 @@
 	 * @param syst	Pointer to the global OSystem object
 	 * @param engine	Pointer to a pointer which the MetaEngine sets to
 	 *					the newly create Engine, or 0 in case of an error
-	 * @return		a PluginError describing the error which occurred, or kNoError
+	 * @return		a Common::Error describing the error which occurred, or kNoError
 	 */
-	virtual PluginError createInstance(OSystem *syst, Engine **engine) const = 0;
+	virtual Common::Error createInstance(OSystem *syst, Engine **engine) const = 0;
 
 	/**
 	 * Return a list of all save states associated with the given target.

Modified: scummvm/trunk/engines/queen/queen.cpp
===================================================================
--- scummvm/trunk/engines/queen/queen.cpp	2008-11-05 15:41:12 UTC (rev 34905)
+++ scummvm/trunk/engines/queen/queen.cpp	2008-11-05 17:24:56 UTC (rev 34906)
@@ -65,7 +65,7 @@
 	virtual SaveStateList listSaves(const char *target) const;
 	virtual void removeSaveState(const char *target, int slot) const;
 
-	virtual PluginError createInstance(OSystem *syst, Engine **engine) const;
+	virtual Common::Error createInstance(OSystem *syst, Engine **engine) const;
 };
 
 const char *QueenMetaEngine::getName() const {
@@ -174,10 +174,10 @@
 	g_system->getSavefileManager()->removeSavefile(filename.c_str());
 }
 
-PluginError QueenMetaEngine::createInstance(OSystem *syst, Engine **engine) const {
+Common::Error QueenMetaEngine::createInstance(OSystem *syst, Engine **engine) const {
 	assert(engine);
 	*engine = new Queen::QueenEngine(syst);
-	return kNoError;
+	return Common::kNoError;
 }
 
 #if PLUGIN_ENABLED_DYNAMIC(QUEEN)

Modified: scummvm/trunk/engines/scumm/detection.cpp
===================================================================
--- scummvm/trunk/engines/scumm/detection.cpp	2008-11-05 15:41:12 UTC (rev 34905)
+++ scummvm/trunk/engines/scumm/detection.cpp	2008-11-05 17:24:56 UTC (rev 34906)
@@ -682,7 +682,7 @@
 	virtual GameDescriptor findGame(const char *gameid) const;
 	virtual GameList detectGames(const Common::FSList &fslist) const;
 	
-	virtual PluginError createInstance(OSystem *syst, Engine **engine) const;
+	virtual Common::Error createInstance(OSystem *syst, Engine **engine) const;
 
 	virtual SaveStateList listSaves(const char *target) const;
 	virtual void removeSaveState(const char *target, int slot) const;
@@ -765,7 +765,7 @@
  *
  * This is heavily based on our MD5 detection scheme.
  */
-PluginError ScummMetaEngine::createInstance(OSystem *syst, Engine **engine) const {
+Common::Error ScummMetaEngine::createInstance(OSystem *syst, Engine **engine) const {
 	assert(syst);
 	assert(engine);
 	const char *gameid = ConfMan.get("gameid").c_str();
@@ -792,7 +792,7 @@
 	Common::FSList fslist;
 	Common::FSNode dir(ConfMan.get("path"));
 	if (!dir.getChildren(fslist, Common::FSNode::kListFilesOnly)) {
-		return kInvalidPathError;
+		return Common::kInvalidPathError;
 	}
 
 	// Invoke the detector, but fixed to the specified gameid.
@@ -801,7 +801,7 @@
 
 	// Unable to locate game data
 	if (results.empty()) {
-		return kNoGameDataFoundError;
+		return Common::kNoGameDataFoundError;
 	}
 
 	// No unique match found. If a platform override is present, try to
@@ -943,7 +943,7 @@
 		error("Engine_SCUMM_create(): Unknown version of game engine");
 	}
 
-	return kNoError;
+	return Common::kNoError;
 }
 
 const char *ScummMetaEngine::getName() const {

Modified: scummvm/trunk/engines/sky/sky.cpp
===================================================================
--- scummvm/trunk/engines/sky/sky.cpp	2008-11-05 15:41:12 UTC (rev 34905)
+++ scummvm/trunk/engines/sky/sky.cpp	2008-11-05 17:24:56 UTC (rev 34906)
@@ -23,8 +23,6 @@
  *
  */
 
-
-
 #include "base/plugins.h"
 
 #include "common/config-manager.h"
@@ -115,7 +113,7 @@
 	virtual GameDescriptor findGame(const char *gameid) const;
 	virtual GameList detectGames(const Common::FSList &fslist) const;	
 
-	virtual PluginError createInstance(OSystem *syst, Engine **engine) const;
+	virtual Common::Error createInstance(OSystem *syst, Engine **engine) const;
 
 	virtual SaveStateList listSaves(const char *target) const;
 };
@@ -203,10 +201,10 @@
 	return detectedGames;
 }
 
-PluginError SkyMetaEngine::createInstance(OSystem *syst, Engine **engine) const {
+Common::Error SkyMetaEngine::createInstance(OSystem *syst, Engine **engine) const {
 	assert(engine);
 	*engine = new Sky::SkyEngine(syst);
-	return kNoError;
+	return Common::kNoError;
 }
 
 SaveStateList SkyMetaEngine::listSaves(const char *target) const {

Modified: scummvm/trunk/engines/sword1/sword1.cpp
===================================================================
--- scummvm/trunk/engines/sword1/sword1.cpp	2008-11-05 15:41:12 UTC (rev 34905)
+++ scummvm/trunk/engines/sword1/sword1.cpp	2008-11-05 17:24:56 UTC (rev 34906)
@@ -23,7 +23,6 @@
  *
  */
 
-
 #include "sword1/sword1.h"
 
 #include "base/plugins.h"
@@ -101,7 +100,7 @@
 	virtual GameList detectGames(const Common::FSList &fslist) const;
 	virtual SaveStateList listSaves(const char *target) const;
 
-	virtual PluginError createInstance(OSystem *syst, Engine **engine) const;
+	virtual Common::Error createInstance(OSystem *syst, Engine **engine) const;
 };
 
 bool SwordMetaEngine::hasFeature(MetaEngineFeature f) const {
@@ -195,10 +194,10 @@
 	return detectedGames;
 }
 
-PluginError SwordMetaEngine::createInstance(OSystem *syst, Engine **engine) const {
+Common::Error SwordMetaEngine::createInstance(OSystem *syst, Engine **engine) const {
 	assert(engine);
 	*engine = new SwordEngine(syst);
-	return kNoError;
+	return Common::kNoError;
 }
 
 SaveStateList SwordMetaEngine::listSaves(const char *target) const {

Modified: scummvm/trunk/engines/sword2/sword2.cpp
===================================================================
--- scummvm/trunk/engines/sword2/sword2.cpp	2008-11-05 15:41:12 UTC (rev 34905)
+++ scummvm/trunk/engines/sword2/sword2.cpp	2008-11-05 17:24:56 UTC (rev 34906)
@@ -87,7 +87,7 @@
 	virtual SaveStateList listSaves(const char *target) const;
 	virtual void removeSaveState(const char *target, int slot) const;
 
-	virtual PluginError createInstance(OSystem *syst, Engine **engine) const;
+	virtual Common::Error createInstance(OSystem *syst, Engine **engine) const;
 };
 
 bool Sword2MetaEngine::hasFeature(MetaEngineFeature f) const {
@@ -211,14 +211,14 @@
 	g_system->getSavefileManager()->removeSavefile(filename.c_str());
 }
 
-PluginError Sword2MetaEngine::createInstance(OSystem *syst, Engine **engine) const {
+Common::Error Sword2MetaEngine::createInstance(OSystem *syst, Engine **engine) const {
 	assert(syst);
 	assert(engine);
 
 	Common::FSList fslist;
 	Common::FSNode dir(ConfMan.get("path"));
 	if (!dir.getChildren(fslist, Common::FSNode::kListAll)) {
-		return kInvalidPathError;
+		return Common::kInvalidPathError;
 	}
 
 	// Invoke the detector
@@ -228,11 +228,11 @@
 	for (uint i = 0; i < detectedGames.size(); i++) {
 		if (detectedGames[i].gameid() == gameid) {
 			*engine = new Sword2::Sword2Engine(syst);
-			return kNoError;
+			return Common::kNoError;
 		}
 	}
 
-	return kNoGameDataFoundError;
+	return Common::kNoGameDataFoundError;
 }
 
 #if PLUGIN_ENABLED_DYNAMIC(SWORD2)

Modified: scummvm/trunk/sound/musicplugin.h
===================================================================
--- scummvm/trunk/sound/musicplugin.h	2008-11-05 15:41:12 UTC (rev 34905)
+++ scummvm/trunk/sound/musicplugin.h	2008-11-05 17:24:56 UTC (rev 34906)
@@ -100,9 +100,9 @@
 	 * @param mixer			Pointer to the global Mixer object
 	 * @param mididriver	Pointer to a pointer which the MusicPluginObject sets
 	 *				to the newly create MidiDriver, or 0 in case of an error
-	 * @return		a PluginError describing the error which occurred, or kNoError
+	 * @return		a Common::Error describing the error which occurred, or kNoError
 	 */
-	virtual PluginError createInstance(Audio::Mixer *mixer, MidiDriver **mididriver) const = 0;
+	virtual Common::Error createInstance(Audio::Mixer *mixer, MidiDriver **mididriver) const = 0;
 };
 
 

Modified: scummvm/trunk/sound/null.cpp
===================================================================
--- scummvm/trunk/sound/null.cpp	2008-11-05 15:41:12 UTC (rev 34905)
+++ scummvm/trunk/sound/null.cpp	2008-11-05 17:24:56 UTC (rev 34906)
@@ -46,13 +46,13 @@
 	}
 
 	MusicDevices getDevices() const;
-	PluginError createInstance(Audio::Mixer *mixer, MidiDriver **mididriver) const;
+	Common::Error createInstance(Audio::Mixer *mixer, MidiDriver **mididriver) const;
 };
 
-PluginError NullMusicPlugin::createInstance(Audio::Mixer *mixer, MidiDriver **mididriver) const {
+Common::Error NullMusicPlugin::createInstance(Audio::Mixer *mixer, MidiDriver **mididriver) const {
 	*mididriver = new MidiDriver_NULL();
 
-	return kNoError;
+	return Common::kNoError;
 }
 
 MusicDevices NullMusicPlugin::getDevices() const {

Modified: scummvm/trunk/sound/softsynth/adlib.cpp
===================================================================
--- scummvm/trunk/sound/softsynth/adlib.cpp	2008-11-05 15:41:12 UTC (rev 34905)
+++ scummvm/trunk/sound/softsynth/adlib.cpp	2008-11-05 17:24:56 UTC (rev 34906)
@@ -1529,7 +1529,7 @@
 	}
 
 	MusicDevices getDevices() const;
-	PluginError createInstance(Audio::Mixer *mixer, MidiDriver **mididriver) const;
+	Common::Error createInstance(Audio::Mixer *mixer, MidiDriver **mididriver) const;
 };
 
 MusicDevices AdlibEmuMusicPlugin::getDevices() const {
@@ -1538,10 +1538,10 @@
 	return devices;
 }
 
-PluginError AdlibEmuMusicPlugin::createInstance(Audio::Mixer *mixer, MidiDriver **mididriver) const {
+Common::Error AdlibEmuMusicPlugin::createInstance(Audio::Mixer *mixer, MidiDriver **mididriver) const {
 	*mididriver = new MidiDriver_ADLIB(mixer);
 
-	return kNoError;
+	return Common::kNoError;
 }
 
 MidiDriver *MidiDriver_ADLIB_create(Audio::Mixer *mixer) {

Modified: scummvm/trunk/sound/softsynth/fluidsynth.cpp
===================================================================
--- scummvm/trunk/sound/softsynth/fluidsynth.cpp	2008-11-05 15:41:12 UTC (rev 34905)
+++ scummvm/trunk/sound/softsynth/fluidsynth.cpp	2008-11-05 17:24:56 UTC (rev 34906)
@@ -230,7 +230,7 @@
 	}
 
 	MusicDevices getDevices() const;
-	PluginError createInstance(Audio::Mixer *mixer, MidiDriver **mididriver) const;
+	Common::Error createInstance(Audio::Mixer *mixer, MidiDriver **mididriver) const;
 };
 
 MusicDevices FluidSynthMusicPlugin::getDevices() const {
@@ -239,7 +239,7 @@
 	return devices;
 }
 
-PluginError FluidSynthMusicPlugin::createInstance(Audio::Mixer *mixer, MidiDriver **mididriver) const {
+Common::Error FluidSynthMusicPlugin::createInstance(Audio::Mixer *mixer, MidiDriver **mididriver) const {
 	*mididriver = new MidiDriver_FluidSynth(mixer);
 
 	return kNoError;

Modified: scummvm/trunk/sound/softsynth/mt32.cpp
===================================================================
--- scummvm/trunk/sound/softsynth/mt32.cpp	2008-11-05 15:41:12 UTC (rev 34905)
+++ scummvm/trunk/sound/softsynth/mt32.cpp	2008-11-05 17:24:56 UTC (rev 34906)
@@ -495,7 +495,7 @@
 	}
 
 	MusicDevices getDevices() const;
-	PluginError createInstance(Audio::Mixer *mixer, MidiDriver **mididriver) const;
+	Common::Error createInstance(Audio::Mixer *mixer, MidiDriver **mididriver) const;
 };
 
 MusicDevices MT32EmuMusicPlugin::getDevices() const {
@@ -504,10 +504,10 @@
 	return devices;
 }
 
-PluginError MT32EmuMusicPlugin::createInstance(Audio::Mixer *mixer, MidiDriver **mididriver) const {
+Common::Error MT32EmuMusicPlugin::createInstance(Audio::Mixer *mixer, MidiDriver **mididriver) const {
 	*mididriver = new MidiDriver_MT32(mixer);
 
-	return kNoError;
+	return Common::kNoError;
 }
 
 MidiDriver *MidiDriver_MT32_create(Audio::Mixer *mixer) {

Modified: scummvm/trunk/sound/softsynth/ym2612.cpp
===================================================================
--- scummvm/trunk/sound/softsynth/ym2612.cpp	2008-11-05 15:41:12 UTC (rev 34905)
+++ scummvm/trunk/sound/softsynth/ym2612.cpp	2008-11-05 17:24:56 UTC (rev 34906)
@@ -766,7 +766,7 @@
 	}
 
 	MusicDevices getDevices() const;
-	PluginError createInstance(Audio::Mixer *mixer, MidiDriver **mididriver) const;
+	Common::Error createInstance(Audio::Mixer *mixer, MidiDriver **mididriver) const;
 };
 
 MusicDevices TownsEmuMusicPlugin::getDevices() const {
@@ -775,10 +775,10 @@
 	return devices;
 }
 
-PluginError TownsEmuMusicPlugin::createInstance(Audio::Mixer *mixer, MidiDriver **mididriver) const {
+Common::Error TownsEmuMusicPlugin::createInstance(Audio::Mixer *mixer, MidiDriver **mididriver) const {
 	*mididriver = new MidiDriver_YM2612(mixer);
 
-	return kNoError;
+	return Common::kNoError;
 }
 
 MidiDriver *MidiDriver_YM2612_create(Audio::Mixer *mixer) {


This was sent by the SourceForge.net collaborative development platform, the world's largest Open Source development site.




More information about the Scummvm-git-logs mailing list