Use stdbyte (#88)

* Refactor to use std::byte
JerboaBurrow · Aug 7, 2024 · 80147c4 · 80147c4
1 parent b452381
commit 80147c4
Show file tree

Hide file tree

Showing 13 changed files with 71 additions and 57 deletions.
diff --git a/examples/Sprite/main.cpp b/examples/Sprite/main.cpp
@@ -36,7 +36,7 @@ int main(int argv, char ** argc)
 
     std::shared_ptr<jGL::Texture> jerboa = jGLInstance->createTexture
     (
-        std::vector<unsigned char>(LOGO32,LOGO32+sizeof(LOGO32)),
+        std::vector(LOGO32.begin(), LOGO32.end()),
         jGL::Texture::Type::RGBA
     );
 

diff --git a/include/jGL/Display/desktopDisplay.h b/include/jGL/Display/desktopDisplay.h
@@ -275,16 +275,17 @@ namespace jGL
             }
         };
 
-        void setIcon(const std::vector<std::vector<unsigned char>> & icons)
+        void setIcon(const std::vector<std::vector<std::byte>> & icons)
         {
             glfwSetWindowIcon(glfwWindow, 0, NULL);
             logo.clear();
             for (auto icon : icons)
             {
                 GLFWimage image;
+                unsigned char * chData = reinterpret_cast<unsigned char*>(icon.data());
                 image.pixels = stbi_load_from_memory
                 (
-                    icon.data(), 
+                    chData, 
                     icon.size(), 
                     &image.width, 
                     &image.height, 

diff --git a/include/jGL/OpenGL/Texture/glTexture.h b/include/jGL/OpenGL/Texture/glTexture.h
@@ -25,7 +25,7 @@ namespace jGL::GL
             glDeleteTextures(1, &id);
         }
 
-        virtual void upload(unsigned char * data) {};
+        virtual void upload(std::vector<std::byte> * data) {};
 
         inline void bind() { glBindTexture(GL_TEXTURE_2D, id); }
 
@@ -46,33 +46,36 @@ namespace jGL::GL
         GLuint id;
         unsigned textureUnit;
 
-        std::vector<unsigned char> load_image(std::filesystem::path imageFilePath)
+        std::vector<std::byte> load_image(std::filesystem::path imageFilePath)
         {
             unsigned char * pixels = stbi_load(imageFilePath.generic_string().c_str(), &width, &height, &channels, 0);
 
             if (!pixels)
             {
                 throw std::runtime_error("Failed to load texture: "+imageFilePath.generic_string());
             }
+            std::byte * bytes = reinterpret_cast<std::byte*>(pixels);
 
             size_t dim = width*height;
             if (channels > 0) { dim *= channels; }
-            std::vector<unsigned char> vdata(pixels, pixels+dim);
+            std::vector<std::byte> vdata(bytes, bytes+dim);
             stbi_image_free(pixels);
             return vdata;
         }
 
-        std::vector<unsigned char> load_image(std::vector<unsigned char> imageFile)
+        std::vector<std::byte> load_image(std::vector<std::byte> imageFile)
         {
-            unsigned char * pixels = stbi_load_from_memory(imageFile.data(), imageFile.size(), &width, &height, &channels, 4);
+            unsigned char * chData = reinterpret_cast<unsigned char*>(imageFile.data());
+            unsigned char * pixels = stbi_load_from_memory(chData, imageFile.size(), &width, &height, &channels, 4);
             if (!pixels)
             {
                 throw std::runtime_error("Failed to load texture from memory");
             }
+            std::byte * bytes = reinterpret_cast<std::byte*>(pixels);
 
             size_t dim = width*height;
             if (channels > 0) { dim *= channels; }
-            std::vector<unsigned char> vdata(pixels, pixels+dim);
+            std::vector<std::byte> vdata(bytes, bytes+dim);
             stbi_image_free(pixels);
             return vdata;
         }
@@ -87,21 +90,21 @@ namespace jGL::GL
         glTexture2DRGB(std::filesystem::path imageFile)
         : glTexture()
         {
-            std::vector<unsigned char> pixels = load_image(imageFile);
+            std::vector<std::byte> pixels = load_image(imageFile);
             create(width, height, channels);
             upload(pixels);
         }
 
-        glTexture2DRGB(std::vector<unsigned char> data)
+        glTexture2DRGB(std::vector<std::byte> data)
         : glTexture()
         {
-            std::vector<unsigned char> pixels = load_image(data);
+            std::vector<std::byte> pixels = load_image(data);
             create(width, height, channels);
             upload(pixels);
         }
 
         void create(int width, int height, int channels);
-        void upload(std::vector<unsigned char> pixels);
+        void upload(std::vector<std::byte> pixels);
     };
 
     class glTexture2DRGBA : public glTexture
@@ -112,21 +115,21 @@ namespace jGL::GL
         glTexture2DRGBA(std::filesystem::path imageFile)
         : glTexture()
         {
-            std::vector<unsigned char> pixels = load_image(imageFile);
+            std::vector<std::byte> pixels = load_image(imageFile);
             create(width, height, channels);
             upload(pixels);
         }
 
-        glTexture2DRGBA(std::vector<unsigned char> data)
+        glTexture2DRGBA(std::vector<std::byte> data)
         : glTexture()
         {
-            std::vector<unsigned char> pixels = load_image(data);
+            std::vector<std::byte> pixels = load_image(data);
             create(width, height, channels);
             upload(pixels);
         }
 
         void create(int width, int height, int channels);
-        void upload(std::vector<unsigned char> pixels);
+        void upload(std::vector<std::byte> pixels);
     };
 
     class glTexture2DByte : public glTexture
