3
edits
Difference between revisions of "SCI/TODO"
Jump to navigation
Jump to search
no edit summary
(savegame task is done) |
MattHargett (talk | contribs) |
||
| 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 | * 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 -> | #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. | ||