Difference between revisions of "SCI/TODO"
< 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 | ||
* 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