@@ -137,28 +140,28 @@ namespace jGL::GL
         glTexture2DByte(std::filesystem::path imageFile)
         : glTexture()
         {
-            std::vector<unsigned char> pixels = load_image(imageFile);
+            std::vector<std::byte> pixels = load_image(imageFile);
             create(width, height);
             upload(pixels);
         }
 
-        glTexture2DByte(std::vector<unsigned char> data)
+        glTexture2DByte(std::vector<std::byte> data)
         : glTexture()
         {
-            std::vector<unsigned char> pixels = load_image(data);
+            std::vector<std::byte> pixels = load_image(data);
             create(width, height);
             upload(pixels);
         }
 
-        glTexture2DByte(std::vector<unsigned char> pixels, int width, int height)
+        glTexture2DByte(std::vector<std::byte> pixels, int width, int height)
         : glTexture()
         {
             create(width, height);
             upload(pixels);
         }
 
         void create(int width, int height);
-        void upload(std::vector<unsigned char> pixels);
+        void upload(std::vector<std::byte> pixels);
 
     };
 }

diff --git a/include/jGL/OpenGL/openGLInstance.h b/include/jGL/OpenGL/openGLInstance.h
@@ -88,7 +88,7 @@ namespace jGL::GL
                 }   
             }
 
-            std::shared_ptr<Texture> createTexture(std::vector<unsigned char> data, Texture::Type type)
+            std::shared_ptr<Texture> createTexture(std::vector<std::byte> data, Texture::Type type)
             { 
                 switch (type)
                 {

diff --git a/include/jGL/Vulkan/Texture/vkTexture.h b/include/jGL/Vulkan/Texture/vkTexture.h
@@ -74,7 +74,7 @@ namespace jGL::Vulkan
             uint32_t height, 
             uint32_t channels, 
             VkFormat format, 
-            std::vector<unsigned char> pixels,
+            std::vector<std::byte> pixels,
             VkSampleCountFlagBits msaaSamples = VK_SAMPLE_COUNT_1_BIT
         );
 
@@ -96,7 +96,7 @@ namespace jGL::Vulkan
             uint32_t height, 
             uint32_t channels, 
             VkFormat format,
-            std::vector<unsigned char> pixels
+            std::vector<std::byte> pixels
         );
 
     protected:
@@ -130,7 +130,7 @@ namespace jGL::Vulkan
         void uploadImage
         (
             const Command & command,
-            std::vector<unsigned char> pixels,
+            std::vector<std::byte> pixels,
             VkFormat format,
             uint32_t width,
             uint32_t height,

diff --git a/include/jGL/Vulkan/vulkanInstance.h b/include/jGL/Vulkan/vulkanInstance.h
@@ -100,7 +100,7 @@ namespace jGL::Vulkan
                 )
             );   
         }
-        std::shared_ptr<Texture> createTexture(std::vector<unsigned char> data, Texture::Type type) 
+        std::shared_ptr<Texture> createTexture(std::vector<std::byte> data, Texture::Type type) 
         { 
             TODO("VulkanInstance::createTexture from bytes undefined");
             return std::static_pointer_cast<Texture>

diff --git a/include/jGL/font.h b/include/jGL/font.h
@@ -37,7 +37,7 @@ namespace jGL
         std::map<unsigned char, glm::vec4> glyphOffset;
 
         std::shared_ptr<Texture> fontBitmap;
-        std::vector<unsigned char> bitmapPixels;
+        std::vector<std::byte> bitmapPixels;
         uint16_t bw, bh;
 
     };

diff --git a/include/jGL/jGL.h b/include/jGL/jGL.h
@@ -52,7 +52,7 @@ namespace jGL
 
         virtual std::shared_ptr<Particles> createParticles(size_t sizeHint) = 0;
         virtual std::shared_ptr<Texture> createTexture(std::filesystem::path imageFile, Texture::Type type) = 0;
-        virtual std::shared_ptr<Texture> createTexture(std::vector<unsigned char> data, Texture::Type type) = 0;
+        virtual std::shared_ptr<Texture> createTexture(std::vector<std::byte> data, Texture::Type type) = 0;
         virtual std::shared_ptr<SpriteRenderer> createSpriteRenderer(size_t sizeHint) = 0;
         virtual std::shared_ptr<ShapeRenderer> createShapeRenderer(size_t sizeHint) = 0;
 

