Open main menu

Difference between revisions of "SCI/TODO"

150 bytes added ,  21:55, 23 February 2009
no edit summary
(savegame task is done)
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:
* Replace all uses of gint8, etc. by our native types, to get rid of the typedefs we use right now.
* Replace all uses of gint8, etc. by our native types, to get rid of the typedefs we use right now.
* check for places that use char or unsigned char for boolean values, and change those to use type bool instead
* check for places that use char or unsigned char for boolean values, and change those to use enum types whose value names express the intention (SearchOptions.RECURSIVE, GraphicsScaleOptions.2X, etc)
* more formatting fixes: The SCI code puts return values on separate lines all over the place, e.g.
* more formatting fixes: The SCI code puts return values on separate lines all over the place, e.g.


Line 42: Line 42:


* Get rid of sci_sched_yield: This is used for two things:
* Get rid of sci_sched_yield: This is used for two things:
#gfx/resource/sci_pic_0.cpp -> analyze why it is used here, find a way to replace it
#gfx/resource/sci_pic_0.cpp -> used here to make sure pic rendering doesn't starve the sound-playing thread(s), resulting in music/sound skipping
#./sfx/mixer/soft.cpp to implement ACQUIRE_LOCK -> replace ACQUIRE_LOCK and RELEASE_LOCK by using our mutex API. (And on the long run, replace the SCI mixer by our mixer :-)
#./sfx/mixer/soft.cpp to implement ACQUIRE_LOCK -> replace ACQUIRE_LOCK and RELEASE_LOCK by using our mutex API. (And on the long run, replace the SCI mixer by our mixer :-)
* In fact: Do use File, SearchManager, FSNode etc. instead of paths, fopen, etc.
* In fact: Do use File, SearchManager, FSNode etc. instead of paths, fopen, etc.