From f4c996c0b98b2e13eca7d8fde21dabd70184c798 Mon Sep 17 00:00:00 2001
From: David Rose <david@ddrose.com>
Date: Fri, 10 Mar 2006 01:07:17 +0000
Subject: [PATCH] fix copy-to-cubemap

---
 .../glstuff/glGraphicsStateGuardian_src.cxx   | 84 +++++++++++--------
 1 file changed, 48 insertions(+), 36 deletions(-)

diff --git a/panda/src/glstuff/glGraphicsStateGuardian_src.cxx b/panda/src/glstuff/glGraphicsStateGuardian_src.cxx
index 052d0dea40..ae120b2464 100644
--- a/panda/src/glstuff/glGraphicsStateGuardian_src.cxx
+++ b/panda/src/glstuff/glGraphicsStateGuardian_src.cxx
@@ -2515,50 +2515,62 @@ framebuffer_copy_to_texture(Texture *tex, int z, const DisplayRegion *dr,
   CLP(TextureContext) *gtc = DCAST(CLP(TextureContext), tc);
   GLenum target = get_texture_target(tex->get_texture_type());
   GLP(BindTexture)(target, gtc->_index);
+
   GLint internal_format = get_internal_image_format(tex->get_format());
-  GLenum imagetarget = (z >= 0) ? (GL_TEXTURE_CUBE_MAP_POSITIVE_X + z) : GL_TEXTURE_2D;
 
-  // If the texture has never been uploaded before, create it.
-  // We cannot use glCopyTexImage2D to create a texture that may be
-  // larger than the screen, so use glTexImage2D with arbitrary data.
+  if (z >= 0) {
+    // Copy to a cube map face.  This doesn't seem to work too well
+    // with CopyTexSubImage2D, so we always use CopyTexImage2D.
+    GLP(CopyTexImage2D)(GL_TEXTURE_CUBE_MAP_POSITIVE_X + z, 0,
+                        get_internal_image_format(tex->get_format()),
+                        xo, yo, w, h, 0);
 
-  if ((gtc->_already_applied == false)||
-      (gtc->_internal_format != internal_format)||
-      (gtc->_width  != tex->get_x_size())||
-      (gtc->_height != tex->get_y_size())||
-      (gtc->_depth  != 1)) {
+  } else {
+    // If the texture has never been uploaded before, create it.  We
+    // cannot use glCopyTexImage2D to create a texture that may be
+    // larger than the screen, so use glTexImage2D with arbitrary
+    // data.
 
-    char *image = new char[tex->get_x_size() * tex->get_y_size()];
-    memset(image, 128, tex->get_x_size() * tex->get_y_size());
-    switch (tex->get_format()) {
-    case Texture::F_depth_component:
-      GLP(TexImage2D)(imagetarget, 0, internal_format,
-                      tex->get_x_size(), tex->get_y_size(), 0,
-                      GL_DEPTH_COMPONENT, GL_UNSIGNED_BYTE, image);
-      break;
-    case Texture::F_stencil_index:
-      GLP(TexImage2D)(imagetarget, 0, internal_format,
-                      tex->get_x_size(), tex->get_y_size(), 0,
-                      GL_STENCIL_INDEX, GL_UNSIGNED_BYTE, image);
-      break;
-    default:
-      GLP(TexImage2D)(imagetarget, 0, internal_format,
-                      tex->get_x_size(), tex->get_y_size(), 0,
-                      GL_LUMINANCE, GL_UNSIGNED_BYTE, image);
-      break;
+    if ((gtc->_already_applied == false)||
+        (gtc->_internal_format != internal_format)||
+        (gtc->_width  != tex->get_x_size())||
+        (gtc->_height != tex->get_y_size())||
+        (gtc->_depth  != 1)) {
+      
+      char *image = new char[tex->get_x_size() * tex->get_y_size()];
+      memset(image, 128, tex->get_x_size() * tex->get_y_size());
+      switch (tex->get_format()) {
+      case Texture::F_depth_component:
+        GLP(TexImage2D)(GL_TEXTURE_2D, 0, internal_format,
+                        tex->get_x_size(), tex->get_y_size(), 0,
+                        GL_DEPTH_COMPONENT, GL_UNSIGNED_BYTE, image);
+        break;
+        
+      case Texture::F_stencil_index:
+        GLP(TexImage2D)(GL_TEXTURE_2D, 0, internal_format,
+                        tex->get_x_size(), tex->get_y_size(), 0,
+                        GL_STENCIL_INDEX, GL_UNSIGNED_BYTE, image);
+        break;
+        
+      default:
+        GLP(TexImage2D)(GL_TEXTURE_2D, 0, internal_format,
+                        tex->get_x_size(), tex->get_y_size(), 0,
+                        GL_LUMINANCE, GL_UNSIGNED_BYTE, image);
+        break;
+      }
+      delete image;
+      
+      gtc->_already_applied = true;
+      gtc->_internal_format = internal_format;
+      gtc->_width  = tex->get_x_size();
+      gtc->_height = tex->get_y_size();
+      gtc->_depth  = 1;
     }
-    delete image;
 
-    gtc->_already_applied = true;
-    gtc->_internal_format = internal_format;
-    gtc->_width  = tex->get_x_size();
-    gtc->_height = tex->get_y_size();
-    gtc->_depth  = 1;
+    // Copy the pixel data from the frame buffer.
+    GLP(CopyTexSubImage2D)(GL_TEXTURE_2D, 0, 0, 0, xo, yo, w, h);
   }
 
-  // Copy the pixel data from the frame buffer.
-  GLP(CopyTexSubImage2D)(imagetarget, 0, 0, 0, xo, yo, w, h);
-
   report_my_gl_errors();
 
   // Force reload of texture state, since we've just monkeyed with it.