mark readonly parameters as const

This commit is contained in:
Recep Aslantas
2019-04-28 12:15:43 +03:00
parent 85ca81ce79
commit 6ed275734b
24 changed files with 237 additions and 234 deletions

View File

@@ -100,7 +100,7 @@
*/
CGLM_INLINE
void
glm_mat4_ucopy(mat4 mat, mat4 dest) {
glm_mat4_ucopy(const mat4 mat, mat4 dest) {
dest[0][0] = mat[0][0]; dest[1][0] = mat[1][0];
dest[0][1] = mat[0][1]; dest[1][1] = mat[1][1];
dest[0][2] = mat[0][2]; dest[1][2] = mat[1][2];
@@ -120,7 +120,7 @@ glm_mat4_ucopy(mat4 mat, mat4 dest) {
*/
CGLM_INLINE
void
glm_mat4_copy(mat4 mat, mat4 dest) {
glm_mat4_copy(const mat4 mat, mat4 dest) {
#ifdef __AVX__
glmm_store256(dest[0], glmm_load256(mat[0]));
glmm_store256(dest[2], glmm_load256(mat[2]));
@@ -170,7 +170,7 @@ glm_mat4_identity(mat4 mat) {
*/
CGLM_INLINE
void
glm_mat4_identity_array(mat4 * __restrict mat, size_t count) {
glm_mat4_identity_array(mat4 * const __restrict mat, size_t count) {
CGLM_ALIGN_MAT mat4 t = GLM_MAT4_IDENTITY_INIT;
size_t i;
@@ -199,7 +199,7 @@ glm_mat4_zero(mat4 mat) {
*/
CGLM_INLINE
void
glm_mat4_pick3(mat4 mat, mat3 dest) {
glm_mat4_pick3(const mat4 mat, mat3 dest) {
dest[0][0] = mat[0][0];
dest[0][1] = mat[0][1];
dest[0][2] = mat[0][2];
@@ -223,7 +223,7 @@ glm_mat4_pick3(mat4 mat, mat3 dest) {
*/
CGLM_INLINE
void
glm_mat4_pick3t(mat4 mat, mat3 dest) {
glm_mat4_pick3t(const mat4 mat, mat3 dest) {
dest[0][0] = mat[0][0];
dest[0][1] = mat[1][0];
dest[0][2] = mat[2][0];
@@ -245,7 +245,7 @@ glm_mat4_pick3t(mat4 mat, mat3 dest) {
*/
CGLM_INLINE
void
glm_mat4_ins3(mat3 mat, mat4 dest) {
glm_mat4_ins3(const mat3 mat, mat4 dest) {
dest[0][0] = mat[0][0];
dest[0][1] = mat[0][1];
dest[0][2] = mat[0][2];
@@ -275,7 +275,7 @@ glm_mat4_ins3(mat3 mat, mat4 dest) {
*/
CGLM_INLINE
void
glm_mat4_mul(mat4 m1, mat4 m2, mat4 dest) {
glm_mat4_mul(const mat4 m1, const mat4 m2, mat4 dest) {
#ifdef __AVX__
glm_mat4_mul_avx(m1, m2, dest);
#elif defined( __SSE__ ) || defined( __SSE2__ )
@@ -333,7 +333,7 @@ glm_mat4_mul(mat4 m1, mat4 m2, mat4 dest) {
*/
CGLM_INLINE
void
glm_mat4_mulN(mat4 * __restrict matrices[], uint32_t len, mat4 dest) {
glm_mat4_mulN(mat4 * const __restrict matrices[], uint32_t len, mat4 dest) {
uint32_t i;
#ifdef DEBUG
@@ -355,7 +355,7 @@ glm_mat4_mulN(mat4 * __restrict matrices[], uint32_t len, mat4 dest) {
*/
CGLM_INLINE
void
glm_mat4_mulv(mat4 m, vec4 v, vec4 dest) {
glm_mat4_mulv(const mat4 m, const vec4 v, vec4 dest) {
#if defined( __SSE__ ) || defined( __SSE2__ )
glm_mat4_mulv_sse2(m, v, dest);
#else
@@ -377,7 +377,7 @@ glm_mat4_mulv(mat4 m, vec4 v, vec4 dest) {
*/
CGLM_INLINE
float
glm_mat4_trace(mat4 m) {
glm_mat4_trace(const mat4 m) {
return m[0][0] + m[1][1] + m[2][2] + m[3][3];
}
@@ -390,7 +390,7 @@ glm_mat4_trace(mat4 m) {
*/
CGLM_INLINE
float
glm_mat4_trace3(mat4 m) {
glm_mat4_trace3(const mat4 m) {
return m[0][0] + m[1][1] + m[2][2];
}
@@ -402,7 +402,7 @@ glm_mat4_trace3(mat4 m) {
*/
CGLM_INLINE
void
glm_mat4_quat(mat4 m, versor dest) {
glm_mat4_quat(const mat4 m, versor dest) {
float trace, r, rinv;
/* it seems using like m12 instead of m[1][2] causes extra instructions */
@@ -453,7 +453,7 @@ glm_mat4_quat(mat4 m, versor dest) {
*/
CGLM_INLINE
void
glm_mat4_mulv3(mat4 m, vec3 v, float last, vec3 dest) {
glm_mat4_mulv3(const mat4 m, const vec3 v, float last, vec3 dest) {
vec4 res;
glm_vec4(v, last, res);
glm_mat4_mulv(m, res, res);
@@ -470,7 +470,7 @@ glm_mat4_mulv3(mat4 m, vec3 v, float last, vec3 dest) {
*/
CGLM_INLINE
void
glm_mat4_transpose_to(mat4 m, mat4 dest) {
glm_mat4_transpose_to(const mat4 m, mat4 dest) {
#if defined( __SSE__ ) || defined( __SSE2__ )
glm_mat4_transp_sse2(m, dest);
#else
@@ -553,7 +553,7 @@ glm_mat4_scale(mat4 m, float s) {
*/
CGLM_INLINE
float
glm_mat4_det(mat4 mat) {
glm_mat4_det(const mat4 mat) {
#if defined( __SSE__ ) || defined( __SSE2__ )
return glm_mat4_det_sse2(mat);
#else
@@ -586,7 +586,7 @@ glm_mat4_det(mat4 mat) {
*/
CGLM_INLINE
void
glm_mat4_inv(mat4 mat, mat4 dest) {
glm_mat4_inv(const mat4 mat, mat4 dest) {
#if defined( __SSE__ ) || defined( __SSE2__ )
glm_mat4_inv_sse2(mat, dest);
#else
@@ -647,7 +647,7 @@ glm_mat4_inv(mat4 mat, mat4 dest) {
*/
CGLM_INLINE
void
glm_mat4_inv_fast(mat4 mat, mat4 dest) {
glm_mat4_inv_fast(const mat4 mat, mat4 dest) {
#if defined( __SSE__ ) || defined( __SSE2__ )
glm_mat4_inv_fast_sse2(mat, dest);
#else
@@ -680,7 +680,7 @@ glm_mat4_swap_col(mat4 mat, int col1, int col2) {
*/
CGLM_INLINE
void
glm_mat4_swap_row(mat4 mat, int row1, int row2) {
glm_mat4_swap_row(mat4 mat, const int row1, const int row2) {
CGLM_ALIGN(16) vec4 tmp;
tmp[0] = mat[0][row1];
tmp[1] = mat[1][row1];
@@ -714,7 +714,7 @@ glm_mat4_swap_row(mat4 mat, int row1, int row2) {
*/
CGLM_INLINE
float
glm_mat4_rmc(vec4 r, mat4 m, vec4 c) {
glm_mat4_rmc(const vec4 r, const mat4 m, const vec4 c) {
vec4 tmp;
glm_mat4_mulv(m, c, tmp);
return glm_vec4_dot(r, tmp);