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

"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?