Opened 18 years ago
Last modified 14 years ago
#140 closed task
Large Scale C++ design — at Version 3
| Reported by: | Jan Wassenberg | Owned by: | Jan Wassenberg |
|---|---|---|---|
| Priority: | Nice to Have | Milestone: | |
| Component: | Core engine | Keywords: | |
| Cc: | Patch: |
Description (last modified by )
Cleanups inspired by book of same name (am currently reading it).
I think cleaning up our code to reduce inter-module dependencies is quite important. It promises to reduce compile time, simplify releases of parts ripped out of our codebase (e.g. a limited actor viewer), and overall make the code easier to modify (without having to wonder: "hm, what all does this break?").
First idea: sdl is dragged into 6 of 7 of our static library components - ugh!
This would be mostly avoidable by just forward-declaring SDL_Event. Unfortunately it is declared as a union (bad design!!), so that is impossible.
Simple solution: wrap it in a struct and forward-declare that.[done]
Use pImpl to hide implementation details of simulation classes. Specifically: boost dependency, which is dragged into network as well.
Change History (3)
comment:1 by , 18 years ago
comment:2 by , 18 years ago
| Description: | modified (diff) |
|---|
comment:3 by , 18 years ago
| Description: | modified (diff) |
|---|

(In [4252]) # sweng forward declare SDL_event. remove SDL dependency of network code. refs #140