Changeset 1788Tweet

Timestamp:

Aug 22, 2012, 7:14:02 PM (11 years ago)

Author:

sam

Message:

gpu: document and tweak some class declarations.

Location:

trunk/src

Files:

• gpu/vertexbuffer.h (modified) (3 diffs)
• video.h (modified) (2 diffs)

trunk/src/gpu/vertexbuffer.h

@@ -37 +37 @@
 };
 
+/* A safe enum to indicate how a vertex stream is going to be used. For
+ * now there is only TexCoord and not TexCoord0 TexCoord1 etc. because
+ * we can always reorganise the vertex declaration for the indices to
+ * match. If the need arises these enums will be added. */
 struct VertexUsage
 {
@@ -58 +62 @@
     m_value;
 
-    inline VertexUsage(Value v) { m_value = v; }
+    inline VertexUsage(Value v) : m_value(v) {}
     inline operator Value() { return m_value; }
 };
 
+/* A safe enum to indicate what kind of primitive to draw. Used in
+ * VertexDeclaration::DrawElements() for instance. */
 struct MeshPrimitive
 {
@@ -71 +77 @@
     m_value;
 
-    inline MeshPrimitive(Value v) { m_value = v; }
+    inline MeshPrimitive(Value v) : m_value(v) {}
     inline operator Value() { return m_value; }
 };

trunk/src/video.h

@@ -28 +28 @@
     {
         /* Note: D3D9 doesn't appear to support the accumulation buffer,
-         * and it is a deprecated OpenGL feature. No reasone to support it. */
+         * and it is a deprecated OpenGL feature. No reason to support it. */
         Color   = 1 << 0,
         Depth   = 1 << 1,
@@ -37 +37 @@
     m_value;
 
-    inline ClearMask(Value v) { m_value = v; }
-    inline ClearMask(uint64_t i) { m_value = (Value)i; }
+    inline ClearMask(Value v) : m_value(v) {}
+    inline ClearMask(uint64_t i) : m_value((Value)i) {}
     inline operator Value() { return m_value; }
 };