Skip to content

Commit

Permalink
Remove an obsolete TODO; use stronger wording to tell people they sho…
Browse files Browse the repository at this point in the history
…uld *ask* us before implementing something (native speakers, feel free to correct my english :-)

svn-id: r18686
  • Loading branch information
fingolfin committed Aug 14, 2005
1 parent 72ed016 commit 251e158
Showing 1 changed file with 12 additions and 7 deletions.
19 changes: 12 additions & 7 deletions TODO
Original file line number Diff line number Diff line change
Expand Up @@ -7,10 +7,18 @@ useful contribution. Note that this list is never complete, and may be
partially outdated, so just because you don't see something here doesn't
mean it is not important.

Before you start work on something, you might want to talk to somebody
from the team. This will help us to prevent double work, i.e. several
people working on the same stuff at once without knowing about each
other.
Before you start work on something, you should first talk to the team!
Ideally ask on scummvm-devel, our mailing list. This will help us to
prevent double work, i.e. several people working on the same stuff at
once without knowing about each other. Furthermore, sometimes entries
on our list are actually obsolete (because the feature has been
implemented, or because for some reason we no longer think it to be
desirable). Special caution should be taken for TODO entries that say
"we may want to" or similar things; that usually means that we aren't
sure whether we really want to implement that feature.

So, to repeat it: Always talk to the team before implementing a change
on this list, or else risk having your patch rejected :-/.

Finally, always make sure to check out our bug tracker and our feature
request tracker for things that need work.
Expand Down Expand Up @@ -83,9 +91,6 @@ General
we set when the application should be quit (e.g. when an EVENT_QUIT is
received). This is useful if multiple levels of event loops have to be ended
* Fix the Map<> template, make it more robust; maybe use a red-black tree?
* Instead of the above, consider switching everything over to the STL.
Maybe a little bit more overhead, but we get instant access to a full-featured
and well-tested code base, with everything we need.
* Make some generic "EventLoop" API/class which all backends and the GUI
use. Initially this would just call the backend poll_event() etc. methods.
But eventually the EventLoop object(s) could be made by the backend.
Expand Down

0 comments on commit 251e158

Please sign in to comment.