Difference between revisions of "User:Fingolfin"

From ScummVM :: Wiki
Jump to navigation Jump to search
(Updated my personal TODO list)
Line 60: Line 60:
  
 
=== Misc ===
 
=== Misc ===
* It might be quite helpful if we followed some consistent naming patterns for functions/methods. For example, a method which return an object pointer and expects the calling code to delete the object after use, should be named makeFOO; so for example, FSNode::openForReading() would become FSNode::makeReadStream()
+
* Follow some consistent naming patterns for functions/methods:
 +
** A method which return an object pointer and expects the calling code to delete the object after use, should be named makeFOO; so for example, FSNode::openForReading() would become FSNode::makeReadStream()
 +
** A method which is supposed to free/delete/dellocate something should have a consistent name. Maybe freeFOO? deallocFOO? destroyFOO?
 
* Remove the 'addDefaultDirectory' calls from runGame in base/main.cpp. Reason: Their presence causes an asymmetry between the "detect a game" and "run a game" use cases, as different files are "seen" in each case. This can lead to subtle bugs, and also causes ugly code duplication in the AdvancedDetector code right now (there to work around asymmetries like this one)
 
* Remove the 'addDefaultDirectory' calls from runGame in base/main.cpp. Reason: Their presence causes an asymmetry between the "detect a game" and "run a game" use cases, as different files are "seen" in each case. This can lead to subtle bugs, and also causes ugly code duplication in the AdvancedDetector code right now (there to work around asymmetries like this one)
 
* add plugin API to fetch a fresh desc string for a given target ?!?
 
* add plugin API to fetch a fresh desc string for a given target ?!?

Revision as of 19:18, 4 November 2008

Fingolfin
Name Max Horn
Team Member since 2002-04-26
Working on SCUMM Engine, SDL backend,
infrastructure
Personal webpage/BLOG -
Email fingolfin AT scummvm DOT org

I am co-lead developer of ScummVM, together with Ender and Sev. I joined the project in April 2002. During this time I worked on the SCUMM engine, the SDL backend, the OSystem backend interface, the sound mixer, the GUI system, the general infrastructure of ScummVM, and many other things.

TODO

Usability

  • Check for the presence of all required data files in all games; or, as a somewhat less extensive measure, at least do this: For games which were shipped on multiple volumes (floppies, CDs), check for the presence of at least one file per volume.
  • Check for the presence of our external support files like kyra.dat, sky.cpt, etc., and also verify that they are up-to-date. For details, see FR #1520433
  • Add/unify "confirm exit" dialog, globally (see FR #1731025)
  • Add global main menu dialog (see also below under GUI)

Web site

  • Add big (green?) shiny buttons in the middle of the page for (1) Donations and (2) Downloads ?
  • simplify screenshots: use a single screenshot "database" file, and use meaningful names on the screenshots, maybe even organize them in subdirs
  • Show "Release Date" on download page
  • Use AJAX for screenshot display?
  • Use AJAX to improve the downloads page, by only showing the most important ports/downloads by default, and then uncovering the rest upon a click by the user (with gracious fallback if Javascript is disabled).

Savefile manager

  • Consider replacing "const char *" with "Common::String" in method parameters. This might or might not be a good idea, though -- don't just do it blindly!

GUI

  • The options dialog may show a button for configuring the savepath even on systems where it is fixed -> not good. This button should be hidden/removed for these systems
  • Highlight the "default" button in dialogs (e.g. the classic MacOS way, drawing a fat border around it; or by using different coloring; or a combination). This falls under "usability", too.
  • Global Main Menu
    • Allow engines to extend this more freely. E.g. for the SCUMM engine, we would want to add a "Help" button
    • Make it possible to reach the "key remapper" and "virtual keyboard" from the GMM, if available?
    • Sugar on the cake: In addition to the ScummVM logo and version, how about showing the engine name and game title at the top of the GMM?

OSystem

  • get to rid of the evil global gBitFormat!
  • Remove slack in OSystem
    • move getScreenChangeID functionality to EventManager
  • Implement the RFC: Flexible keymapping via new EVENT_ (post 0.10)
  • Change backends to use namespaces ?! (Not really that useful, I guess, except for the Doxygen pages)

Sound system

Mixer

  • document the semantics of sound handles and sound ids
  • cleanup the API:
    • get rid of the evil Mixer::setReady hack
    • We could make it possible to change the output sample rate the mixer uses on the fly (this would be a private API, only available to the backend, for backends which can change the output sample rate). This would essentially require changing the Channel::_converter objects
  • Are there backends which could benefit from a 'custom' mixer? If so, what would be required to implement such a thing? Needs talking to porters

Audio CD handling

  • add a "pause" feature to the AudioCDManager (might require us to extend the OSystem CD API, too). Useful to be able to fully pause the currently running engine

SCUMM

  • Once the new SaveFileManager::remove method is there, add a "remove" button to the save/load dialog
  • Work around actor (re)color bugs in Indy3 FM-TOWNS / VGA. See http://home.comcast.net/~ervind/ij3misc.html
  • Might also be nice to use the trick they used in AGI/Sarien to add custom recoloring to costumes at least for the EGA games. E.g. in MM, an actor with blue trousers working on a blue ground looks... stealthy :)

