From 1e0e31cdeef144f9847533b793820a1ac4af61e5 Mon Sep 17 00:00:00 2001
From: dmaivel <dvmaivel@gmail.com>
Date: Fri, 5 Jul 2024 22:39:20 -0400
Subject: [PATCH] Add `glimpl_get_pixel_size`

---
 src/client/glimpl.c | 53 ++++++++++++++++-----------------------------
 1 file changed, 19 insertions(+), 34 deletions(-)

diff --git a/src/client/glimpl.c b/src/client/glimpl.c
index ffe0b86..b2f2abc 100644
--- a/src/client/glimpl.c
+++ b/src/client/glimpl.c
@@ -520,54 +520,49 @@ static struct gl_vertex_attrib_pointer *glimpl_get_enabled_vap()
     return NULL;
 }
 
-static void glimpl_upload_texture(GLsizei width, GLsizei height, GLsizei depth, GLenum format, const void* pixels)
+static inline size_t glimpl_get_pixel_size(GLenum format)
 {
-    if (pixels == NULL) {
-        pb_push(SGL_CMD_VP_NULL);
-        return;
-    }
-
-    size_t pixel_size;
-    size_t total_size;
-
     switch (format) {
     case GL_RGBA:
     case GL_RGBA8:
     case GL_BGRA:
     case GL_COMPRESSED_SRGB_ALPHA:
-        pixel_size = 4;
-        break;
+        return 4;
     case GL_RGB:
     case GL_BGR:
     case GL_RGB8:
-        pixel_size = 3;
-        break;
+        return 3;
     case GL_LUMINANCE_ALPHA:
     case GL_RG:
     case GL_RG8:
-        pixel_size = 2;
-        break;
+        return 2;
     case GL_LUMINANCE:
     case GL_ALPHA:
     case GL_RED:
     case GL_R8:
-        pixel_size = 1;
-        break;
+        return 1;
     case GL_RGB16F:
     case GL_RGB16I:
     case GL_RGB16UI:
-        pixel_size = 6;
-        break;
+        return 6;
     case GL_RGBA16F:
     case GL_RGBA16I:
     case GL_RGBA16UI:
-        pixel_size = 8;
-        break;
+        return 8;
     default:
+        return 1;
+    }
+}
+
+static void glimpl_upload_texture(GLsizei width, GLsizei height, GLsizei depth, GLenum format, const void* pixels)
+{
+    if (pixels == NULL) {
+        pb_push(SGL_CMD_VP_NULL);
         return;
     }
 
-    total_size = width * height * depth * pixel_size;
+    size_t total_size = width * height * depth * glimpl_get_pixel_size(format);
+
     pb_push(SGL_CMD_VP_UPLOAD);
     pb_push(CEIL_DIV(total_size, 4));
     pb_memcpy((void*)pixels, total_size);
@@ -595,18 +590,8 @@ static inline void glimpl_push_client_pointers(int mode, int max_index)
     if (glimpl_color_ptr.in_use) {
         int true_size = glimpl_color_ptr.size;
 
-        if (true_size > 4) {
-            switch (true_size) {
-            case GL_RGB:
-            case GL_BGR:
-                true_size = 3;
-                break;
-            case GL_RGBA:
-            case GL_BGRA:
-                true_size = 4;
-                break;
-            }
-        } 
+        if (true_size > 8)
+            true_size = glimpl_get_pixel_size(true_size);
 
         pb_push(SGL_CMD_VP_UPLOAD);
         pb_push(max_index);