On Saturday 06 August 2005 12:19 pm, Bob Jamison wrote:
David Yip wrote:
Out of curiosity, how is it breaking the build? I have tried building Inkscape CVS with and without --enable-inkboard, and the build process and resulting binary behave correctly in both situations.
Well, I guess you saw the compile problems. These are all minor, but I would still like to wait until after the release.
Hmm, alright -- I have no problems with that. Are you referring to the compiler warnings and such that occur?
By the way, what is needed, I believe, is to call lm_initialize() once before any other calls to Loudmouth code. This is a no-op on Linux, but on Windows it calls WSAStartup(), which is necessary for any code that wants to use Winsock.
Ah, okay -- I didn't see that in the Loudmouth documentation or API.
The proper place to insert lm_initialize() would probably be in src/jabber_whiteboard/connection-establishment.cpp in SessionManager::connectToServer, at approximately line 203 (before the call to lm_connection_new()). I've made the change here, but I don't want to commit the code I have just yet -- it's currently undergoing a rewrite, and is not yet reasonably stable.
-- David