[Supertux-Commit] r5978 - branches/supertux-milestone2-grumbel
grumbel at cummiskey.dreamhost.com
grumbel at cummiskey.dreamhost.com
Sun Nov 15 12:33:01 PST 2009
Author: grumbel
Date: 2009-11-15 12:33:01 -0800 (Sun, 15 Nov 2009)
New Revision: 5978
Modified:
branches/supertux-milestone2-grumbel/TODO
Log:
TODO updates
Modified: branches/supertux-milestone2-grumbel/TODO
===================================================================
--- branches/supertux-milestone2-grumbel/TODO 2009-11-15 20:28:01 UTC (rev 5977)
+++ branches/supertux-milestone2-grumbel/TODO 2009-11-15 20:33:01 UTC (rev 5978)
@@ -19,7 +19,7 @@
* move bugtracker to http://code.google.com (much simpler, less useless)
-* normalize #include directives (all rever to top level dir)
+* normalize #include directives (all refer to top level dir)
* use SCons instead of CMake
@@ -39,8 +39,34 @@
"-Winit-self", # only works with >= -O1
"-Wno-unused-parameter",
-* do not use pointer
+* do not use raw pointer, especially not for Sprite and Surface
+* properly separate data members and member functions, don't mix them
+ in the same section
+
+* write namespaces like: "namespace NameSpace {", no newline before the {
+
+* only do one variable initialization per line, not multiple as its
+ currently often done in initialization list
+
+* conditional includes should be indended (makes it easier to handle in include optimization):
+
+#ifdef FOOBAR
+# include "foobar.hpp"
+#endif
+
+* GameObject::RemoveListenerListEntry: Ughs, somebody trying to
+ implement a list class within in the GameObject?!
+
+* remove overuse of multi-inheritance
+
+* remove overuse of friend'ship
+
+* mark interfaces as interfaces (ISerializable or SerializableInterface)
+
+* replace random generator with mersene twister and/or move to external/
+
+
Comments
========
More information about the Supertux-Commit
mailing list