Today, i added a patch to CL_SetupDisplay::CL_SetupDisplay()

It called XInitThreads() when "#ifndef WIN32"

Now, I feel this is wrong for 2 reasons.

1) The ClanDisplay may not need X - The #ifndef WIN32 is wrong
2) It should really be in Display/X11/x11_window.cpp, called once before any other XLib function. (Not in CL_SetupDisplay)

However, do we need this function?

Looking at http://tronche.com/gui/x/xlib/display/XInitThreads.html

"The XInitThreads() function initializes Xlib support for concurrent threads"
Should ClanLib call the function at all, or should it be up to the client application?