mat4x2: add tables to docs & fix comments

Signed-off-by: Vincent Davis Jr <vince@underview.tech>

include/cglm/call/mat4x2.h

@@ -15,11 +15,11 @@ extern "C" {
 
 CGLM_EXPORT
 void
-glmc_mat4x2_copy(mat4x2 mat, mat4x2 dest);
+glmc_mat4x2_copy(mat4x2 src, mat4x2 dest);
 
 CGLM_EXPORT
 void
-glmc_mat4x2_zero(mat4x2 mat);
+glmc_mat4x2_zero(mat4x2 m);
 
 CGLM_EXPORT
 void
@@ -35,7 +35,7 @@ glmc_mat4x2_mulv(mat4x2 m, vec4 v, vec2 dest);
 
 CGLM_EXPORT
 void
-glmc_mat4x2_transpose(mat4x2 m, mat2x4 dest);
+glmc_mat4x2_transpose(mat4x2 src, mat2x4 dest);
 
 CGLM_EXPORT
 void

include/cglm/mat4x2.h

@@ -11,12 +11,12 @@
    GLM_MAT4X2_ZERO
 
  Functions:
-   CGLM_INLINE void glm_mat4x2_copy(mat4x2 mat, mat4x2 dest);
-   CGLM_INLINE void glm_mat4x2_zero(mat4x2 mat);
+   CGLM_INLINE void glm_mat4x2_copy(mat4x2 src, mat4x2 dest);
+   CGLM_INLINE void glm_mat4x2_zero(mat4x2 m);
    CGLM_INLINE void glm_mat4x2_make(const float * __restrict src, mat4x2 dest);
    CGLM_INLINE void glm_mat4x2_mul(mat4x2 m1, mat2x4 m2, mat2 dest);
    CGLM_INLINE void glm_mat4x2_mulv(mat4x2 m, vec4 v, vec2 dest);
-   CGLM_INLINE void glm_mat4x2_transpose(mat4x2 m, mat2x4 dest);
+   CGLM_INLINE void glm_mat4x2_transpose(mat4x2 src, mat2x4 dest);
    CGLM_INLINE void glm_mat4x2_scale(mat4x2 m, float s);
  */
 
@@ -31,44 +31,37 @@
 #define GLM_MAT4X2_ZERO GLM_MAT4X2_ZERO_INIT
 
 /*!
- * @brief copy all members of [mat] to [dest]
+ * @brief Copy mat4x2 (src) to mat4x2 (dest).
  *
- * @param[in]  mat  source
- * @param[out] dest destination
+ * @param[in]  src  mat4x2 (left)
+ * @param[out] dest destination (result, mat4x2)
  */
 CGLM_INLINE
 void
-glm_mat4x2_copy(mat4x2 mat, mat4x2 dest) {
-  dest[0][0] = mat[0][0];
-  dest[0][1] = mat[0][1];
-
-  dest[1][0] = mat[1][0];
-  dest[1][1] = mat[1][1];
-
-  dest[2][0] = mat[2][0];
-  dest[2][1] = mat[2][1];
-
-  dest[3][0] = mat[3][0];
-  dest[3][1] = mat[3][1];
+glm_mat4x2_copy(mat4x2 src, mat4x2 dest) {
+  glm_vec2_copy(src[0], dest[0]);
+  glm_vec2_copy(src[1], dest[1]);
+  glm_vec2_copy(src[2], dest[2]);
+  glm_vec2_copy(src[3], dest[3]);
 }
 
 /*!
- * @brief make given matrix zero.
+ * @brief Zero out the mat4x2 (m).
  *
- * @param[in, out]  mat  matrix
+ * @param[in, out] mat4x2 (src, dest)
  */
 CGLM_INLINE
 void
-glm_mat4x2_zero(mat4x2 mat) {
+glm_mat4x2_zero(mat4x2 m) {
   CGLM_ALIGN_MAT mat4x2 t = GLM_MAT4X2_ZERO_INIT;
-  glm_mat4x2_copy(t, mat);
+  glm_mat4x2_copy(t, m);
 }
 
 /*!
- * @brief Create mat4x2 matrix from pointer
+ * @brief Create mat4x2 (dest) from pointer (src).
  *
- * @param[in]  src  pointer to an array of floats
- * @param[out] dest matrix
+ * @param[in]  src  pointer to an array of floats (left)
+ * @param[out] dest destination (result, mat4x2)
  */
 CGLM_INLINE
 void
@@ -87,15 +80,15 @@ glm_mat4x2_make(const float * __restrict src, mat4x2 dest) {
 }
 
 /*!
- * @brief multiply m1 and m2 to dest
+ * @brief Multiply mat4x2 (m1) by mat2x4 (m2) and store in mat2 (dest).
  *
  * @code
  * glm_mat4x2_mul(mat4x2, mat2x4, mat2);
  * @endcode
  *
- * @param[in]  m1   left matrix (mat4x2)
- * @param[in]  m2   right matrix (mat2x4)
- * @param[out] dest destination matrix (mat2)
+ * @param[in]  m1   mat4x2 (left)
+ * @param[in]  m2   mat2x4 (right)
+ * @param[out] dest destination (result, mat2)
  */
 CGLM_INLINE
 void
@@ -116,11 +109,11 @@ glm_mat4x2_mul(mat4x2 m1, mat2x4 m2, mat2 dest) {
 }
 
 /*!
- * @brief multiply matrix with column vector and store in dest column vector
+ * @brief Multiply mat4x2 (m) by vec4 (v) and store in vec2 (dest).
  *
- * @param[in]  m    matrix (left)
- * @param[in]  v    vector (right, column vector)
- * @param[out] dest result vector
+ * @param[in]  m    mat4x2 (left)
+ * @param[in]  v    vec4 (right, column vector)
+ * @param[out] dest destination (result, column vector)
  */
 CGLM_INLINE
 void
@@ -132,31 +125,23 @@ glm_mat4x2_mulv(mat4x2 m, vec4 v, vec2 dest) {
 }
 
 /*!
- * @brief transpose matrix and store in dest
+ * @brief Transpose mat4x2 (src) and store in mat2x4 (dest).
  *
- * @param[in]  m     matrix
- * @param[out] dest  result
+ * @param[in]  src  mat4x2 (left)
+ * @param[out] dest destination (result, mat2x4)
  */
 CGLM_INLINE
 void
 glm_mat4x2_transpose(mat4x2 m, mat2x4 dest) {
-  dest[0][0] = m[0][0];
-  dest[0][1] = m[1][0];
-  dest[0][2] = m[2][0];
-  dest[0][3] = m[3][0];
-  dest[1][0] = m[0][1];
-  dest[1][1] = m[1][1];
-  dest[1][2] = m[2][1];
-  dest[1][3] = m[3][1];
+  dest[0][0] = m[0][0]; dest[0][1] = m[1][0]; dest[0][2] = m[2][0]; dest[0][3] = m[3][0];
+  dest[1][0] = m[0][1]; dest[1][1] = m[1][1]; dest[1][2] = m[2][1]; dest[1][3] = m[3][1];
 }
 
 /*!
- * @brief scale (multiply with scalar) matrix
+ * @brief Multiply mat4x2 (m) by scalar constant (s).
  *
- * multiply matrix with scalar
- *
- * @param[in, out] m matrix
- * @param[in]    s scalar
+ * @param[in, out] m (src, dest)
+ * @param[in]      s float (scalar)
  */
 CGLM_INLINE
 void

include/cglm/struct/mat4x2.h

@@ -36,87 +36,85 @@
 
 
 /*!
- * @brief make given matrix zero.
+ * @brief Zero out the mat4x2s (dest).
  *
- * @param[in, out]  mat  matrix
+ * @return[out] dest constructed mat4x2s from raw pointer
  */
 CGLM_INLINE
 mat4x2s
 glms_mat4x2_(zero)(void) {
-  mat4x2s r;
-  glm_mat4x2_zero(r.raw);
-  return r;
+  mat4x2s dest;
+  glm_mat4x2_zero(dest.raw);
+  return dest;
 }
 
 /*!
- * @brief Create mat4x2 matrix from pointer
+ * @brief Create mat4x2s (dest) from pointer (src).
  *
- * @param[in]  src  pointer to an array of floats
- * @return constructed matrix from raw pointer
+ * @param[in]   src  pointer to an array of floats
+ * @return[out] dest constructed mat4x2s from raw pointer
  */
 CGLM_INLINE
 mat4x2s
 glms_mat4x2_(make)(const float * __restrict src) {
-  mat4x2s r;
-  glm_mat4x2_make(src, r.raw);
-  return r;
+  mat4x2s dest;
+  glm_mat4x2_make(src, dest.raw);
+  return dest;
 }
 
 /*!
- * @brief multiply m1 and m2 to dest
+ * @brief Multiply mat4x2s (m1) by mat2x4s (m2) and store in mat2s (dest).
  *
  * @code
  * r = glms_mat4x2_mul(mat4x2s, mat2x4s);
  * @endcode
  *
- * @param[in]  m1   left matrix (mat4x2s)
- * @param[in]  m2   right matrix (mat2x4s)
- * @returns destination matrix (mat2s)
+ * @param[in]   m1   mat4x2s (left)
+ * @param[in]   m2   mat2x4s (right)
+ * @return[out] dest constructed mat2s from raw pointers
  */
 CGLM_INLINE
 mat2s
 glms_mat4x2_(mul)(mat4x2s m1, mat2x4s m2) {
-  mat2s r;
-  glm_mat4x2_mul(m1.raw, m2.raw, r.raw);
-  return r;
+  mat2s dest;
+  glm_mat4x2_mul(m1.raw, m2.raw, dest.raw);
+  return dest;
 }
 
 /*!
- * @brief multiply matrix with column vector and store in dest column vector
+ * @brief Multiply mat4x2s (m) by vec4s (v) and store in vec2s (dest).
  *
- * @param[in]  m    matrix (left)
- * @param[in]  v    vector (right, column vector)
- * @returns destination vector (vec2s)
+ * @param[in]   m    mat4x2s (left)
+ * @param[in]   v    vec4s (right, column vector)
+ * @return[out] dest constructed vec2s from raw pointers
  */
 CGLM_INLINE
 vec2s
 glms_mat4x2_(mulv)(mat4x2s m, vec4s v) {
-  vec2s r;
-  glm_mat4x2_mulv(m.raw, v.raw, r.raw);
-  return r;
+  vec2s dest;
+  glm_mat4x2_mulv(m.raw, v.raw, dest.raw);
+  return dest;
 }
 
 /*!
- * @brief transpose matrix and store in dest
+ * @brief Transpose mat4x2s (m) and store in mat2x4s (dest).
  *
- * @param[in]  m     matrix
- * @param[out] dest  result
+ * @param[in]   m    mat4x2s (left)
+ * @return[out] dest constructed mat2x4s from raw pointers
  */
 CGLM_INLINE
 mat2x4s
 glms_mat4x2_(transpose)(mat4x2s m) {
-  mat2x4s r;
-  glm_mat4x2_transpose(m.raw, r.raw);
-  return r;
+  mat2x4s dest;
+  glm_mat4x2_transpose(m.raw, dest.raw);
+  return dest;
 }
 
 /*!
- * @brief scale (multiply with scalar) matrix
+ * @brief Multiply mat4x2s (m) by scalar constant (s).
  *
- * multiply matrix with scalar
- *
- * @param[in, out] m matrix
- * @param[in]    s scalar
+ * @param[in, out] m mat4x2s (src, dest)
+ * @param[in]      s float (scalar)
  */
 CGLM_INLINE
 mat4x2s