Rename ShaderRenderer conventions enum

AcademySoftwareFoundation · Aug 24, 2023 · d0d21fc · d0d21fc
1 parent 414623c
commit d0d21fc
Show file tree

Hide file tree

Showing 4 changed files with 12 additions and 12 deletions.
diff --git a/source/MaterialXRender/ShaderRenderer.cpp b/source/MaterialXRender/ShaderRenderer.cpp
@@ -25,23 +25,23 @@ const float DEFAULT_FAR_PLANE = 100.0f;
 // ShaderRenderer methods
 //
 
-ShaderRenderer::ShaderRenderer(unsigned int width, unsigned int height, Image::BaseType baseType, ConventionAPI conventionAPI) :
+ShaderRenderer::ShaderRenderer(unsigned int width, unsigned int height, Image::BaseType baseType, MatrixConvention matrixConvention) :
     _width(width),
     _height(height),
     _baseType(baseType),
-    _conventionAPI(conventionAPI)
+    _matrixConvention(matrixConvention)
 {
     // Initialize a default camera.
     float fH = std::tan(DEFAULT_FIELD_OF_VIEW / 360.0f * PI) * DEFAULT_NEAR_PLANE;
     float fW = fH * 1.0f;
     _camera = Camera::create();
     _camera->setViewMatrix(Camera::createViewMatrix(DEFAULT_EYE_POSITION, DEFAULT_TARGET_POSITION, DEFAULT_UP_VECTOR));
 
-    if (_conventionAPI == ShaderRenderer::ConventionAPI::METAL)
+    if (_matrixConvention == ShaderRenderer::MatrixConvention::Metal)
     {
         _camera->setProjectionMatrix(Camera::createPerspectiveMatrixZP(-fW, fW, -fH, fH, DEFAULT_NEAR_PLANE, DEFAULT_FAR_PLANE));
     }
-    else // ConventionAPI::OPENGL (default)
+    else // MatrixConvention::OpenGL (default)
     {
         _camera->setProjectionMatrix(Camera::createPerspectiveMatrix(-fW, fW, -fH, fH, DEFAULT_NEAR_PLANE, DEFAULT_FAR_PLANE));
     }

diff --git a/source/MaterialXRender/ShaderRenderer.h b/source/MaterialXRender/ShaderRenderer.h
@@ -30,11 +30,11 @@ using ShaderRendererPtr = std::shared_ptr<class ShaderRenderer>;
 class MX_RENDER_API ShaderRenderer
 {
   public:
-    /// API viewing conventions designation (default to OpenGL).
-    enum class ConventionAPI
+    /// Viewing API matrix conventions designation (default to OpenGL).
+    enum class MatrixConvention
     {
-        OPENGL = 0,
-        METAL  = 1
+        OpenGL = 0,
+        Metal  = 1
     };
     /// A map with name and source code for each shader stage.
     using StageMap = StringMap;
@@ -130,14 +130,14 @@ class MX_RENDER_API ShaderRenderer
 
   protected:
     ShaderRenderer(unsigned int width, unsigned int height, Image::BaseType baseType,
-        ConventionAPI conventionAPI = ConventionAPI::OPENGL);
+        MatrixConvention matrixConvention = MatrixConvention::OpenGL);
 
   protected:
     unsigned int _width;
     unsigned int _height;
     Image::BaseType _baseType;
 
-    ConventionAPI _conventionAPI;
+    MatrixConvention _matrixConvention;
 
     CameraPtr _camera;
     ImageHandlerPtr _imageHandler;

diff --git a/source/MaterialXRenderGlsl/GlslRenderer.cpp b/source/MaterialXRenderGlsl/GlslRenderer.cpp
@@ -25,7 +25,7 @@ GlslRendererPtr GlslRenderer::create(unsigned int width, unsigned int height, Im
 }
 
 GlslRenderer::GlslRenderer(unsigned int width, unsigned int height, Image::BaseType baseType) :
-    ShaderRenderer(width, height, baseType, ConventionAPI::OPENGL),
+    ShaderRenderer(width, height, baseType, MatrixConvention::OpenGL),
     _initialized(false),
     _screenColor(DEFAULT_SCREEN_COLOR_LIN_REC709)
 {

diff --git a/source/MaterialXRenderMsl/MslRenderer.mm b/source/MaterialXRenderMsl/MslRenderer.mm
@@ -29,7 +29,7 @@
 }
 
 MslRenderer::MslRenderer(unsigned int width, unsigned int height, Image::BaseType baseType) :
-    ShaderRenderer(width, height, baseType, ConventionAPI::METAL),
+    ShaderRenderer(width, height, baseType, MatrixConvention::Metal),
     _initialized(false),
     _screenColor(DEFAULT_SCREEN_COLOR_LIN_REC709)
 {