415
edits
Eriktorbjorn (talk | contribs) (Add note about missing Broken Sword 2 feature.) |
|||
(82 intermediate revisions by 19 users not shown) | |||
Line 1: | Line 1: | ||
{{Infobox_TODO| | |||
taskname=Main TODO List| | |||
techcontact=The ScummVM Team| | |||
subsystem=Generic| | |||
}} | |||
Here is a list of things we consider doing in the future. Note that we don't promise to do any of these, nor when we will do them. It's just a list of what we hope to be able to do one day, and some items on the list really are more like ideas than real proposals, so take it with a grain of salt. | Here is a list of things we consider doing in the future. Note that we don't promise to do any of these, nor when we will do them. It's just a list of what we hope to be able to do one day, and some items on the list really are more like ideas than real proposals, so take it with a grain of salt. | ||
Line 10: | Line 16: | ||
== | == Documentation == | ||
See also the [[Documentation/TODO]] page. | See also the [[Documentation/TODO]] page. | ||
=== General === | === General === | ||
* Write a high level overview of how ScummVM and its engines work? | * Write a high level overview of how ScummVM and its engines work? | ||
=== | == Web sites == | ||
=== | === [[Buildbot]] === | ||
=== | === [[Doxygen]] === | ||
* Make it run again, upgrade | |||
* | |||
== Common code, infrastructure == | == Common code, infrastructure == | ||
=== Audio === | === Audio === | ||
==== Mixer ==== | ==== Mixer ==== | ||
* the "isReady" code is irritating. Most engines ignore it anyway. It's primary purpose is to allow ScummVM to run even if not digitial audio out is available. A better solution would likely be to enforce that all backends always return a valid mixer. In the worst case, if a backend cannot produce audio output, it should still instantiate an Audio::MixerImpl, with some arbitrary sample rate (I recommend 22050 Hz), and a fake playback thread, which acts like an audio output callback but simply discards the generated audio data. (Note: It doesn't suffice to just throw away input data in Mixer::playInputStream, as this breaks with QueuingAudioStream). | |||
* the "isReady" code is irritating. Most engines ignore it anyway. It's primary purpose is to allow ScummVM to run even if not digitial audio out is available. A better solution would likely be to | |||
==== CD ==== | ==== CD ==== | ||
* Consider extending the Audio CD Manager with WAV or VOC support (essentially, the WAV/VOC code would need to be enhanced by adding a factory function similar to that provided for the FLAC/Vorbis/MP3 streams). | * Consider extending the Audio CD Manager with WAV or VOC support (essentially, the WAV/VOC code would need to be enhanced by adding a factory function similar to that provided for the FLAC/Vorbis/MP3 streams). | ||
* 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 | |||
==== MIDI ==== | ==== MIDI ==== | ||
* See also [[Music drivers redesign]] | * See also [[Music drivers redesign]] | ||
* MIDI: Add API to OSystem which allows client code to query for "native" MIDI drivers (like Windows, ALSA, Zodiac, CoreAudio). Then change the MIDI interfacing code (GUI, command line, config file, etc.) and also MidiDriver::createMidi() to use both that list and the list of 'emulated' MIDI devices ( | * MIDI: Add API to OSystem which allows client code to query for "native" MIDI drivers (like Windows, ALSA, Zodiac, CoreAudio). Then change the MIDI interfacing code (GUI, command line, config file, etc.) and also MidiDriver::createMidi() to use both that list and the list of 'emulated' MIDI devices (AdLib, MT-32, PC Speaker, etc.) in an appropriate way. | ||
* Make it possible to pass options to the midi/music drivers. This would allow us to get rid of the getenv hacks in the alsa & seq drivers. | * Make it possible to pass options to the midi/music drivers. This would allow us to get rid of the getenv hacks in the alsa & seq drivers. | ||
**A simple approach would be to allow the user to specify "extended" driver descriptions. E.g. "alsa:1234" could indicate the alsa driver with port 1234; alternatively, "alsa:port=1234". | **A simple approach would be to allow the user to specify "extended" driver descriptions. E.g. "alsa:1234" could indicate the alsa driver with port 1234; alternatively, "alsa:port=1234". | ||
**This doesn't allow for easy tweaking of those settings via the GUI.... to achieve that, one approach would be to invent a 'parameter description language' which allows a driver to specify which options with which kind of values it supports... very flexible, but possibly overkill. | **This doesn't allow for easy tweaking of those settings via the GUI.... to achieve that, one approach would be to invent a 'parameter description language' which allows a driver to specify which options with which kind of values it supports... very flexible, but possibly overkill. | ||
**Instead, one could add a hook method to MIDI drivers that let's them add a few controls of their own to the options dialog. This is probably a lot easier to implement, though not quite as "clean"... | **Instead, one could add a hook method to MIDI drivers that let's them add a few controls of their own to the options dialog. This is probably a lot easier to implement, though not quite as "clean"... | ||
* Our XMIDI parser seems to have a couple of missing features, but it's unclear which ones are actually needed. Possible references for a better implementation include the [http://exult.sf.net Exult] project and the [http://www.thegleam.com/ke5fx/ source code for AIL Version 2]. | * Our XMIDI parser seems to have a couple of missing features, but it's unclear which ones are actually needed. Possible references for a better implementation include the [http://exult.sf.net Exult] project and the [http://www.thegleam.com/ke5fx/ source code for AIL Version 2]. The KYRA engine contains some more complete XMIDI implementation on top of our XMIDI parser class too, so it might also be a place to look at for reference. | ||
=== Config Manager === | === Config Manager === | ||
* Modify the ConfigManager to make use of the ConfigFile class. | * Modify the ConfigManager to make use of the ConfigFile class. This is more difficult than it sounds at first, because currently our ConfigManager is not just used for managing the active config, but also ab-used for editing config data (once for specific targets, in the launcher's "edit game" dialog, and once for editing the "active config" in the engines/dialog.cpp ConfigDialog). This mess needs to be untangled first. | ||
* Maybe even follow the pentagram (http://pentagram.sf.net) approach and have three classes: | * Maybe even follow the pentagram (http://pentagram.sf.net) approach and have three classes: | ||
** SettingsManager (like our current ConfigManager) | ** SettingsManager (like our current ConfigManager) | ||
Line 118: | Line 57: | ||
** ConfigFile (a simple .ini file accessor). | ** ConfigFile (a simple .ini file accessor). | ||
*: This makes it easy to add additional config sources (e.g. XMLConfigFile); makes it possible to treat the command line data like another config file (CommandLineConfig); and simply follows the good old MVC approach, which is always a good idea. | *: This makes it easy to add additional config sources (e.g. XMLConfigFile); makes it possible to treat the command line data like another config file (CommandLineConfig); and simply follows the good old MVC approach, which is always a good idea. | ||
* Add a proper version to the Config file, which indicates true changes in the format / data in it. Right now, we only have a "fake" version number where any ScummVM instance will just write its version in. Which is totally useless. A proper version would allow us to ... | |||
** ... produce a warning if an older ScummVM version tries to load a config file written by a newer version with potentially incompatible changes (such as the switch from language code "hb" to "he" for Hebrew) | |||
* | ** ... detect when we an old config file is loaded which needs to be updated (e.g. by changing language "hb" to "he"; by adding "guioptions" to targets; by fixing trailing slashes in "path"; maybe even updating old style; and maybe in the future by adding "engineid" fields to all targets) | ||
* | |||
=== GUI === | === GUI === | ||
Line 132: | Line 70: | ||
=== Launcher === | === Launcher === | ||
* Enhance the Mass detector to show a list with the results (and optionally, allow the user to edit/cull that list before adding it). | * Enhance the Mass detector to show a list with the results (and optionally, allow the user to edit/cull that list before adding it). | ||
* Make the gameid editable via an "advanced" tab entry? | * Make the gameid editable via an "advanced" tab entry? | ||
* | * The global 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 | ||
* separate launcher code even more from rest of ScummVM, to make custom launchers easier? | |||
** maybe separate launcher using MVC approach? Separate code which scans for games etc. from the presentation layer, to make it easier to write custom launchers with behavior matching that of the default launcher? | |||
* show cover art in launcher | |||
** this would only be done on "high end" systems, must be possible to disable code | |||
** we can't ship artwork directly, due to copyright concerns; so only ship artwork where it is legally possible, and otherwise allow users to setup "artwork packs" | |||
** control what artwork is shown using a config key | |||
=== Global Main Menu/Return to Launcher === | === Global Main Menu/Return to Launcher === | ||
* Double | * Confirm exit dialog woes: | ||
** Double "confirm exit" dialog: When confirm exit is enabled in ScummVM, the ScummVM quit confirm dialog shows up in addition to the quit confirm dialogs for those games which have their own. For example, when you select 'Quit' from Beneath a Steel Sky's in-game menu, it asks you if you really want to quit. Then, after answering, ScummVM gives you another dialog asking you if you really want to quit. So, ScummVM should not show the quit confirm dialog if the in-game dialog has already been displayed. This affects any game which has it's own menu system from where you can quit, or has a native quit confirm dialog. So this means most games. | |||
** Add/unify "confirm exit" dialog, globally (see [https://sourceforge.net/tracker/index.php?func=detail&aid=1731025&group_id=37116&atid=418823 FR #1731025]) | |||
* Options Dialog: Implement engine flags to define which features in the Options dialog are supported by the current running game. Use these flags to hide/show the appropriate sliders and buttons in the Options Dialog | * Options Dialog: Implement engine flags to define which features in the Options dialog are supported by the current running game. Use these flags to hide/show the appropriate sliders and buttons in the Options Dialog | ||
* Improve the look of the GMM: This could include displaying the engine name and the | * Improve the look of the GMM: This could include displaying the engine name and the game name on the top of the GMM. | ||
* Config Dialog code: Resolve the FIXME in engines/dialogs.cpp which pertains to having to use the empty string as the domain name. | * Config Dialog code: Resolve the FIXME in engines/dialogs.cpp which pertains to having to use the empty string as the domain name. This is a bigger task, but will enable many other changes (thus as finally changing ConfigManager to use ConfigFile). Essentially, the current game config dialog tries to abuse the system by editing the *active* settings via the config manager, in order to fully reduce the options dialog code from the launcher. But there is a big difference between editing the config settings of a specific target, and the active settings. So, one probably needs to rewrite the code for the config dialog shown from the GMM | ||
=== Error handling === | |||
Sometimes, bugs are reported that are not easily reproducible, e.g. [https://sourceforge.net/p/scummvm/bugs/6751/ Bug #6751]. In such cases, it would be useful to have the error handler in ScummVM generate (and, optionally, automatically send) minidumps that can be used by us to successfully examine crashed ScummVM processes. Integration of something like [https://chromium.googlesource.com/breakpad/breakpad/ Google Breakpad] is one reasonable option. While it is not likely to be able to get such crash handling into all ports, just supporting the most popular platforms would be enough to make this feature very useful. | |||
== Engines / frontends == | == Engines / frontends == | ||
Line 158: | Line 105: | ||
=== [[Cine]] === | === [[Cine]] === | ||
* see [[Cine/TODO|Cine TODO]] list | * see [[Cine/TODO|Cine TODO]] list | ||
=== [[CruisE]] === | |||
* see [[Cruise/TODO|CruisE TODO]] list | |||
=== [[Drascula]] === | === [[Drascula]] === | ||
Line 164: | Line 114: | ||
=== [[Gob]] === | === [[Gob]] === | ||
* see [[Gob/TODO|Gob TODO]] list | * see [[Gob/TODO|Gob TODO]] list | ||
=== [[Groovie]] === | |||
* see [[Groovie/TODO|Groovie TODO]] list | |||
=== [[Hopkins]] === | |||
* see [[Hopkins/TODO|Hopkins TODO]] list | |||
=== [[Hugo]] === | |||
* see [[Hugo/TODO|Hugo TODO]] list | |||
=== [[Kyra]] === | === [[Kyra]] === | ||
* see [[Kyra/TODO|Kyra TODO]] list | * see [[Kyra/TODO|Kyra TODO]] list | ||
=== [[Lastexpress]] === | |||
* see [[Lastexpress/TODO|Lastexpress TODO]] list | |||
=== [[MADE]] === | === [[MADE]] === | ||
* see [[MADE/TODO|MADE TODO]] list | * see [[MADE/TODO|MADE TODO]] list | ||
=== [[Mohawk]] === | |||
* see [[Mohawk/TODO|Mohawk TODO]] list | |||
=== [[Parallaction]] === | === [[Parallaction]] === | ||
Line 176: | Line 141: | ||
=== [[SAGA]] === | === [[SAGA]] === | ||
* see [[SAGA/TODO|SAGA TODO]] list | * see [[SAGA/TODO|SAGA TODO]] list | ||
=== [[SCI]] === | |||
* see [[SCI/TODO|SCI TODO]] list | |||
=== [[Sword1|Broken Sword 1]] === | === [[Sword1|Broken Sword 1]] === | ||
* Modify MoviePlayer class to use ManagedArray for _movieTexts. | * Modify MoviePlayer class to use ManagedArray for _movieTexts. | ||
* Add support for Bink RDFT compressed audio in the Smacker videos (used in some later releases). | |||
=== [[Sword2|Broken Sword 2]] === | === [[Sword2|Broken Sword 2]] === | ||
* Fix the credits so they look more like the original. (Did we ever get the source code for that?) | * Fix the credits so they look more like the original. (Did we ever get the source code for that?) | ||
* Maybe work around script bug which causes the mop to disappear briefly when trying to pick it up from the top of the boat at the London docks. (The event to hide the mop is sent too early.) | * Maybe work around script bug which causes the mop to disappear briefly when trying to pick it up from the top of the boat at the London docks. (The event to hide the mop is sent too early.) {{Tracker|id=2058}}. | ||
* Maybe (but probably not) work around a graphics glitch where George can be drawn partially in front of a metal beam he's walking behind. (He's obscured properly by the beam sprite, but part of the beam is only background.) {{Tracker|id=4483}}. | |||
* Unify some of the code with Broken Sword 1 (e.g. in router.cpp). | * Unify some of the code with Broken Sword 1 (e.g. in router.cpp). | ||
*Lower the music volume when a character is speaking, like the original did. | |||
=== [[Sword25]] === | |||
* see [[Sword25/TODO|Sword25 TODO]] list | |||
=== [[Tinsel]] === | |||
* see [[Tinsel/TODO|Tinsel TODO]] list | |||
=== [[Tony]] === | |||
* see [[Tony/TODO|Tony TODO]] list | |||
=== [[Toon]] === | |||
* see [[Toon/TODO|Tooon TODO]] list | |||
=== [[Touche]] === | === [[Touche]] === | ||
* see [[Touche/TODO|Touche TODO]] list | * see [[Touche/TODO|Touche TODO]] list | ||
== Backends == | === [[TsAGE]] === | ||
* see [[TsAGE/TODO|TsAGE TODO]] list | |||
=== [[Tucker]] === | |||
* see [[Tucker/TODO|Tucker TODO]] list | |||
=== [[Wintermute]] === | |||
* see [[Wintermute/TODO|Wintermute TODO]] list | |||
== Backends and ports == | |||
=== General === | === General === | ||
* Add API to query backend for a list of available music engines. Useful for Options dialog. See [[Music drivers redesign]]. | * Add API to query backend for a list of available music engines. Useful for Options dialog. See [[Music drivers redesign]]. | ||
* Right now gBitFormat is part of common/scaler.cpp; we might want to move it to common/system.cpp, or replace it with something better. No hasty changes here, please, make sure you understand how it is used right now before messing with it ;-) | * Right now gBitFormat is part of common/scaler.cpp; we might want to move it to common/system.cpp, or replace it with something better. No hasty changes here, please, make sure you understand how it is used right now before messing with it ;-) | ||
== Tools == | == Tools == | ||
Line 206: | Line 190: | ||
=== General === | === General === | ||
* Use doxygen (or something else) to turn the short descs from the README into short man pages AND a single text file (replacing README) and a HTML file... | * Use doxygen (or something else) to turn the short descs from the README into short man pages AND a single text file (replacing README) and a HTML file... | ||
* | |||
=== GUI === | |||
* One of the GUI assistants pages shows the following text to the user: "Note: Some tools display file picker here, this should perhaps be changed to always be directory output, since often don't want to name the output file." Clearly this should *not* be shown to the user, but rather is something we devs need to address. | |||
* The last page of the assistant is weird. E.g. after successful run, it shows a "Finish" button in the lower right, and two checkboxes: "Open output folder" and "Process another file". This seems unintuitive and artificial to me. An IMHO better alternative would be to remove the "Finish" button, and instead show some big buttons with captions like "Process another file" (which does just that), "Open output folder", and "Quit". One can probably still do better, somebody should think about this. | |||
=== Compression tools === | === Compression tools === | ||
* (Semi-)automatically provide showhelp function | |||
* Modify encodeAudio to accept "input streams", so that we can avoid creating temporary WAVE files | * Modify encodeAudio to accept "input streams", so that we can avoid creating temporary WAVE files | ||
* Improved error checking !!! esp. for FT we got several reports about problems caused by the compression tool being called from the wrong directory. This should be avoidable by implementing suitable checks. | * Improved error checking !!! esp. for FT we got several reports about problems caused by the compression tool being called from the wrong directory. This should be avoidable by implementing suitable checks. | ||
=== Descumm === | === Descumm === |
edits