BLASTriangleDesc: fixed VertexValueType and VertexComponentCount handling

2 files changed, 10 insertions, 14 deletions

diff --git a/Graphics/GraphicsEngine/interface/BottomLevelAS.h b/Graphics/GraphicsEngine/interface/BottomLevelAS.h
index c93319da..be9eb0c1 100644
--- a/Graphics/GraphicsEngine/interface/BottomLevelAS.h
+++ b/Graphics/GraphicsEngine/interface/BottomLevelAS.h
@@ -58,19 +58,14 @@ struct BLASTriangleDesc
     Uint32                    MaxVertexCount        DEFAULT_INITIALIZER(0);
 
     /// The type of vertices in this geometry, see Diligent::VALUE_TYPE.
-
-    /// \remarks Only the following combinations of VertexValueType and VertexComponentCount
-    ///          are allowed:
-    ///          * FLOAT32 x 2 (third component is assumed 0)
-    ///          * FLOAT32 x 3
-    ///          * FLOAT16 x 2 (third component is assumed 0)
-    ///          * FLOAT16 x 4 (fourth component is ignored)
-    ///          * INT16   x 2 (16-bit signed-normalized, third component is assumed 0)
-    ///          * INT16   x 4 (16-bit signed-normalized, fourth component is ignored)
+    /// 
+    /// \remarks Only the following values are allowed: VT_FLOAT32, VT_FLOAT16, VT_INT16.
+    ///          VT_INT16 defines 16-bit signed normalized vertex components.
     VALUE_TYPE                VertexValueType       DEFAULT_INITIALIZER(VT_UNDEFINED);
 
     /// The number of components in the vertex.
-    /// See VertexValueType for the list of allowed VertexValueType and VertexComponentCount combinations.
+    /// 
+    /// \remarks Only 2 or 3 are allowed values. For 2-component formats, the third component is assumed 0.
     Uint8                     VertexComponentCount  DEFAULT_INITIALIZER(0);
 
     /// The maximum primitive count in this geometry.
diff --git a/Graphics/GraphicsEngine/src/BottomLevelASBase.cpp b/Graphics/GraphicsEngine/src/BottomLevelASBase.cpp
index c0e8f058..1059f451 100644
--- a/Graphics/GraphicsEngine/src/BottomLevelASBase.cpp
+++ b/Graphics/GraphicsEngine/src/BottomLevelASBase.cpp
@@ -63,11 +63,12 @@ void ValidateBottomLevelASDesc(const BottomLevelASDesc& Desc) noexcept(false)
             if (tri.GeometryName == nullptr)
                 LOG_BLAS_ERROR_AND_THROW("pTriangles[", i, "].GeometryName must not be null");
 
-            if (!((tri.VertexValueType == VT_FLOAT32 && (tri.VertexComponentCount == 2 || tri.VertexComponentCount == 3)) ||
-                  (tri.VertexValueType == VT_FLOAT16 && (tri.VertexComponentCount == 2 || tri.VertexComponentCount == 4)) ||
-                  (tri.VertexValueType == VT_INT16 && (tri.VertexComponentCount == 2 || tri.VertexComponentCount == 4))))
+            if (tri.VertexValueType != VT_FLOAT32 && tri.VertexValueType != VT_FLOAT16 && tri.VertexValueType != VT_INT16)
                 LOG_BLAS_ERROR_AND_THROW("pTriangles[", i, "].VertexValueType (", GetValueTypeString(tri.VertexValueType),
-                                         ") and .VertexComponentCount (", tri.VertexComponentCount, ") is not a valid combination");
+                                         ") is invalid. Only the following values are allowed: VT_FLOAT32, VT_FLOAT16, VT_INT16");
+
+            if (tri.VertexComponentCount != 2 && tri.VertexComponentCount != 3)
+                LOG_BLAS_ERROR_AND_THROW("pTriangles[", i, "].VertexComponentCount (", tri.VertexComponentCount, ") is invalid. Only 2 or 3 are allowed.");
 
             if (tri.MaxVertexCount == 0)
                 LOG_BLAS_ERROR_AND_THROW("pTriangles[", i, "].MaxVertexCount must be greater than 0");