]> git.saurik.com Git - wxWidgets.git/commitdiff
Added CS_OWNDC fix to wxGLCanvas and corrected sample makefiles
authorJulian Smart <julian@anthemion.co.uk>
Thu, 28 Oct 1999 11:35:45 +0000 (11:35 +0000)
committerJulian Smart <julian@anthemion.co.uk>
Thu, 28 Oct 1999 11:35:45 +0000 (11:35 +0000)
git-svn-id: https://svn.wxwidgets.org/svn/wx/wxWidgets/trunk@4240 c3d73ce0-8a6f-49c7-b76d-6d57e0e08775

src/msw/cursor.cpp
utils/glcanvas/samples/cube/makefile.vc
utils/glcanvas/samples/isosurf/makefile.vc
utils/glcanvas/samples/penguin/makefile.vc
utils/glcanvas/win/glcanvas.cpp
utils/glcanvas/win/glcanvas.h
utils/glcanvas/win/makefile.vc

index 56c4a8b9a3e47f1c70b8c485d8298f518c392295..b483ad01451534e23e735e94aeec3c3eff653996 100644 (file)
@@ -78,7 +78,7 @@ wxCursor::wxCursor(const wxString& cursor_file, long flags, int hotSpotX, int ho
   M_CURSORDATA->m_destroyCursor = FALSE;
   M_CURSORDATA->m_hCursor = 0;
   M_CURSORDATA->m_ok = FALSE;
-  if (flags & wxBITMAP_TYPE_CUR_RESOURCE)
+  if (flags == wxBITMAP_TYPE_CUR_RESOURCE)
   {
 #ifdef __WIN95__
     M_CURSORDATA->m_hCursor = (WXHCURSOR) LoadImage(wxGetInstance(), cursor_file, IMAGE_CURSOR, 0, 0, 0);
@@ -90,7 +90,7 @@ wxCursor::wxCursor(const wxString& cursor_file, long flags, int hotSpotX, int ho
     else
       M_CURSORDATA->m_ok = FALSE;
   }
-  else if (flags & wxBITMAP_TYPE_CUR)
+  else if (flags == wxBITMAP_TYPE_CUR)
   {
 #ifdef __WIN95__
     M_CURSORDATA->m_hCursor = (WXHCURSOR) LoadImage(wxGetInstance(), cursor_file, IMAGE_CURSOR, 0, 0, LR_LOADFROMFILE);
@@ -101,14 +101,14 @@ wxCursor::wxCursor(const wxString& cursor_file, long flags, int hotSpotX, int ho
 #endif
 #endif
   }
-  else if (flags & wxBITMAP_TYPE_ICO)
+  else if (flags == wxBITMAP_TYPE_ICO)
   {
 #if wxUSE_RESOURCE_LOADING_IN_MSW
     M_CURSORDATA->m_hCursor = (WXHCURSOR) IconToCursor(WXSTRINGCAST cursor_file, wxGetInstance(), hotSpotX, hotSpotY, &M_CURSORDATA->m_width, &M_CURSORDATA->m_height);
     M_CURSORDATA->m_destroyCursor = TRUE;
 #endif
   }
-  else if (flags & wxBITMAP_TYPE_BMP)
+  else if (flags == wxBITMAP_TYPE_BMP)
   {
 #if wxUSE_RESOURCE_LOADING_IN_MSW
     HBITMAP hBitmap = 0;
index 997616ea305d5c15eeb83f3ed03b6a35008009ec..f944029f9e5219bbc601d892c7d8499aba7e098c 100644 (file)
 # Set WXDIR for your system
 WXDIR = $(WXWIN)
 
+!if "$(FINAL)" == "1"
+!else
+LIBEXT=_d
+!endif
+
 PROGRAM=cube
 OBJECTS = $(PROGRAM).obj
 EXTRAINC=-I..\..\win
-EXTRALIBS=$(WXDIR)\lib\glcanvas.lib glu32.lib opengl32.lib
+EXTRALIBS=$(WXDIR)\lib\glcanvas$(LIBEXT).lib glu32.lib opengl32.lib
 
 !include $(WXDIR)\src\makeprog.vc
 
index be397eddd65c0a20f7244f3d1f27ce4113f8f24f..7da10310ff8eeba899db0bc073515d8cd9744005 100644 (file)
 # Set WXDIR for your system
 WXDIR = $(WXWIN)
 
+!if "$(FINAL)" == "1"
+!else
+LIBEXT=_d
+!endif
+
 PROGRAM=isosurf
 OBJECTS = $(PROGRAM).obj
 EXTRAINC=-I..\..\win
-EXTRALIBS=$(WXDIR)\lib\glcanvas.lib glu32.lib opengl32.lib
+EXTRALIBS=$(WXDIR)\lib\glcanvas$(LIBEXT).lib glu32.lib opengl32.lib
 EXTRATARGETS=isosurf.dat
 
 !include $(WXDIR)\src\makeprog.vc
index 27e4721eaa72806709f7d3362bc3ff73199d1c30..32e14d829c6625bfddc1b8038f9cd9839dbea30d 100644 (file)
 # Set WXDIR for your system
 WXDIR = $(WXWIN)
 
+!if "$(FINAL)" == "1"
+!else
+LIBEXT=_d
+!endif
+
 PROGRAM=penguin
 OBJECTS = $(PROGRAM).obj trackball.obj lw.obj
 EXTRAINC=-I..\..\win
-EXTRALIBS=$(WXDIR)\lib\glcanvas.lib glu32.lib opengl32.lib
+EXTRALIBS=$(WXDIR)\lib\glcanvas$(LIBEXT).lib glu32.lib opengl32.lib
 
 !include $(WXDIR)\src\makeprog.vc
 
index e90b555a01fa0d8dd5ace354462b762b3ba41f92..fd1e1ef44baeb9b2e15c3e106d5f98fee5769b9d 100644 (file)
 #include <wx/frame.h>
 #endif
 
+#include <wx/msw/private.h>
+
 #include "glcanvas.h"
 
+wxChar wxGLCanvasClassName[]        = wxT("wxGLCanvasClass");
+
+LRESULT APIENTRY _EXPORT wxWndProc(HWND hWnd, UINT message,
+                                   WPARAM wParam, LPARAM lParam);
+
 /*
  * GLContext implementation
  */
@@ -131,18 +138,28 @@ wxGLCanvas::wxGLCanvas(wxWindow *parent, wxWindowID id,
 
   m_glContext = new wxGLContext(TRUE, this, palette);
 }
+
 wxGLCanvas::wxGLCanvas( wxWindow *parent, 
               const wxGLContext *shared, wxWindowID id,
               const wxPoint& pos, const wxSize& size, long style, const wxString& name,
-              int *attribList, const wxPalette& palette ) :
-  wxScrolledWindow(parent, id, pos, size, style, name)
+              int *attribList, const wxPalette& palette )
+  : wxScrolledWindow()
+//  : wxScrolledWindow(parent, id, pos, size, style, name)
 {                      
-  m_hDC = (WXHDC) ::GetDC((HWND) GetHWND());
+    bool ret = Create(parent, id, pos, size, style, name);
 
-  SetupPixelFormat();
-  SetupPalette(palette);
+    if ( ret )
+    {
+        SetBackgroundColour(wxSystemSettings::GetSystemColour(wxSYS_COLOUR_3DFACE));
+        SetFont(wxSystemSettings::GetSystemFont(wxSYS_DEFAULT_GUI_FONT));
+    }
 
-  m_glContext = new wxGLContext(TRUE, this, palette, shared );
+    m_hDC = (WXHDC) ::GetDC((HWND) GetHWND());
+
+    SetupPixelFormat();
+    SetupPalette(palette);
+
+    m_glContext = new wxGLContext(TRUE, this, palette, shared );
 }
 
 wxGLCanvas::~wxGLCanvas()
@@ -153,6 +170,103 @@ wxGLCanvas::~wxGLCanvas()
   ::ReleaseDC((HWND) GetHWND(), (HDC) m_hDC);
 }
 
+// Replaces wxWindow::Create functionality, since we need to use a different window class
+bool wxGLCanvas::Create(wxWindow *parent, wxWindowID id,
+              const wxPoint& pos, const wxSize& size, long style, const wxString& name)
+{
+    static bool registeredGLCanvasClass = FALSE;
+
+    // We have to register a special window class because we need
+    // the CS_OWNDC style for GLCanvas.
+
+/*
+    Here are two snips from a dicussion in the OpenGL Gamedev list that explains
+    how this problem can be fixed:
+
+    "There are 5 common DCs available in Win95. These are aquired when you call
+    GetDC or GetDCEx from a window that does _not_ have the OWNDC flag.
+    OWNDC flagged windows do not get their DC from the common DC pool, the issue
+    is they require 800 bytes each from the limited 64Kb local heap for GDI."
+
+    "The deal is, if you hold onto one of the 5 shared DC's too long (as GL apps
+    do), Win95 will actually "steal" it from you.  MakeCurrent fails,
+    apparently, because Windows re-assigns the HDC to a different window.  The
+    only way to prevent this, the only reliable means, is to set CS_OWNDC."
+*/
+
+    if (!registeredGLCanvasClass)
+    {
+        WNDCLASS wndclass;
+
+        static const long styleNormal = CS_HREDRAW | CS_VREDRAW | CS_DBLCLKS | CS_OWNDC;
+
+        // the fields which are common to all classes
+        wndclass.lpfnWndProc   = (WNDPROC)wxWndProc;
+        wndclass.cbClsExtra    = 0;
+        wndclass.cbWndExtra    = sizeof( DWORD ); // VZ: what is this DWORD used for?
+        wndclass.hInstance     = wxhInstance;
+        wndclass.hIcon         = (HICON) NULL;
+        wndclass.hCursor       = ::LoadCursor((HINSTANCE)NULL, IDC_ARROW);
+        wndclass.lpszMenuName  = NULL;
+
+        // Register the GLCanvas class name
+        wndclass.hbrBackground = (HBRUSH)NULL;
+        wndclass.lpszClassName = wxGLCanvasClassName;
+        wndclass.style         = styleNormal;
+
+        if ( !RegisterClass(&wndclass) )
+        {
+            wxLogLastError("RegisterClass(wxGLCanvasClass)");
+
+            return FALSE;
+        }
+    }
+
+    wxCHECK_MSG( parent, FALSE, wxT("can't create wxWindow without parent") );
+
+    if ( !CreateBase(parent, id, pos, size, style, wxDefaultValidator, name) )
+        return FALSE;
+
+    parent->AddChild(this);
+
+    DWORD msflags = 0;
+    if ( style & wxBORDER )
+        msflags |= WS_BORDER;
+    if ( style & wxTHICK_FRAME )
+        msflags |= WS_THICKFRAME;
+
+    msflags |= WS_CHILD | WS_VISIBLE;
+    if ( style & wxCLIP_CHILDREN )
+        msflags |= WS_CLIPCHILDREN;
+
+    bool want3D;
+    WXDWORD exStyle = Determine3DEffects(WS_EX_CLIENTEDGE, &want3D);
+
+    // Even with extended styles, need to combine with WS_BORDER
+    // for them to look right.
+    if ( want3D || (m_windowStyle & wxSIMPLE_BORDER) || (m_windowStyle & wxRAISED_BORDER ) ||
+        (m_windowStyle & wxSUNKEN_BORDER) || (m_windowStyle & wxDOUBLE_BORDER))
+    {
+        msflags |= WS_BORDER;
+    }
+
+    // calculate the value to return from WM_GETDLGCODE handler
+    if ( GetWindowStyleFlag() & wxWANTS_CHARS )
+    {
+        // want everything: i.e. all keys and WM_CHAR message
+        m_lDlgCode = DLGC_WANTARROWS | DLGC_WANTCHARS |
+                     DLGC_WANTTAB | DLGC_WANTMESSAGE;
+    }
+
+    MSWCreate(m_windowId, parent, wxGLCanvasClassName, this, NULL,
+              pos.x, pos.y,
+              WidthDefault(size.x), HeightDefault(size.y),
+              msflags, NULL, exStyle);
+
+    return TRUE;
+
+}
+
 void wxGLCanvas::SetupPixelFormat() // (HDC hDC)
 {
     PIXELFORMATDESCRIPTOR pfd = {
index 186df14fafdf27f5629b61fea3fe91b3a0e82769..6dd7e60c6c6beae0f156a454784a62b37cf4314d 100644 (file)
@@ -79,6 +79,10 @@ class wxGLCanvas: public wxScrolledWindow
 
    ~wxGLCanvas();
 
+   // Replaces wxWindow::Create functionality, since we need to use a different window class
+   bool Create(wxWindow *parent, wxWindowID id,
+              const wxPoint& pos, const wxSize& size, long style, const wxString& name);
+
    void SetCurrent();
    void SetColour(const char *colour);
    void SwapBuffers();
index 077cb4032410667792e1f0dcd548aefb1baaf2de..ebe99e4e97fdd7e6054a163d7ac0794c5de0a1b3 100644 (file)
@@ -15,7 +15,7 @@
 WXDIR = $(WXWIN)
 GLDIR = $(WXDIR)\utils\glcanvas
 THISDIR = $(GLDIR)\win
-EXTRALIBS=$(WXDIR)\lib\glcanvas.lib
+EXTRALIBS=$(WXDIR)\lib\glcanvas$(LIBEXT).lib
 DOCDIR=$(WXDIR)\docs
 LOCALDOCDIR=$(WXDIR)\utils\glcanvas\docs