Hi everyone,
I took the time a bit earlier to have a look at the code in the other git branches on sourceforge, to see what could be merged back, before I start proposing putting some source files in their own sub-folder, as discussed earlier with oln.
After a somewhat thorough review, I do think the branches listed below can be deleted for the reasons given each time:
(This will permit to have a cleaner playground for the team, IMHO.)
- Rooms/
The branch has been merged in development/ and the code is now obsolete due to recent addition to the room code.
- windows/
The branch has also been merged back to development/.
- tiles/
The branch is representing some early work on the tile culling algorithm and the development branch is much better than this one, thanks to Paul.
- tile2/
This is somewhat the most twisted branch as it involves work on the culling algorithm, the splitting of the console command out of the ODFrameListener, and work on the minimap, including the libmagick+ library.
Fortunately, the work on the culling algorithm is in better shape on the development branch, the split of the console command has been merged back on development/ and the minimap code obsolete thanks to MCMic's latest patches for the minimap. I must also say I'm not fond of adding the libmagick+ library to the dependency list of OD, anyway.
- broken/
A more recent branch which include work on XML files (not code) mostly, AngelScript, the camera manager, ...
To me, all the work here has been merged back to development/ except for the few patches that broke the game, namely: The renumeration of tile types, which have since been reverted.
- better_exceptions/
This branch is mostly about adding better exceptions (as you may have guessed ;]) which has been merged back to development/ and about the small event system snippet which is also in development in devel_tools/
This branch seems to have code not in the development branch:
- xml/
There is some work here toward the integration and use of tinyXML2. The work done is very slight, though. The changes are very few and I don't know whether we can consider this obsolete or not.
Paul, Oln, could you guys tell me whether I'm wrong somewhere here? Once we agree about certain branches being useless or duplicate, I'll delete them to avoid cluttering the repo with useless and confusing stuff.
Regards,