[Scummvm-cvs-logs] SF.net SVN: scummvm:[52112] scummvm/branches/gsoc2010-plugins

toneman1138 at users.sourceforge.net toneman1138 at users.sourceforge.net
Mon Aug 16 10:41:04 CEST 2010


Revision: 52112
          http://scummvm.svn.sourceforge.net/scummvm/?rev=52112&view=rev
Author:   toneman1138
Date:     2010-08-16 08:41:04 +0000 (Mon, 16 Aug 2010)

Log Message:
-----------
added a todo to plugins.cpp and a comment to elf32.h; Collapsed plugin providers for a few ports into their .h files, removing the corresponding .cpp files

Modified Paths:
--------------
    scummvm/branches/gsoc2010-plugins/backends/module.mk
    scummvm/branches/gsoc2010-plugins/backends/plugins/ds/ds-provider.h
    scummvm/branches/gsoc2010-plugins/backends/plugins/elf32.h
    scummvm/branches/gsoc2010-plugins/backends/plugins/ps2/ps2-provider.h
    scummvm/branches/gsoc2010-plugins/backends/plugins/psp/psp-provider.h
    scummvm/branches/gsoc2010-plugins/base/plugins.cpp

Removed Paths:
-------------
    scummvm/branches/gsoc2010-plugins/backends/plugins/ds/ds-provider.cpp
    scummvm/branches/gsoc2010-plugins/backends/plugins/ps2/ps2-provider.cpp
    scummvm/branches/gsoc2010-plugins/backends/plugins/psp/psp-provider.cpp

Modified: scummvm/branches/gsoc2010-plugins/backends/module.mk
===================================================================
--- scummvm/branches/gsoc2010-plugins/backends/module.mk	2010-08-16 04:27:05 UTC (rev 52111)
+++ scummvm/branches/gsoc2010-plugins/backends/module.mk	2010-08-16 08:41:04 UTC (rev 52112)
@@ -49,8 +49,7 @@
 ifeq ($(BACKEND),ds)
 MODULE_OBJS += \
 	fs/ds/ds-fs-factory.o \
-	fs/ds/ds-fs.o \
-	plugins/ds/ds-provider.o
+	fs/ds/ds-fs.o
 endif
 
 ifeq ($(BACKEND),n64)
@@ -61,15 +60,13 @@
 
 ifeq ($(BACKEND),ps2)
 MODULE_OBJS += \
-	fs/ps2/ps2-fs-factory.o \
-	plugins/ps2/ps2-provider.o
+	fs/ps2/ps2-fs-factory.o
 endif
 
 ifeq ($(BACKEND),psp)
 MODULE_OBJS += \
 	fs/psp/psp-fs-factory.o \
 	fs/psp/psp-stream.o \
-	plugins/psp/psp-provider.o \
 	saves/psp/psp-saves.o \
 	timer/psp/timer.o
 endif

Deleted: scummvm/branches/gsoc2010-plugins/backends/plugins/ds/ds-provider.cpp
===================================================================
--- scummvm/branches/gsoc2010-plugins/backends/plugins/ds/ds-provider.cpp	2010-08-16 04:27:05 UTC (rev 52111)
+++ scummvm/branches/gsoc2010-plugins/backends/plugins/ds/ds-provider.cpp	2010-08-16 08:41:04 UTC (rev 52112)
@@ -1,45 +0,0 @@
-/* ScummVM - Graphic Adventure Engine
- *
- * ScummVM is the legal property of its developers, whose names
- * are too numerous to list here. Please refer to the COPYRIGHT
- * file distributed with this source distribution.
- *
- * This program is free software; you can redistribute it and/or
- * modify it under the terms of the GNU General Public License
- * as published by the Free Software Foundation; either version 2
- * of the License, or (at your option) any later version.
-
- * This program is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
- * GNU General Public License for more details.
-
- * You should have received a copy of the GNU General Public License
- * along with this program; if not, write to the Free Software
- * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301, USA.
- *
- * $URL$
- * $Id$
- *
- */
-
-#if defined(DYNAMIC_MODULES) && defined(__DS__)
-
-#include "backends/plugins/arm-loader.h"
-#include "backends/plugins/elf-provider.h"
-#include "backends/plugins/ds/ds-provider.h"
-
-
-class DSPlugin : public ELFPlugin {
-public:
-	DSPlugin(const Common::String &filename) : ELFPlugin(filename) {}
-
-	DLObject *makeDLObject() { return new ARMDLObject(); }
-};
-
-Plugin* DSPluginProvider::createPlugin(const Common::FSNode &node) const {
-	return new DSPlugin(node.getPath());
-}
-
-#endif // defined(DYNAMIC_MODULES) && defined(ELF_LOADER_TARGET)
-

