integration of client and server code
Comparing the differences between some Client and Server files shows up inconsistence between them.
Such stuff includes either simple things like formatting issues.. Or more seriously, code differences.
This is an ongoing bug that will try to rectify some of these issues, as Denis pointed out, some inconsistencies/hacks are there to get things to work. So be aware.
A tool such as WinMerge can be used to view the differences, however a list will be posted soon.
Created attachment 80 [details]
A list of differences
Heres the list
depend on bug 21, which has been closed. this bug carries the torch.
*** Bug 627 has been marked as a duplicate of this bug. ***
Current list of files to be merged (16 cpp, 12 h):
Also, KDiff3 seems to work pretty nicely for this.
Most of the sound and rendering code in the server files is gone.
For sound and rendering, it is probably a good idea to move the remaining code into existing common files and delete the server files, rather than copying over the missing code to the server files and merging them with the client files, in order to avoid bloating the server. r_draw.cpp seems like one of the most obvious offenders to me.
However, another issue is that some common files include headers that exist on the client but aren't needed on the server. Without a major code refactoring, either stubs for server headers will have to be left, or an #ifdef will have to be used to conditionally include them depending on whether the client or server is being compiled.
Update: I moved the only variable in r_draw.cpp/h to r_data.cpp/h, which is a common file. r_draw.cpp should now be ready for deletion. However, r_draw.h is unfortunately #included by a common file, so I recommend just blanking it for now.
All headers have been merged and moved, will discuss in #odadev how to proceed from here (besides updating IDE project files).
At this point is there much left to do on this?
Per discussion with Sean, marking fixed. "I think we can close 404 and open tickets for more specific areas of code if we find there's still stuff that can be move to /common" -Dr. Sean