From: Mikko Rasa <tdb@tdb.fi>
Date: Sun, 25 Sep 2022 16:13:43 +0000 (+0300)
Subject: Refactor handling of texture storage properties in the Blender exporter
X-Git-Url: http://git.tdb.fi/?p=libs%2Fgl.git;a=commitdiff_plain;h=6fbc3217471932e1e46a694e4c4376673010f814

Refactor handling of texture storage properties in the Blender exporter
---

diff --git a/blender/io_mspgl/__init__.py b/blender/io_mspgl/__init__.py
index 4b6e2ff4..c23b60f1 100644
--- a/blender/io_mspgl/__init__.py
+++ b/blender/io_mspgl/__init__.py
@@ -10,7 +10,7 @@ files = ("animation", "armature", "context", "datafile", "export",
 	"export_animation", "export_armature", "export_camera", "export_light",
 	"export_material", "export_mesh", "export_object", "export_scene",
 	"export_texture", "material", "mesh", "operators", "properties", "scene",
-	"util")
+	"texture", "util")
 
 if "bpy" in locals():
 	import imp
diff --git a/blender/io_mspgl/export_texture.py b/blender/io_mspgl/export_texture.py
index 8e107a71..2269e27a 100644
--- a/blender/io_mspgl/export_texture.py
+++ b/blender/io_mspgl/export_texture.py
@@ -38,9 +38,8 @@ class TextureExporter:
 		if tex_node.use_mipmap:
 			tex_res.statements.append(Statement("generate_mipmap", True))
 
-		colorspace = image.colorspace_settings.name
-		if len(channels)==1 and colorspace=='sRGB':
-			raise Exception("Unsupported configuration on texture {}: Grayscale with sRGB".format(image.name))
+		from .texture import Texture
+		texture = Texture(tex_node, channels)
 
 		invert_mask = sum(1<<i for i, c in enumerate(channels) if c[0]=='~')
 
@@ -61,17 +60,10 @@ class TextureExporter:
 		if not invert_mask and channels==native_channels:
 			if not tex_node.use_mipmap:
 				tex_res.statements.append(Statement("mipmap_levels", 1))
-			srgb = "_srgb" if colorspace=='sRGB' else ""
+			srgb = "_srgb" if texture.srgb else ""
 			tex_res.statements.append(Statement("external_image"+srgb, fn))
 		else:
-			if len(channels)==4:
-				fmt = 'SRGB8_ALPHA8' if colorspace=='sRGB' else 'RGBA8'
-			elif len(channels)==1:
-				fmt = 'LUMINANCE8' if channels[0]=='Y' else 'R8'
-			else:
-				fmt = 'SRGB8' if colorspace=='sRGB' else 'RGB8'
-
-			tex_res.statements.append(Statement("storage", Token(fmt), image.size[0], image.size[1]))
+			tex_res.statements.append(Statement("storage", Token(texture.pixelformat), texture.width, texture.height))
 
 			pixels = tuple(image.pixels)
 			texdata = ""
diff --git a/blender/io_mspgl/texture.py b/blender/io_mspgl/texture.py
new file mode 100644
index 00000000..288c6627
--- /dev/null
+++ b/blender/io_mspgl/texture.py
@@ -0,0 +1,17 @@
+class Texture:
+	def __init__(self, tex_node, channels):
+		self.image = tex_node.image
+
+		self.srgb = self.image.colorspace_settings.name=='sRGB'
+		if len(channels)==1 and self.srgb:
+			raise Exception("Unsupported configuration on texture {}: Grayscale with sRGB".format(self.image.name))
+
+		if len(channels)==4:
+			self.pixelformat = 'SRGB8_ALPHA8' if self.srgb else 'RGBA8'
+		elif len(channels)==1:
+			self.pixelformat = 'LUMINANCE8' if channels[0]=='Y' else 'R8'
+		else:
+			self.pixelformat = 'SRGB8' if self.srgb else 'RGB8'
+
+		self.width = self.image.size[0]
+		self.height = self.image.size[1]