author Campbell Barton Tue, 7 May 2019 23:20:12 +0000 (09:20 +1000) committer Campbell Barton Tue, 7 May 2019 23:33:06 +0000 (09:33 +1000)

index ef791b3..6ac1ac7 100644 (file)
@@ -1018,13 +1018,16 @@ bool invert_m4(float m)
return success;
}

-/* computes the inverse of mat and puts it in inverse.  Returns
- * true on success (i.e. can always find a pivot) and false on failure.
+/**
+ * Computes the inverse of mat and puts it in inverse.
* Uses Gaussian Elimination with partial (maximal column) pivoting.
- * Mark Segal - 1992
- * note this is less performant than EIG_invert_m4_m4 (Eigen), but e.g.
+ * \return true on success (i.e. can always find a pivot) and false on failure.
+ * Mark Segal - 1992.
+ *
+ * \note this is less performant than #EIG_invert_m4_m4 (Eigen), but e.g.
* for non-invertible scale matrices, findinging a partial solution can
- * be useful to have a valid local transform center, see T57767 */
+ * be useful to have a valid local transform center, see T57767.
+ */
bool invert_m4_m4_fallback(float inverse, const float mat)
{
if (EIG_invert_m4_m4(inverse, mat)) {
@@ -1070,7 +1073,7 @@ bool invert_m4_m4_fallback(float inverse, const float mat)
}

if (UNLIKELY(tempmat[i][i] == 0.0f)) {
-      return false;  /* No non-zero pivot */
+      return false; /* No non-zero pivot */
}
temp = (double)tempmat[i][i];
for (k = 0; k < 4; k++) {
@@ -1720,9 +1723,11 @@ void mat4_to_size(float size, const float mat)
size = len_v3(mat);
}

-/* This computes the overall volume scale factor of a transformation matrix.
+/**
+ * This computes the overall volume scale factor of a transformation matrix.
* For an orthogonal matrix, it is the product of all three scale values.
- * Returns a negative value if the transform is flipped by negative scale. */
+ * Returns a negative value if the transform is flipped by negative scale.
+ */
float mat3_to_volume_scale(const float mat)
{
return determinant_m3_array(mat);
@@ -1733,9 +1738,11 @@ float mat4_to_volume_scale(const float mat)
return determinant_m4_mat3_array(mat);
}

-/* this gets the average scale of a matrix, only use when your scaling
+/**
+ * This gets the average scale of a matrix, only use when your scaling
* data that has no idea of scale axis, examples are bone-envelope-radius
- * and curve radius */
+ */
float mat3_to_scale(const float mat)
{
/* unit length vector */
@@ -2095,10 +2102,12 @@ bool equals_m4m4(const float mat1, const float mat2)
equals_v4v4(mat1, mat2) && equals_v4v4(mat1, mat2));
}

-/* make a 4x4 matrix out of 3 transform components */
-/* matrices are made in the order: scale * rot * loc */
-/* TODO: need to have a version that allows for rotation order... */
-
+/**
+ * Make a 4x4 matrix out of 3 transform components.
+ * Matrices are made in the order: `scale * rot * loc`
+ *
+ * TODO: need to have a version that allows for rotation order...
+ */
void loc_eul_size_to_mat4(float mat,
const float loc,
const float eul,
@@ -2123,9 +2132,10 @@ void loc_eul_size_to_mat4(float mat,
mat = loc;
}

-/* make a 4x4 matrix out of 3 transform components */
-
-/* matrices are made in the order: scale * rot * loc */
+/**
+ * Make a 4x4 matrix out of 3 transform components.
+ * Matrices are made in the order: `scale * rot * loc`
+ */
void loc_eulO_size_to_mat4(float mat,
const float loc,
const float eul,
@@ -2151,9 +2161,10 @@ void loc_eulO_size_to_mat4(float mat,
mat = loc;
}

-/* make a 4x4 matrix out of 3 transform components */
-
-/* matrices are made in the order: scale * rot * loc */
+/**
+ * Make a 4x4 matrix out of 3 transform components.
+ * Matrices are made in the order: `scale * rot * loc`
+ */
void loc_quat_size_to_mat4(float mat,
const float loc,
const float quat,
index 9dae425..21a5dd3 100644 (file)
@@ -8954,7 +8954,6 @@ void MESH_OT_smoothen_normals(struct wmOperatorType *ot)

static int edbm_mod_weighted_strength_exec(bContext *C, wmOperator *op)
{
-  Scene *scene = CTX_data_scene(C);
Object *obedit = CTX_data_edit_object(C);
BMEditMesh *em = BKE_editmesh_from_object(obedit);
BMesh *bm = em->bm;