Open main menu

SCI/TODO

< SCI
Revision as of 00:06, 28 September 2009 by Wjp (talk | contribs) (add SCI FileSelector crash bug)
  • Easy
    • rename the sfx/ dir to sound/ or audio/ (it doesn't just deal with "sound effects).
    • change the many nice existing function documentation comments to use doxygen syntax
    • Change #define lists into enums
  • Medium
    • Convert stuff from "C pseudo classes" to real classes.
    • check for places that use char or unsigned char for boolean values, and change those to bool. Even nicer would be to change it to use enum types whose value names express the intention (SearchOptions.RECURSIVE, GraphicsScaleOptions.2X, etc), but just going for bool will already be a big improvement.
    • Replace various generic FreeSCI code by their ScummVM counterparts, where it make sense.
    • Check old FreeSCI-scummvm branch / repos by Jordi for change that could be salvaged, e.g. the files sfx/device/scumm-midi.cpp and sfx/softseq/scumm-adlib.cpp, and stuff.
  • Difficult
    • Turn code into templates in following files (now they generate code via #defines):
      • gfx/picfill.cpp
    • make the SCI tools compilable (to avoid bitrot, while we change code that they depend on), and finally usable again.
    • rewrite the YACC parser code in said.y in pure, readable C++, so we don't need YACC/Bison.
    • remove the "marked for deletion"/"unmark" concepts from the garbage collector as they are fundamentelly broken: unmarking an object doesn't restore its dependencies.
    • use direct drawing instead of using widgets for rendering to be compatible with SSCI.


  • Bugs
    • The FileSelector (SCI) class used in the import screens in QfG2 and QfG3 has a 'text' pointer, and 'cursor' and 'lsTop'/'topString' pointers that should point to inside 'text'. However, in FileSelector::readFiles, 'text' is freed and reallocated, causing the other pointers to sometimes become invalid.