Added a few FIXMEs.
authorRyan C. Gordon <icculus@icculus.org>
Tue, 21 Apr 2015 09:46:48 -0400
changeset 10010 cc07869515b2
parent 10009 8c6b05b0a27e
child 10011 08f381fd2279
Added a few FIXMEs.
include/SDL_video.h
src/video/cocoa/SDL_cocoawindow.m
src/video/x11/SDL_x11window.c
--- a/include/SDL_video.h	Mon Jan 04 22:58:38 2016 -0500
+++ b/include/SDL_video.h	Tue Apr 21 09:46:48 2015 -0400
@@ -95,6 +95,7 @@
  */
 typedef enum
 {
+    /* !!! FIXME: change this to name = (1<<x). */
     SDL_WINDOW_FULLSCREEN = 0x00000001,         /**< fullscreen window */
     SDL_WINDOW_OPENGL = 0x00000002,             /**< window usable with OpenGL context */
     SDL_WINDOW_SHOWN = 0x00000004,              /**< window is visible */
--- a/src/video/cocoa/SDL_cocoawindow.m	Mon Jan 04 22:58:38 2016 -0500
+++ b/src/video/cocoa/SDL_cocoawindow.m	Tue Apr 21 09:46:48 2015 -0400
@@ -196,6 +196,7 @@
     }
 }
 
+/* !!! FIXME: this should use a hint callback. */
 static int
 GetHintCtrlClickEmulateRightClick()
 {
--- a/src/video/x11/SDL_x11window.c	Mon Jan 04 22:58:38 2016 -0500
+++ b/src/video/x11/SDL_x11window.c	Tue Apr 21 09:46:48 2015 -0400
@@ -130,6 +130,7 @@
 {
     SDL_VideoData *videodata = (SDL_VideoData *) _this->driverdata;
     Display *display = videodata->display;
+    /* !!! FIXME: just dereference videodata below instead of copying to locals. */
     Atom _NET_WM_STATE = videodata->_NET_WM_STATE;
     /* Atom _NET_WM_STATE_HIDDEN = videodata->_NET_WM_STATE_HIDDEN; */
     Atom _NET_WM_STATE_FOCUSED = videodata->_NET_WM_STATE_FOCUSED;