Tidy up the screenshot functionality by making the write_png function…

… hidden.
pioneerspacesim · Jan 8, 2024 · 987cc11 · 987cc11
1 parent 2ac462c
commit 987cc11
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 8 deletions.
diff --git a/src/PngWriter.cpp b/src/PngWriter.cpp
@@ -8,6 +8,7 @@
 
 #include <SDL_image.h>
 
+namespace {
 void write_png(FileSystem::FileSourceFS &fs, const std::string &path, const Uint8 *bytes, int width, int height, int stride, int bytes_per_pixel, bool strip_alpha)
 {
 	// Set up the pixel format color masks for RGB(A) byte arrays.
@@ -39,7 +40,6 @@ void write_png(FileSystem::FileSourceFS &fs, const std::string &path, const Uint
 	}
 
 	// create a surface
-	//SDL_Surface *surface = SDL_CreateRGBSurfaceFrom((void*)bytes, width, height, bytes_per_pixel * 8, width * bytes_per_pixel, rmask, gmask, bmask, amask);
 	SDL_Surface *surface = SDL_CreateRGBSurface(0, width, height, dest_bpp * 8, rmask, gmask, bmask, amask);
 
 	// flip the image vertically and copy to destination surface
@@ -85,6 +85,7 @@ void write_png(FileSystem::FileSourceFS &fs, const std::string &path, const Uint
 	SDL_FreeSurface(surface);
 	surface = nullptr;
 }
+} //namespace
 
 void write_screenshot(const Graphics::ScreendumpState &sd, const char *destFile)
 {

diff --git a/src/PngWriter.h b/src/PngWriter.h
@@ -7,17 +7,10 @@
 #include <SDL_stdinc.h>
 #include <string>
 
-namespace FileSystem {
-	class FileSourceFS;
-}
-
 namespace Graphics {
 	struct ScreendumpState;
 }
 
-// stride is in bytes (bytes per row)
-void write_png(FileSystem::FileSourceFS &fs, const std::string &path, const Uint8 *bytes, int width, int height, int stride, int bytes_per_pixel, bool strip_alpha = false);
-
 void write_screenshot(const Graphics::ScreendumpState &sd, const char *destFile);
 
 #endif