Window loops now run on the render thread

CamelotClient/CamelotClient.cpp

@@ -27,8 +27,8 @@ using namespace CamelotEngine;
 
 int _tmain(int argc, _TCHAR* argv[])
 {
-	//gApplication().startUp("CamelotGLRenderSystem", "CamelotForwardRenderer");
-	gApplication().startUp("CamelotD3D9RenderSystem", "CamelotForwardRenderer");
+	gApplication().startUp("CamelotGLRenderSystem", "CamelotForwardRenderer");
+	//gApplication().startUp("CamelotD3D9RenderSystem", "CamelotForwardRenderer");
 
 	RenderSystem* renderSystem = RenderSystemManager::getActive();
 	RenderWindow* renderWindow = gApplication().getPrimaryRenderWindow();

CamelotD3D9Renderer/Source/CmD3D9RenderSystemFactory.cpp

@@ -7,7 +7,7 @@ namespace CamelotEngine
 	#ifdef CM_STATIC_LIB
 		HINSTANCE hInst = GetModuleHandle( NULL );
 	#else
-		HINSTANCE hInst = GetModuleHandle( "CamelotEngine.dll" ); // TODO - Change name if I plan on using external dll
+		HINSTANCE hInst = GetModuleHandle( "CamelotD3D9RenderSystem.dll" );
 	#endif
 
 		return RenderSystemPtr(new D3D9RenderSystem(hInst));

CamelotGLRenderer/Source/CmWin32Window.cpp

@@ -74,11 +74,7 @@ namespace CamelotEngine {
 #ifdef CM_STATIC_LIB
 		HINSTANCE hInst = GetModuleHandle( NULL );
 #else
-#  if CM_DEBUG_MODE == 1
-		HINSTANCE hInst = GetModuleHandle("RenderSystem_GL_d.dll");
-#  else
-		HINSTANCE hInst = GetModuleHandle("RenderSystem_GL.dll");
-#  endif
+		HINSTANCE hInst = GetModuleHandle("CamelotGLRenderSystem.dll");
 #endif
 
 		mHWnd = 0;
@@ -342,7 +338,7 @@ namespace CamelotEngine {
 			}
 
 			// Pass pointer to self as WM_CREATE parameter
-			mHWnd = CreateWindowEx(dwStyleEx, "OgreGLWindow", title.c_str(),
+			mHWnd = CreateWindowEx(dwStyleEx, "GLWindow", title.c_str(),
 				dwStyle, mLeft, mTop, mWidth, mHeight, parent, 0, hInst, this);
 
 			WindowEventUtilities::_addRenderWindow(this);			

CamelotRenderer/Include/CmApplication.h

@@ -40,6 +40,11 @@ namespace CamelotEngine
 		 * @brief	Callback called from the render thread in order to initialize resources.
 		 */
 		void updateResourcesCallback();
+
+		/**
+		 * @brief	Runs the OS specific message pump.
+		 */
+		void updateMessagePump();
 	};
 
 	CM_EXPORT Application& gApplication();

CamelotRenderer/Source/CmApplication.cpp

@@ -62,6 +62,7 @@ namespace CamelotEngine
 
 		loadPlugin("CamelotOISInput"); // TODO - Load this automatically somehow
 
+		renderSystem->addPreRenderThreadUpdateCallback(boost::bind(&Application::updateMessagePump, this));
 		renderSystem->addPreRenderThreadUpdateCallback(boost::bind(&Application::updateResourcesCallback, this));
 	}
 
@@ -69,8 +70,6 @@ namespace CamelotEngine
 	{
 		while(true)
 		{
-			WindowEventUtilities::messagePump();
-
 			gSceneManager().update();
 
 			RendererManager::getActive()->renderAll();
@@ -83,6 +82,11 @@ namespace CamelotEngine
 		}
 	}
 
+	void Application::updateMessagePump()
+	{
+		WindowEventUtilities::messagePump();
+	}
+
 	void Application::updateResourcesCallback()
 	{
 		gResources().update();

CamelotRenderer/TODO.txt

@@ -16,6 +16,7 @@ High-level TODO:
 
 
 Command buffer TODO:
+ - Make sure window gets created on the main, and not the render thread
  - My current approach doesn't allow multiple threads to use the RenderSystem (contexts should be handled differently)
    - Instead of requiring the user to constantly call setActiveContext, make the call peristent per thread. 
    - Store sequential thread idx in local thread storage, and then we can easily look up a context for a thread and