Modified: scummvm/branches/gsoc2010-plugins/backends/plugins/ds/ds-provider.h
===================================================================
--- scummvm/branches/gsoc2010-plugins/backends/plugins/ds/ds-provider.h	2010-08-16 04:27:05 UTC (rev 52111)
+++ scummvm/branches/gsoc2010-plugins/backends/plugins/ds/ds-provider.h	2010-08-16 08:41:04 UTC (rev 52112)
@@ -26,9 +26,20 @@
 #if defined(DYNAMIC_MODULES) && defined(__DS__)
 
 #include "backends/plugins/elf-provider.h"
+#include "backends/plugins/arm-loader.h"
 
 class DSPluginProvider : public ELFPluginProvider {
-	Plugin* createPlugin(const Common::FSNode &node) const;
+	class DSPlugin : public ELFPlugin {
+	public:
+		DSPlugin(const Common::String &filename) : ELFPlugin(filename) {}
+
+		DLObject *makeDLObject() { return new ARMDLObject(); }
+	};
+
+public:
+	Plugin* createPlugin(const Common::FSNode &node) const {
+		return new DSPlugin(node.getPath());
+	}
 };
 
-#endif // defined(DYNAMIC_MODULES) && defined(ELF_LOADER_TARGET)
+#endif // defined(DYNAMIC_MODULES) && defined(__DS__)

Modified: scummvm/branches/gsoc2010-plugins/backends/plugins/elf32.h
===================================================================
--- scummvm/branches/gsoc2010-plugins/backends/plugins/elf32.h	2010-08-16 04:27:05 UTC (rev 52111)
+++ scummvm/branches/gsoc2010-plugins/backends/plugins/elf32.h	2010-08-16 08:41:04 UTC (rev 52112)
@@ -26,7 +26,11 @@
 #ifndef BACKENDS_ELF_H
 #define BACKENDS_ELF_H
 
-/* ELF stuff */
+/**
+ *  ELF stuff:
+ *  The contents of this file were gathered mainly from the SYSTEM V APPLICATION BINARY INTERFACE.
+ *  Processor-specific things were garnered from processor-specific supplements to the abi.
+ */
 
 typedef unsigned short Elf32_Half, Elf32_Section;
 typedef unsigned int Elf32_Word, Elf32_Addr, Elf32_Off;

Deleted: scummvm/branches/gsoc2010-plugins/backends/plugins/ps2/ps2-provider.cpp
===================================================================
--- scummvm/branches/gsoc2010-plugins/backends/plugins/ps2/ps2-provider.cpp	2010-08-16 04:27:05 UTC (rev 52111)
+++ scummvm/branches/gsoc2010-plugins/backends/plugins/ps2/ps2-provider.cpp	2010-08-16 08:41:04 UTC (rev 52112)
@@ -1,44 +0,0 @@
-/* ScummVM - Graphic Adventure Engine
- *
- * ScummVM is the legal property of its developers, whose names
- * are too numerous to list here. Please refer to the COPYRIGHT
- * file distributed with this source distribution.
- *
- * This program is free software; you can redistribute it and/or
- * modify it under the terms of the GNU General Public License
- * as published by the Free Software Foundation; either version 2
- * of the License, or (at your option) any later version.
-
- * This program is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
- * GNU General Public License for more details.
-
- * You should have received a copy of the GNU General Public License
- * along with this program; if not, write to the Free Software
- * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301, USA.
- *
- * $URL$
- * $Id$
- *
- */
-
-#if defined(DYNAMIC_MODULES) && defined(__PLAYSTATION2__)
-
-#include "backends/plugins/mips-loader.h"
-#include "backends/plugins/elf-provider.h"
-#include "backends/plugins/ps2/ps2-provider.h"
-
-
-class PS2Plugin : public ELFPlugin {
-public:
-	PS2Plugin(const Common::String &filename) : ELFPlugin(filename) {}
-
-	DLObject *makeDLObject() { return new MIPSDLObject(); }
-};
-
-Plugin* PS2PluginProvider::createPlugin(const Common::FSNode &node) const {
-	return new PS2Plugin(node.getPath());
-}
-
-#endif // defined(DYNAMIC_MODULES) && defined(ELF_LOADER_TARGET)

Modified: scummvm/branches/gsoc2010-plugins/backends/plugins/ps2/ps2-provider.h
===================================================================
--- scummvm/branches/gsoc2010-plugins/backends/plugins/ps2/ps2-provider.h	2010-08-16 04:27:05 UTC (rev 52111)
+++ scummvm/branches/gsoc2010-plugins/backends/plugins/ps2/ps2-provider.h	2010-08-16 08:41:04 UTC (rev 52112)
@@ -26,10 +26,21 @@
 #if defined(DYNAMIC_MODULES) && defined(__PLAYSTATION2__)
 
 #include "backends/plugins/elf-provider.h"
