[Freeciv-Dev] is_server kludges

["Brandon J. Van Every" <vanevery@xxxxxxxxxxxxxxxxxxx>]

I'm working from Freeciv 1.14.1 stable sources.  I'm trying to compile
the code in freeciv\common as a DLL.  I find there is some kludge:

is_server - a global variable in both \client and \server
dealloc_id - has different definitions in \common, \client, and \server
send_unit_info - has different definitions in \client and \server

I know you guys weren't trying to have \common be a DLL, but it's rather
unclean to have a library implement modality using globals in .exe's
that aren't part of the library.  Do you guys have an opinion on
cleaning this up?  If so, I could submit a patch based on your input.
If not, I'll get on with doing it properly.


Cheers,                     www.indiegamedesign.com
Brandon Van Every           Seattle, WA

Taking risk where others will not.