diff --git a/src/cocoa_init.m b/src/cocoa_init.m index b17d572e..2491e7eb 100644 --- a/src/cocoa_init.m +++ b/src/cocoa_init.m @@ -111,8 +111,6 @@ int _glfwPlatformInit(void) void _glfwPlatformTerminate(void) { - // TODO: Probably other cleanup - if (_glfw.ns.eventSource) { CFRelease(_glfw.ns.eventSource); diff --git a/src/cocoa_joystick.m b/src/cocoa_joystick.m index 8540e6db..c218cc0d 100644 --- a/src/cocoa_joystick.m +++ b/src/cocoa_joystick.m @@ -444,10 +444,7 @@ void _glfwTerminateJoysticks(void) int _glfwPlatformGetJoystickParam(int joy, int param) { if (!_glfw.ns.joysticks[joy].present) - { - // TODO: Figure out if this is an error return GL_FALSE; - } switch (param) { @@ -478,10 +475,7 @@ int _glfwPlatformGetJoystickAxes(int joy, float* axes, int numaxes) _GLFWjoy joystick = _glfw.ns.joysticks[joy]; if (!joystick.present) - { - // TODO: Figure out if this is an error return 0; - } numaxes = numaxes < joystick.numAxes ? numaxes : joystick.numAxes; @@ -518,10 +512,7 @@ int _glfwPlatformGetJoystickButtons(int joy, unsigned char* buttons, _GLFWjoy joystick = _glfw.ns.joysticks[joy]; if (!joystick.present) - { - // TODO: Figure out if this is an error return 0; - } // Update joystick state pollJoystickEvents(); diff --git a/src/cocoa_window.m b/src/cocoa_window.m index ebaf9396..6e781723 100644 --- a/src/cocoa_window.m +++ b/src/cocoa_window.m @@ -794,8 +794,6 @@ void _glfwPlatformDestroyWindow(_GLFWwindow* window) [window->ns.object close]; window->ns.object = nil; - - // TODO: Probably more cleanup } void _glfwPlatformSetWindowTitle(_GLFWwindow* window, const char *title) diff --git a/src/egl_context.c b/src/egl_context.c index 6e362cea..b1abe6bc 100644 --- a/src/egl_context.c +++ b/src/egl_context.c @@ -345,8 +345,6 @@ int _glfwCreateContext(_GLFWwindow* window, if (window->egl.context == EGL_NO_CONTEXT) { - // TODO: Handle all the various error codes here - _glfwInputError(GLFW_PLATFORM_ERROR, "EGL: Failed to create context: %s", getErrorString(eglGetError())); diff --git a/src/win32_window.c b/src/win32_window.c index d0fca337..504d3348 100644 --- a/src/win32_window.c +++ b/src/win32_window.c @@ -671,11 +671,8 @@ static void getFullWindowSize(_GLFWwindow* window, int* fullWidth, int* fullHeight) { RECT rect = { 0, 0, clientWidth, clientHeight }; - - // Adjust according to window styles - AdjustWindowRectEx(&rect, window->win32.dwStyle, FALSE, window->win32.dwExStyle); - - // Calculate width and height of full window + AdjustWindowRectEx(&rect, window->win32.dwStyle, + FALSE, window->win32.dwExStyle); *fullWidth = rect.right - rect.left; *fullHeight = rect.bottom - rect.top; } @@ -688,21 +685,21 @@ static ATOM registerWindowClass(void) ATOM classAtom; // Set window class parameters - wc.style = CS_HREDRAW | CS_VREDRAW | CS_OWNDC; // Redraw on... - wc.lpfnWndProc = (WNDPROC) windowProc; // Message handler - wc.cbClsExtra = 0; // No extra class data - wc.cbWndExtra = sizeof(void*) + sizeof(int); // Make room for one pointer - wc.hInstance = GetModuleHandle(NULL); // Set instance - wc.hCursor = LoadCursor(NULL, IDC_ARROW); // Load arrow pointer - wc.hbrBackground = NULL; // No background - wc.lpszMenuName = NULL; // No menu - wc.lpszClassName = _GLFW_WNDCLASSNAME; // Set class name + wc.style = CS_HREDRAW | CS_VREDRAW | CS_OWNDC; + wc.lpfnWndProc = (WNDPROC) windowProc; + wc.cbClsExtra = 0; // No extra class data + wc.cbWndExtra = sizeof(void*) + sizeof(int); // Make room for one pointer + wc.hInstance = GetModuleHandle(NULL); + wc.hCursor = LoadCursor(NULL, IDC_ARROW); + wc.hbrBackground = NULL; // No background + wc.lpszMenuName = NULL; // No menu + wc.lpszClassName = _GLFW_WNDCLASSNAME; // Load user-provided icon if available wc.hIcon = LoadIcon(GetModuleHandle(NULL), L"GLFW_ICON"); if (!wc.hIcon) { - // User-provided icon not found; load default icon + // No user-provided icon found, load default icon wc.hIcon = LoadIcon(NULL, IDI_WINLOGO); } @@ -735,7 +732,6 @@ static int createWindow(_GLFWwindow* window, window->win32.dwStyle |= WS_POPUP; _glfwPlatformGetMonitorPos(wndconfig->monitor, &xpos, &ypos); - fullWidth = wndconfig->width; fullHeight = wndconfig->height; } @@ -782,7 +778,7 @@ static int createWindow(_GLFWwindow* window, NULL, // No parent window NULL, // No window menu GetModuleHandle(NULL), - window); // Pass GLFW window to WM_CREATE + window); // Pass object to WM_CREATE free(wideTitle);