diff --git a/include/logo.h b/include/logo.h
diff --git a/src/jGL/Display/desktopDisplay.cpp b/src/jGL/Display/desktopDisplay.cpp
@@ -97,11 +97,11 @@ namespace jGL
         glfwSetMouseButtonCallback(glfwWindow, mouseButtonCallback);
         glfwSetScrollCallback(glfwWindow, mouseScrollCallback);
 
-        std::vector<std::vector<unsigned char>> logos =
+        std::vector<std::vector<std::byte>> logos =
         {
-            {std::vector<unsigned char>(LOGO32, LOGO32+sizeof(LOGO32))},
-            {std::vector<unsigned char>(LOGO64, LOGO64+sizeof(LOGO64))},
-            {std::vector<unsigned char>(LOGO128, LOGO128+sizeof(LOGO128))}
+            {std::vector(LOGO32.begin(), LOGO32.end())},
+            {std::vector(LOGO64.begin(), LOGO64.end())},
+            {std::vector(LOGO128.begin(), LOGO128.end())}
         };
 
         setIcon(logos);

diff --git a/src/jGL/OpenGL/Texture/glTexture.cpp b/src/jGL/OpenGL/Texture/glTexture.cpp
@@ -50,7 +50,7 @@ namespace jGL::GL
 
     }
 
-    void glTexture2DRGB::upload(std::vector<unsigned char> pixels)
+    void glTexture2DRGB::upload(std::vector<std::byte> pixels)
     {
         bind();
 
@@ -114,7 +114,7 @@ namespace jGL::GL
         glBindTexture(GL_TEXTURE_2D, 0);
     }
 
-    void glTexture2DRGBA::upload(std::vector<unsigned char> pixels)
+    void glTexture2DRGBA::upload(std::vector<std::byte> pixels)
     {
         bind();
         glTexImage2D(
@@ -178,7 +178,7 @@ namespace jGL::GL
 
     }
 
-    void glTexture2DByte::upload(std::vector<unsigned char> pixels)
+    void glTexture2DByte::upload(std::vector<std::byte> pixels)
     {
         bind();
 

diff --git a/src/jGL/Vulkan/Texture/vkTexture.cpp b/src/jGL/Vulkan/Texture/vkTexture.cpp
@@ -33,14 +33,17 @@ namespace jGL::Vulkan
             this->channels = 4;
             format = VK_FORMAT_R8G8B8A8_SRGB;         
         }
+        std::byte * bytes = reinterpret_cast<std::byte*>(pixels);
 
         this->width = width;
         this->height = height;
+        int dim = width*height;
+        if (channels > 0){ dim *= channels; }
 
         uploadImage
         (
             command,
-            std::vector<unsigned char> {pixels, pixels+width*height*channels},
+            std::vector<std::byte>(bytes, bytes+dim),
             format,
             this->width,
             this->height,
@@ -64,7 +67,7 @@ namespace jGL::Vulkan
         uint32_t height, 
         uint32_t channels, 
         VkFormat format, 
-        std::vector<unsigned char> pixels,
+        std::vector<std::byte> pixels,
         VkSampleCountFlagBits msaaSamples
     )
     : device(device), msaaSamples(msaaSamples)
@@ -103,7 +106,7 @@ namespace jGL::Vulkan
         height, 
         channels, 
         format,
-        std::vector<unsigned char>(width*height*channels, 0),
+        std::vector<std::byte>(width*height*channels),
         msaaSamples
       )
     {}
@@ -299,7 +302,7 @@ namespace jGL::Vulkan
     void vkTexture::uploadImage
     (
         const Command & command,
-        std::vector<unsigned char> pixels,
+        std::vector<std::byte> pixels,
         VkFormat format,
         uint32_t width,
         uint32_t height,
@@ -373,7 +376,7 @@ namespace jGL::Vulkan
         uint32_t height, 
         uint32_t channels, 
         VkFormat format, 
-        std::vector<unsigned char> pixels
+        std::vector<std::byte> pixels
     )
     {
         if 

diff --git a/src/jGL/font.cpp b/src/jGL/font.cpp
@@ -30,7 +30,7 @@ namespace jGL
         int chWidth = s;
         int chHeight = s;
 
-        bitmapPixels = std::vector<unsigned char>(chWidth*chHeight*128, 0);
+        bitmapPixels = std::vector<std::byte>(chWidth*chHeight*128);
 
         uint16_t cursorX = 0;
         uint16_t cursorY = 0;
@@ -70,7 +70,7 @@ namespace jGL
                     px = cursorX;
                     py += 1;
                 }
-                bitmapPixels[py*width+px+cursorX*chWidth] = chPixels[i];
+                bitmapPixels[py*width+px+cursorX*chWidth] = std::byte(chPixels[i]);
                 px++;
             }
-Original file line number
+Diff line change
@@ Expand Up / @@ -88,7 +88,7 @@ namespace jGL::GL @@
                     }
                 }
-                std::shared_ptr<Texture> createTexture(std::vector<unsigned char> data, Texture::Type type)
+                std::shared_ptr<Texture> createTexture(std::vector<std::byte> data, Texture::Type type)
                 {
                     switch (type)
                     {
@@ Expand Down @@