Added raw pointer overload for tc::img::image_save method.

Author: StefanoLusardiTC

image/include/teiacare/image/image_io.hpp

@@ -60,6 +60,21 @@ auto image_load_from_memory(
     uint8_t* memory_data,
     std::size_t memory_data_size) -> std::tuple<std::vector<uint8_t>, int, int, int>;
 
+/*!
+ * \brief Save image data to a file.
+ * \param output_path Path where the image file should be saved
+ * \param image_data Pointer to the image pixel data buffer
+ * \param width Width of the image in pixels
+ * \param height Height of the image in pixels
+ * \param channels Number of color channels (e.g., 3 for RGB, 4 for RGBA)
+ */
+void image_save(
+    const std::filesystem::path& image_path,
+    const uint8_t* image_data_ptr,
+    int width,
+    int height,
+    int channels);
+
 /*!
  * \brief Save image data to a file.
  * \param output_path Path where the image file should be saved

image/src/image_io.cpp

@@ -77,26 +77,26 @@ auto image_load_from_memory(uint8_t* memory_data, std::size_t memory_data_size)
     return create_image_data(image_data, width, height, channels);
 }
 
-void image_save(const std::filesystem::path& image_path, const std::vector<uint8_t>& image_data, int width, int height, int channels)
+void image_save(const std::filesystem::path& image_path, const uint8_t* image_data_ptr, int width, int height, int channels)
 {
     bool ok = false;
     const std::string image_ext = (image_path.has_extension() ? image_path.extension().string() : "png");
 
     if (image_ext == ".png")
     {
-        ok = stbi_write_png(image_path.string().c_str(), width, height, channels, image_data.data(), width * channels) != 0;
+        ok = stbi_write_png(image_path.string().c_str(), width, height, channels, image_data_ptr, width * channels) != 0;
     }
     else if (image_ext == ".jpg" || image_ext == ".jpeg")
     {
-        ok = stbi_write_jpg(image_path.string().c_str(), width, height, channels, image_data.data(), /*quality*/ 100) != 0;
+        ok = stbi_write_jpg(image_path.string().c_str(), width, height, channels, image_data_ptr, /*quality*/ 100) != 0;
     }
     else if (image_ext == ".bmp")
     {
-        ok = stbi_write_bmp(image_path.string().c_str(), width, height, channels, image_data.data()) != 0;
+        ok = stbi_write_bmp(image_path.string().c_str(), width, height, channels, image_data_ptr) != 0;
     }
     else if (image_ext == ".tga")
     {
-        ok = stbi_write_tga(image_path.string().c_str(), width, height, channels, image_data.data()) != 0;
+        ok = stbi_write_tga(image_path.string().c_str(), width, height, channels, image_data_ptr) != 0;
     }
     else
     {
@@ -108,4 +108,9 @@ void image_save(const std::filesystem::path& image_path, const std::vector<uint8
     }
 }
 
+void image_save(const std::filesystem::path& image_path, const std::vector<uint8_t>& image_data, int width, int height, int channels)
+{
+    return image_save(image_path, image_data.data(), width, height, channels);
+}
+
 }

image/tests/test_image_io.cpp

@@ -230,6 +230,18 @@ TEST_F(image_io_test, image_save_png)
     EXPECT_GT(std::filesystem::file_size(output_file), 0);
 }
 
+// Test tc::img::image_save with raw pointer version
+TEST_F(image_io_test, image_save_png_raw_pointer)
+{
+    int width = 4, height = 3, channels = 3;
+    auto image_data = createUniformImageData(width, height, channels, 128);
+    auto output_file = temp_dir_ / "test_ptr.png";
+
+    EXPECT_NO_THROW(tc::img::image_save(output_file, image_data.data(), width, height, channels));
+    EXPECT_TRUE(std::filesystem::exists(output_file));
+    EXPECT_GT(std::filesystem::file_size(output_file), 0);
+}
+
 // Test tc::img::image_save with JPEG format
 TEST_F(image_io_test, image_save_jpeg)
 {