commit 950192dbe35d3c0d3895e1bdcac89d86b6122dea
author Amit Feller <amit@graphtech.co.il> Thu Jun 23 13:55:43 2011 +0300
committer Guy Zadickario <guy@graphtech.co.il> Sun Jul 03 15:14:59 2011 +0300
tree e864f2f2aabc3075866c93c8dde2081ed2952d2c
parent 6adfa495baf3dc29a8b08fae02e5afd4ec9520a6

opengl translator: EGL Windows bug fix

Fixed bug caused the creation of context
without setting it's pixel format.
Also removing unnecessary printfs.

tools/emulator/opengl/host/libs/Translator/EGL/EglWindowsApi.cpp

diff --git a/tools/emulator/opengl/host/libs/Translator/EGL/EglWindowsApi.cpp b/tools/emulator/opengl/host/libs/Translator/EGL/EglWindowsApi.cpp
index f92a010..cb1188e 100644
--- a/tools/emulator/opengl/host/libs/Translator/EGL/EglWindowsApi.cpp
+++ b/tools/emulator/opengl/host/libs/Translator/EGL/EglWindowsApi.cpp
@@ -44,6 +44,7 @@
      void setCurrent(int configurationIndex,const DisplayInfo& info);
      bool isCurrentPixelFormatSet(){ return m_map[m_current].isPixelFormatSet;}
      void currentPixelFormatWasSet(){m_map[m_current].isPixelFormatSet = true;}
+     bool needToSetCurrent(int configurationIndex);
      void releaseAll();
 private:
     std::map<int,DisplayInfo> m_map;
@@ -57,8 +58,11 @@
     }
 }
 
+bool WinDisplay::needToSetCurrent(int configurationIndex){
+    return m_map.find(configurationIndex) == m_map.end();
+}
+
 void WinDisplay::setCurrent(int configurationIndex,const DisplayInfo& info){
-    if(m_map.find(configurationIndex) != m_map.end()) return;
     m_map[configurationIndex] = info;
     m_current = configurationIndex;
 }
@@ -321,7 +325,7 @@
 }
 
 static bool setPixelFormat(HDC dc,EglConfig* cfg) {
-   EGLint red,green,blue,alpha,depth,stencil;
+    EGLint red,green,blue,alpha,depth,stencil;
     bool   gotAttribs = cfg->getConfAttrib(EGL_RED_SIZE,&red)     &&
                         cfg->getConfAttrib(EGL_GREEN_SIZE,&green) &&
                         cfg->getConfAttrib(EGL_BLUE_SIZE,&blue)   &&
@@ -457,9 +461,14 @@
 EGLNativeContextType createContext(EGLNativeDisplayType display,EglConfig* cfg,EGLNativeContextType sharedContext) {
 
     EGLNativeContextType ctx = NULL;
-    HWND hwnd = createDummyWindow();
-    HDC  dpy  =  GetDC(hwnd);
-    display->setCurrent(cfg->id(),DisplayInfo(dpy,hwnd));
+    HDC  dpy  = NULL;
+    if(display->needToSetCurrent(cfg->id())){
+        HWND hwnd = createDummyWindow();
+        HDC  dpy  = GetDC(hwnd);
+        display->setCurrent(cfg->id(),DisplayInfo(dpy,hwnd));
+    } else {
+        dpy = display->getCurrentDC();
+    }
 
     if(!display->isCurrentPixelFormatSet()){
         if(!setPixelFormat(dpy,cfg)) return NULL;
@@ -523,9 +532,7 @@
 
 
     if(hdcRead == hdcDraw){
-            printf("making current read == draw\n");
             bool ret =  wglMakeCurrent(hdcDraw,ctx);
-            printf("last error is %d\n",GetLastError());
             return ret;
     } else if (!s_wglExtProcs->wglMakeContextCurrentARB ) {
         return false;