+#include "backends/plugins/mips-loader.h"
 
 class PS2PluginProvider : public ELFPluginProvider {
-	Plugin* createPlugin(const Common::FSNode &node) const;
+	class PS2Plugin : public ELFPlugin {
+	public:
+		PS2Plugin(const Common::String &filename) : ELFPlugin(filename) {}
+
+		DLObject *makeDLObject() { return new MIPSDLObject(); }
+	};
+
+public:
+	Plugin* PS2PluginProvider::createPlugin(const Common::FSNode &node) const {
+		return new PS2Plugin(node.getPath());
+	}
 };
 
-#endif // defined(DYNAMIC_MODULES) && defined(ELF_LOADER_TARGET)
+#endif // defined(DYNAMIC_MODULES) && defined(__PLAYSTATION2__)
 

Deleted: scummvm/branches/gsoc2010-plugins/backends/plugins/psp/psp-provider.cpp
===================================================================
--- scummvm/branches/gsoc2010-plugins/backends/plugins/psp/psp-provider.cpp	2010-08-16 04:27:05 UTC (rev 52111)
+++ scummvm/branches/gsoc2010-plugins/backends/plugins/psp/psp-provider.cpp	2010-08-16 08:41:04 UTC (rev 52112)
@@ -1,44 +0,0 @@
-/* ScummVM - Graphic Adventure Engine
- *
- * ScummVM is the legal property of its developers, whose names
- * are too numerous to list here. Please refer to the COPYRIGHT
- * file distributed with this source distribution.
- *
- * This program is free software; you can redistribute it and/or
- * modify it under the terms of the GNU General Public License
- * as published by the Free Software Foundation; either version 2
- * of the License, or (at your option) any later version.
-
- * This program is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
- * GNU General Public License for more details.
-
- * You should have received a copy of the GNU General Public License
- * along with this program; if not, write to the Free Software
- * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301, USA.
- *
- * $URL$
- * $Id$
- *
- */
-
-#if defined(DYNAMIC_MODULES) && defined(__PSP__)
-
-#include "backends/plugins/mips-loader.h"
-#include "backends/plugins/elf-provider.h"
-#include "backends/plugins/psp/psp-provider.h"
-
-
-class PSPPlugin : public ELFPlugin {
-public:
-	PSPPlugin(const Common::String &filename) : ELFPlugin(filename) {}
-
-	DLObject *makeDLObject() { return new MIPSDLObject(); }
-};
-
-Plugin* PSPPluginProvider::createPlugin(const Common::FSNode &node) const {
-	return new PSPPlugin(node.getPath());
-}
-
-#endif // defined(DYNAMIC_MODULES) && defined(__PSP__)

Modified: scummvm/branches/gsoc2010-plugins/backends/plugins/psp/psp-provider.h
===================================================================
--- scummvm/branches/gsoc2010-plugins/backends/plugins/psp/psp-provider.h	2010-08-16 04:27:05 UTC (rev 52111)
+++ scummvm/branches/gsoc2010-plugins/backends/plugins/psp/psp-provider.h	2010-08-16 08:41:04 UTC (rev 52112)
@@ -29,9 +29,21 @@
 #define BACKENDS_PLUGINS_PSP_PSP_PROVIDER_H
 
 #include "backends/plugins/elf-provider.h"
+#include "backends/plugins/mips-loader.h"
 
 class PSPPluginProvider : public ELFPluginProvider {
-	Plugin* createPlugin(const Common::FSNode &node) const;
+	class PSPPlugin : public ELFPlugin {
+	public:
+		PSPPlugin(const Common::String &filename) : ELFPlugin(filename) {}
+
+		DLObject *makeDLObject() { return new MIPSDLObject(); }
+	};
+
+public:
+	Plugin* PSPPluginProvider::createPlugin(const Common::FSNode &node) const {
+		return new PSPPlugin(node.getPath());
+	}
+}
 };
 
 #endif /* BACKENDS_PLUGINS_PSP_PSP_PROVIDER_H */

Modified: scummvm/branches/gsoc2010-plugins/base/plugins.cpp
===================================================================
--- scummvm/branches/gsoc2010-plugins/base/plugins.cpp	2010-08-16 04:27:05 UTC (rev 52111)
+++ scummvm/branches/gsoc2010-plugins/base/plugins.cpp	2010-08-16 08:41:04 UTC (rev 52112)
@@ -329,7 +329,7 @@
 	//this loop is for loading all non-engine plugins and the first engine plugin.
 	while (true) {
 		assert(tryLoadPlugin(*_currentPlugin));
-		if ((*_currentPlugin)->getType() == PLUGIN_TYPE_ENGINE) {
+		if ((*_currentPlugin)->getType() == PLUGIN_TYPE_ENGINE) { //TODO: This assumes all non-engine plugins will precede the first engine plugin!
 			break;
 		}
 		_nonEnginePlugs++;


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