From b59441ced5ea1a88ab41be8ef230f13fd9812a91 Mon Sep 17 00:00:00 2001
From: "Erik W. Gren" <ErikWDev@gmail.com>
Date: Sat, 7 Dec 2024 22:03:31 +0100
Subject: [PATCH] support blitting to/from texture, not just renderbuffers

---
 blade-graphics/src/gles/command.rs | 66 +++++++++++++++++++-----------
 1 file changed, 42 insertions(+), 24 deletions(-)

diff --git a/blade-graphics/src/gles/command.rs b/blade-graphics/src/gles/command.rs
index d55dcce..1f140c9 100644
--- a/blade-graphics/src/gles/command.rs
+++ b/blade-graphics/src/gles/command.rs
@@ -813,37 +813,55 @@ impl super::Command {
 
             Self::BlitFramebuffer { from, to } => {
                 /*
-                    TODO(ErikWDev): Validate
+                    TODO: Framebuffers could be re-used instead of being created on the fly.
+                          Currently deleted down below
                 */
-
-                let target_from = match from.inner {
-                    super::TextureInner::Renderbuffer { raw } => raw,
-                    _ => panic!("Unsupported resolve between non-renderbuffers"),
-                };
-                let target_to = match to.inner {
-                    super::TextureInner::Renderbuffer { raw } => raw,
-                    _ => panic!("Unsupported resolve between non-renderbuffers"),
-                };
-
                 let framebuf_from = gl.create_framebuffer().unwrap();
                 let framebuf_to = gl.create_framebuffer().unwrap();
 
                 gl.bind_framebuffer(glow::READ_FRAMEBUFFER, Some(framebuf_from));
-                gl.framebuffer_renderbuffer(
-                    glow::READ_FRAMEBUFFER,
-                    glow::COLOR_ATTACHMENT0, // NOTE: Assuming color attachment
-                    glow::RENDERBUFFER,
-                    Some(target_from),
-                );
+                match from.inner {
+                    super::TextureInner::Renderbuffer { raw } => {
+                        gl.framebuffer_renderbuffer(
+                            glow::READ_FRAMEBUFFER,
+                            glow::COLOR_ATTACHMENT0, // NOTE: Assuming color attachment
+                            glow::RENDERBUFFER,
+                            Some(raw),
+                        );
+                    }
+                    super::TextureInner::Texture { raw, target } => {
+                        gl.framebuffer_texture_2d(
+                            glow::READ_FRAMEBUFFER,
+                            glow::COLOR_ATTACHMENT0,
+                            target,
+                            Some(raw),
+                            0,
+                        );
+                    }
+                }
 
                 gl.bind_framebuffer(glow::DRAW_FRAMEBUFFER, Some(framebuf_to));
-                gl.framebuffer_renderbuffer(
-                    glow::DRAW_FRAMEBUFFER,
-                    glow::COLOR_ATTACHMENT0, // NOTE: Assuming color attachment
-                    glow::RENDERBUFFER,
-                    Some(target_to),
-                );
-                assert_eq!(
+                match to.inner {
+                    super::TextureInner::Renderbuffer { raw } => {
+                        gl.framebuffer_renderbuffer(
+                            glow::DRAW_FRAMEBUFFER,
+                            glow::COLOR_ATTACHMENT0, // NOTE: Assuming color attachment
+                            glow::RENDERBUFFER,
+                            Some(raw),
+                        );
+                    }
+                    super::TextureInner::Texture { raw, target } => {
+                        gl.framebuffer_texture_2d(
+                            glow::DRAW_FRAMEBUFFER,
+                            glow::COLOR_ATTACHMENT0,
+                            target,
+                            Some(raw),
+                            0,
+                        );
+                    }
+                }
+
+                debug_assert_eq!(
                     gl.check_framebuffer_status(glow::DRAW_FRAMEBUFFER),
                     glow::FRAMEBUFFER_COMPLETE,
                     "DRAW_FRAMEBUFFER is not complete"