Difference between revisions of "SCI/TODO"

From ScummVM :: Wiki
< SCI
Jump to navigation Jump to search
(Some TODOs are done)
(One todo is gone, one came)
Line 1: Line 1:
 
The following list is taken from Max's mails to -devel:
 
The following list is taken from Max's mails to -devel:
 
* Fixing all warnings
 
* 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
 
* 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)
 
* 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)
Line 11: Line 10:
 
* apply our code formatting guidelines
 
* apply our code formatting guidelines
 
* change the many nice existing function documentation comments to use doxygen syntax
 
* change the many nice existing function documentation comments to use doxygen syntax
 +
* Turn code into templates in following files (now they generate code via #defines):
 +
** gfx/gfx_pixmap_scale.cpp
 +
** gfx/gfx_line.cpp
 +
** gfx/gfx_crossblit.cpp
 +
** gfx/resource/sci_picfill.cpp
 +
** gfx/resource/sci_picfill_aux.cpp

Revision as of 11:49, 15 February 2009

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

  • Fixing all warnings
  • 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.
  • In fact: Do use File, SearchManager, FSNode etc. instead of paths, fopen, etc.
  • Don't use fstat
  • 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
  • Turn code into templates in following files (now they generate code via #defines):
    • gfx/gfx_pixmap_scale.cpp
    • gfx/gfx_line.cpp
    • gfx/gfx_crossblit.cpp
    • gfx/resource/sci_picfill.cpp
    • gfx/resource/sci_picfill_aux.cpp