Merge branch 'model-factory-textures'

3 files changed, 36 insertions, 15 deletions

diff --git a/gfx/models/texture.cpp b/gfx/models/texture.cpp
index ef6d7e7..efc76e1 100644
--- a/gfx/models/texture.cpp
+++ b/gfx/models/texture.cpp
@@ -12,34 +12,36 @@
 

 Cache<Texture, std::filesystem::path> Texture::cachedTexture;

 

-Texture::Texture(const std::filesystem::path & fileName) :

-	Texture {Image {Resource::mapPath(fileName).c_str(), STBI_rgb_alpha}}

+Texture::Texture(const std::filesystem::path & fileName, TextureOptions to) :

+	Texture {Image {Resource::mapPath(fileName).c_str(), STBI_rgb_alpha}, to}

 {

 }

 

-Texture::Texture(const Image & tex) :

-	Texture {static_cast<GLsizei>(tex.width), static_cast<GLsizei>(tex.height), tex.data.data()}

+Texture::Texture(const Image & tex, TextureOptions to) :

+	Texture {static_cast<GLsizei>(tex.width), static_cast<GLsizei>(tex.height), tex.data.data(), to}

 {

 }

 

-Texture::Texture(GLsizei width, GLsizei height, const void * data)

+Texture::Texture(GLsizei width, GLsizei height, TextureOptions to) : Texture {width, height, nullptr, to} { }

+

+Texture::Texture(GLsizei width, GLsizei height, const void * data, TextureOptions to) : type {to.type}

 {

-	glBindTexture(GL_TEXTURE_2D, m_texture);

+	glBindTexture(type, m_texture);

 	glPixelStorei(GL_UNPACK_ALIGNMENT, 1);

 

-	glTexParameteri(GL_TEXTURE_2D, GL_TEXTURE_WRAP_S, GL_REPEAT);

-	glTexParameteri(GL_TEXTURE_2D, GL_TEXTURE_WRAP_T, GL_REPEAT);

+	glTexParameteri(type, GL_TEXTURE_WRAP_S, to.wrap);

+	glTexParameteri(type, GL_TEXTURE_WRAP_T, to.wrap);

 

-	glTexParameteri(GL_TEXTURE_2D, GL_TEXTURE_MIN_FILTER, GL_LINEAR);

-	glTexParameteri(GL_TEXTURE_2D, GL_TEXTURE_MAG_FILTER, GL_LINEAR);

-	glTexImage2D(GL_TEXTURE_2D, 0, GL_RGBA, width, height, 0, GL_RGBA, GL_UNSIGNED_BYTE, data);

+	glTexParameteri(type, GL_TEXTURE_MIN_FILTER, to.minFilter);

+	glTexParameteri(type, GL_TEXTURE_MAG_FILTER, to.magFilter);

+	glTexImage2D(type, 0, GL_RGBA, width, height, 0, GL_RGBA, GL_UNSIGNED_BYTE, data);

 }

 

 void

 Texture::bind(GLenum unit) const

 {

 	glActiveTexture(unit);

-	glBindTexture(GL_TEXTURE_2D, m_texture);

+	glBindTexture(type, m_texture);

 }

 

 void

@@ -61,6 +63,12 @@ Texture::save(const glTexture & texture, GLenum format, GLenum type, const glm::
 }

 

 void

+Texture::save(const glm::ivec2 & size, const char * path) const

+{

+	save(m_texture, GL_BGR, GL_UNSIGNED_BYTE, size, 3, path, 2);

+}

+

+void

 Texture::save(const glTexture & texture, const glm::ivec2 & size, const char * path)

 {

 	save(texture, GL_BGR, GL_UNSIGNED_BYTE, size, 3, path, 2);

diff --git a/gfx/models/texture.h b/gfx/models/texture.h
index 1aad1e0..ffc9a4a 100644
--- a/gfx/models/texture.h
+++ b/gfx/models/texture.h
@@ -8,16 +8,24 @@
 // IWYU pragma: no_forward_declare Cache

 class Image;

 

+struct TextureOptions {

+	GLint wrap {GL_REPEAT};

+	GLint minFilter {GL_LINEAR}, magFilter {GL_LINEAR};

+	GLenum type {GL_TEXTURE_2D};

+};

+

 class Texture {

 public:

-	explicit Texture(const std::filesystem::path & fileName);

-	explicit Texture(const Image & image);

-	explicit Texture(GLsizei width, GLsizei height, const void * data);

+	explicit Texture(const std::filesystem::path & fileName, TextureOptions = {});

+	explicit Texture(const Image & image, TextureOptions = {});

+	explicit Texture(GLsizei width, GLsizei height, TextureOptions = {});

+	explicit Texture(GLsizei width, GLsizei height, const void * data, TextureOptions = {});

 

 	static Cache<Texture, std::filesystem::path> cachedTexture;

 

 	void bind(GLenum unit = GL_TEXTURE0) const;

 

+	void save(const glm::ivec2 & size, const char * path) const;

 	static void save(const glTexture &, const glm::ivec2 & size, const char * path);

 	static void saveDepth(const glTexture &, const glm::ivec2 & size, const char * path);

 	static void saveNormal(const glTexture &, const glm::ivec2 & size, const char * path);

@@ -27,4 +35,5 @@ private:
 			const char * path, short tgaFormat);

 

 	glTexture m_texture;

+	GLenum type;

 };

diff --git a/gfx/models/vertex.hpp b/gfx/models/vertex.hpp
index 325aab1..64ec3d0 100644
--- a/gfx/models/vertex.hpp
+++ b/gfx/models/vertex.hpp
@@ -4,10 +4,14 @@
 
 class Vertex {
 public:
+#ifndef __cpp_aggregate_paren_init
 	constexpr Vertex(glm::vec3 pos, glm::vec2 texCoord, glm::vec3 normal, glm::vec4 colour = {}) :
 		pos {std::move(pos)}, texCoord {std::move(texCoord)}, normal {std::move(normal)}, colour {std::move(colour)}
 	{
 	}
+#endif
+
+	bool operator==(const Vertex &) const = default;
 
 	glm::vec3 pos;
 	glm::vec2 texCoord;