Difference between revisions of "SCI/TODO"

SCI/TODO (view source)

243 bytes removed , 12:57, 27 February 2009

Code formatting is done

1,510

edits

@@ Line 1: / Line 1: @@
 * 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.
-* more formatting fixes: The SCI code puts return values on separate lines all over the place, e.g.
-<syntax type="C++">
-  int
-  _vocab_cmp_words(const void *word1, const void *word2) {
-</syntax>
-Take care of that and other formatting issues.
 * Change #define lists into enums
 * Rearrange the files under engines/sci: I.e. get rid of include/ subdir (just move its contents to engines/sci, at some point at least). Some source files should also be renamed for clarity.