Misc

  • Follow some consistent naming patterns for functions/methods:
    • A method which return an object pointer and expects the calling code to delete the object after use, should be named makeFOO; so for example, FSNode::openForReading() would become FSNode::makeReadStream()
    • A method which is supposed to free/delete/dellocate something should have a consistent name. Maybe freeFOO? deallocFOO? destroyFOO?
  • Remove the 'addDefaultDirectory' calls from runGame in base/main.cpp. Reason: Their presence causes an asymmetry between the "detect a game" and "run a game" use cases, as different files are "seen" in each case. This can lead to subtle bugs, and also causes ugly code duplication in the AdvancedDetector code right now (there to work around asymmetries like this one)
  • add plugin API to fetch a fresh desc string for a given target ?!?
  • completely get rid of #pragma pack for increased portability (what I mean here is not to simply remove it, but rather to change code which relies on structs to be packed to not do so anymore)
  • rename scummsys.h; and/or split it (types.h, defs.h ... ?)
  • Implement a "Main" class, from which backends derive. The "Main" of each backend then would become:
int main() {
	Main *m = new MyCustomMain();
	int retval = MyCustomMain->run();
	delete MyCustomMain;
	return retval;
}

class Main {
	StringMap settings;
	String command;
	String specialDebug;
	
	OSystem *system;
public:
	virtual int run(int argc, const char *argv[]) {
		registerDefaults();
		parseCommandLine(argc, argv);
			// A port w/o command line would simply set settings to a more suitable value
	
		loadConfig();
		PluginManager::instance().loadPlugins();

		system.initBackend();

		system.setWindowCaption(gScummVMFullVersion);
	}

	int registerDefaults()
	int parseCommandLine(int argc, const char *argv[]);
	void loadConfig();
	int processSettings();
	
	int runLauncherDialog();

};

Wish list

ScummVM supports many many games. Not just that, it also supports many variants of each of these games. Unfortunately, often bugs will only occur in one specific variant of a game, so unless you happen to own that particular version, it is very difficult to fix the problem (think of surgery while being blindfolded, with one hand tied behind your back).

In particular, being from Germany, I often only own the german version of a given game. While that is on the one hand useful (we have quite a big user base in Germany, it seems), the drawback is that often I lack an english variant, making it that much harder to work on bugs reported by English users.

Due to all this, here is a list of games resp. game variants that I lack and which might be useful. If you have a spare copy of any of these, feel free to donate it. I am only interested in originals, though, no warez! In addition, I also try from time to time to obtain these games e.g. via Ebay, so make sure to ask me before sending me anything, just in case I bought it in the meantime :-)

  • SCUMM
    • Zak McKracken: FM-TOWNS (OK, I am probably being unrealistic here :-) ), classic (V1) English, any other languages
    • Indiana Jones and the last Crusade: EGA German
    • Loom: FM-TOWNS (see above), EGA German
    • Monkey Island: EGA (*any* language, English or German preferred)
    • Indiana Jones and the Fate of Atlantis: Mac version
    • Sam & Max: German PC
    • The Dig: Mac version
  • Others
    • Any AGI games