Note: Controversial list of things currently broken and controversial solutions for them. Coding Standard =============== * use SCons instead of CMake * make code clean: "-O2", "-g3", "-ansi", "-pedantic", "-Wall", "-Wextra", "-Wnon-virtual-dtor", "-Weffc++", "-Wconversion", "-Werror", "-Wshadow", "-Wcast-qual", "-Winit-self", # only works with >= -O1 "-Wno-unused-parameter", * do not use raw pointer, especially not for Sprite and Surface * only do one variable initialization per line, not multiple as its currently often done in initialization list * remove overuse of multi-inheritance * remove overuse of friend'ship * maybe mark interfaces as interfaces (ISerializable or SerializableInterface) * split files with multiple classes into multiple files with one class each TODO ==== * GameObject::RemoveListenerListEntry: Ughs, somebody trying to implement a list class within in the GameObject?! * replace random generator with mersene twister and/or move to external/ * check the code with Valgrind * static vs anonymous namespace * use Vector in Physics for 'a' and 'v' * add --datadir DIR (data/) and --userdir DIR (~/.supertux/) * make gravity a constant * funky side effect of too much global variables: when having a savegame with large or firetux and then starting that game, Tux in the menu background will grow and be visible that way for a fraction of a second * write scripts for include sorting and include guard checking that can be run automatically * md5.hpp and random_generator.hpp could go to external/ * rename Vector -> Vector2f * get rid of global SDL_Screen* screen variable * identify all global variables and make them ugly (g_ or globals::) * get rid of SCREEN_WIDTH/SCREEN_HEIGHT * is version.h actually needed? * resolution menu entry moves the wrong way around * write scripts to automatically check for: - all includes are relative to top level dir - include guards are proper * move SVN to http://code.google.com (maybe one day) * move bugtracker to http://code.google.com (much simpler, less useless) * convert worldmap from const lisp::Lisp* lisp to "const Reader&" * having dictionary_manager in Lisp is extremely ugly * enforce proper naming of files to match their class * get rid of DEBUG and conditional compilation, these should be reserved for a few tiny cases, not spread all over the code * split particlesystem_interactive * centralize menus * make a proper class out of supertux/resources.hpp # EOF #