SCI/TODO

From ScummVM :: Wiki
< SCI
Revision as of 08:45, 15 February 2009 by Jvprat (talk | contribs) (New page: =SCI TODO list= The following list is taken from Max's mails to -devel: * Fixing all warnings * Rename source files from *.c to *.cpp, and get rid of the %.c build rule in engines/sci/modu...)
(diff) ← Older revision | Latest revision (diff) | Newer revision → (diff)
Jump to navigation Jump to search
The printable version is no longer supported and may have rendering errors. Please update your browser bookmarks and please use the default browser print function instead.

SCI TODO list

The following list is taken from Max's mails to -devel:

  • Fixing all warnings
  • Rename source files from *.c to *.cpp, and get rid of the %.c build rule in engines/sci/module.mk
  • Find usage of SCUMMVM #define and resolve them, then in the end stop adding -DSCUMMVM to CPPFLAGS in module.mk
  • Rearranging the files under engines/sci: I.e. get rid of include/ subdir (just move its contents to engines/sci, at some point at least)
  • Converting stuff from "C pseudo classes" to real classes
  • Replacing various generic FreeSCI stuff by their ScummVM counterparts. E.g. don't bother to rewrite engines/sci/include/int_hashmap.h -- instead change code to use a HashMap<int, ...> etc.
  • Rename engines/sci/scummvm/scummvm_engine.* to engines/sci/sci.*
  • Move engines/sci/scummvm/detection.cpp to engines/sci/detection.cpp
  • In fact: Do use File, SearchManager, FSNode etc. instead of paths, fopen, etc.
  • Don't use fstat, PATH_MAX
  • const correctness in engines/sci/engine/savegame.c (this is a generated file, so change the source or generator, whatever is necessary)
  • apply our code formatting guidelines
  • change the many nice existing function documentation comments to use doxygen syntax