Page MenuHome
Differential D12137 Diff 40352 source/blender/blenkernel/intern/mesh.c

Changeset View

Standalone View

View Options

source/blender/blenkernel/intern/mesh.c

/* /*
* This program is free software; you can redistribute it and/or * This program is free software; you can redistribute it and/or
* modify it under the terms of the GNU General Public License * modify it under the terms of the GNU General Public License
* as published by the Free Software Foundation; either version 2 * as published by the Free Software Foundation; either version 2
* of the License, or (at your option) any later version. * of the License, or (at your option) any later version.
* *
* This program is distributed in the hope that it will be useful, * This program is distributed in the hope that it will be useful,
* but WITHOUT ANY WARRANTY; without even the implied warranty of * but WITHOUT ANY WARRANTY; without even the implied warranty of
* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
▲ Show 20 Lines • Show All 374 Lines • ▼ Show 20 Linesenum {
MESHCMP_LOOPCOLMISMATCH, MESHCMP_LOOPCOLMISMATCH,
MESHCMP_LOOPUVMISMATCH, MESHCMP_LOOPUVMISMATCH,
MESHCMP_LOOPMISMATCH, MESHCMP_LOOPMISMATCH,
MESHCMP_POLYVERTMISMATCH, MESHCMP_POLYVERTMISMATCH,
MESHCMP_POLYMISMATCH, MESHCMP_POLYMISMATCH,
MESHCMP_EDGEUNKNOWN, MESHCMP_EDGEUNKNOWN,
MESHCMP_VERTCOMISMATCH, MESHCMP_VERTCOMISMATCH,
MESHCMP_CDLAYERS_MISMATCH, MESHCMP_CDLAYERS_MISMATCH,
MESHCMP_ATTR_VAL_MISMATCH,
JacquesLuckeUnsubmitted
Done
Inline Actions

There is no need to abbreviate the words here.

JacquesLucke: There is no need to abbreviate the words here.
}; };
static const char *cmpcode_to_str(int code) static const char *cmpcode_to_str(int code)
{ {
switch (code) { switch (code) {
case MESHCMP_DVERT_WEIGHTMISMATCH: case MESHCMP_DVERT_WEIGHTMISMATCH:
return "Vertex Weight Mismatch"; return "Vertex Weight Mismatch";
case MESHCMP_DVERT_GROUPMISMATCH: case MESHCMP_DVERT_GROUPMISMATCH:
Show All 11 Lines switch (code) {
case MESHCMP_POLYMISMATCH: case MESHCMP_POLYMISMATCH:
return "Loop Vert Mismatch"; return "Loop Vert Mismatch";
case MESHCMP_EDGEUNKNOWN: case MESHCMP_EDGEUNKNOWN:
return "Edge Mismatch"; return "Edge Mismatch";
case MESHCMP_VERTCOMISMATCH: case MESHCMP_VERTCOMISMATCH:
return "Vertex Coordinate Mismatch"; return "Vertex Coordinate Mismatch";
case MESHCMP_CDLAYERS_MISMATCH: case MESHCMP_CDLAYERS_MISMATCH:
return "CustomData Layer Count Mismatch"; return "CustomData Layer Count Mismatch";
case MESHCMP_ATTR_VAL_MISMATCH:
return "Attribute Value Mismatch";
default: default:
return "Mesh Comparison Code Unknown"; return "Mesh Comparison Code Unknown";
} }
} }
/** Thresh is threshold for comparing vertices, UV's, vertex colors, weights, etc. */ /** Thresh is threshold for comparing vertices, UV's, vertex colors, weights, etc. */
static int customdata_compare( static int customdata_compare(
CustomData *c1, CustomData *c2, Mesh *m1, Mesh *m2, const float thresh) CustomData *c1, CustomData *c2, const int total_length, Mesh *m1, Mesh *m2, const float thresh)
{ {
const float thresh_sq = thresh * thresh; const float thresh_sq = thresh * thresh;
CustomDataLayer *l1, *l2; CustomDataLayer *l1, *l2;
int i, i1 = 0, i2 = 0, tot, j; int i, i1 = 0, i2 = 0, tot, j;
for (i = 0; i < c1->totlayer; i++) { for (i = 0; i < c1->totlayer; i++) {
if (ELEM(c1->layers[i].type, if (ELEM(c1->layers[i].type,
CD_MVERT, CD_MVERT,
Show All 19 Linesstatic int customdata_compare(
} }
if (i1 != i2) { if (i1 != i2) {
return MESHCMP_CDLAYERS_MISMATCH; return MESHCMP_CDLAYERS_MISMATCH;
} }
l1 = c1->layers; l1 = c1->layers;
l2 = c2->layers; l2 = c2->layers;
for (i1 = 0; i1 < c1->totlayer; i1++) {
l1 = c1->layers + i1;
if ((CD_TYPE_AS_MASK(l1->type) & CD_MASK_PROP_ALL) == 0) {
/* Skip non generic attribute layers. */
continue;
}
bool found_corresponding_layer = false;
for (i2 = 0; i2 < c2->totlayer; i2++) {
l2 = c2->layers + i2;
if (l1->type != l2->type || !STREQ(l1->name, l2->name)) {
continue;
}
found_corresponding_layer = true;
/* At this point `l1` and `l2` have the same name and type, so they should be compared. */
switch (l1->type) {
case CD_PROP_FLOAT: {
JacquesLuckeUnsubmitted
Done
Inline Actions

Missing break, some in the cases below.

JacquesLucke: Missing `break`, some in the cases below.
const float *l1_data = l1->data;
const float *l2_data = l2->data;
for (int i = 0; i < total_length; i++) {
JacquesLuckeUnsubmitted
Done
Inline Actions

This i shadows a previous declaration. You can remove the function-wide declaration of i and declare it in the for loops where it is used.
The same scope reduction should be done for l1 and l2 at some point, but that can be done later.

JacquesLucke: This `i` shadows a previous declaration. You can remove the function-wide declaration of `i`…
if (fabs(l1_data[i] - l2_data[i]) > thresh) {
return MESHCMP_ATTR_VAL_MISMATCH;
}
}
}
case CD_PROP_FLOAT2: {
const float(*l1_data)[2] = l1->data;
const float(*l2_data)[2] = l2->data;
for (int i = 0; i < total_length; i++) {
JacquesLuckeUnsubmitted
Done
Inline Actions

Remove unnecessary newlines.

JacquesLucke: Remove unnecessary newlines.
if (len_squared_v2v2(l1_data, l2_data) > thresh_sq) {
JacquesLuckeUnsubmitted
Done
Inline Actions

This is missing [i], same below.

JacquesLucke: This is missing `[i]`, same below.
return MESHCMP_ATTR_VAL_MISMATCH;
}
}
}
case CD_PROP_FLOAT3: {
const float(*l1_data)[3] = l1->data;
const float(*l2_data)[3] = l2->data;
JacquesLuckeUnsubmitted
Done
Inline Actions

Remove unnecessary newlines between individual cases.

JacquesLucke: Remove unnecessary newlines between individual cases.
for (i = 0; i < total_length; i++) {
JacquesLuckeUnsubmitted
Done
Inline Actions

Use clang format.

JacquesLucke: Use clang format.
if (len_squared_v3v3(l1_data, l2_data) > thresh_sq) {
return MESHCMP_ATTR_VAL_MISMATCH;
}
}
}
default: {
int element_size = CustomData_sizeof(l1->type);
for (i = 0; i < total_length; i++) {
if (!CustomData_data_equals(l1->type,
POINTER_OFFSET(l1->data, element_size + i),
POINTER_OFFSET(l2->data, element_size + i))) {
return MESHCMP_ATTR_VAL_MISMATCH;
}
}
}
}
}
if (!found_corresponding_layer) {
return MESHCMP_CDLAYERS_MISMATCH;
}
}
l1 = c1->layers;
l2 = c2->layers;
tot = i1; tot = i1;
i1 = 0; i1 = 0;
i2 = 0; i2 = 0;
for (i = 0; i < tot; i++) { for (i = 0; i < tot; i++) {
while ( while (
i1 < c1->totlayer && i1 < c1->totlayer &&
!ELEM(l1->type, CD_MVERT, CD_MEDGE, CD_MPOLY, CD_MLOOPUV, CD_MLOOPCOL, CD_MDEFORMVERT)) { !ELEM(l1->type, CD_MVERT, CD_MEDGE, CD_MPOLY, CD_MLOOPUV, CD_MLOOPCOL, CD_MDEFORMVERT)) {
i1++; i1++;
▲ Show 20 Lines • Show All 151 Lines • ▼ Show 20 Linesconst char *BKE_mesh_cmp(Mesh *me1, Mesh *me2, float thresh)
if (me1->totpoly != me2->totpoly) { if (me1->totpoly != me2->totpoly) {
return "Number of faces don't match"; return "Number of faces don't match";
} }
if (me1->totloop != me2->totloop) { if (me1->totloop != me2->totloop) {
return "Number of loops don't match"; return "Number of loops don't match";
} }
if ((c = customdata_compare(&me1->vdata, &me2->vdata, me1, me2, thresh))) { if ((c = customdata_compare(&me1->vdata, &me2->vdata, me1->totvert, me1, me2, thresh))) {
return cmpcode_to_str(c); return cmpcode_to_str(c);
} }
if ((c = customdata_compare(&me1->edata, &me2->edata, me1, me2, thresh))) { if ((c = customdata_compare(&me1->edata, &me2->edata, me1->totedge, me1, me2, thresh))) {
return cmpcode_to_str(c); return cmpcode_to_str(c);
} }
if ((c = customdata_compare(&me1->ldata, &me2->ldata, me1, me2, thresh))) { if ((c = customdata_compare(&me1->ldata, &me2->ldata, me1->totloop, me1, me2, thresh))) {
return cmpcode_to_str(c); return cmpcode_to_str(c);
} }
if ((c = customdata_compare(&me1->pdata, &me2->pdata, me1, me2, thresh))) { if ((c = customdata_compare(&me1->pdata, &me2->pdata, me1->totpoly, me1, me2, thresh))) {
return cmpcode_to_str(c); return cmpcode_to_str(c);
} }
return NULL; return NULL;
} }
static void mesh_ensure_tessellation_customdata(Mesh *me) static void mesh_ensure_tessellation_customdata(Mesh *me)
{ {
▲ Show 20 Lines • Show All 1,533 Lines • Show Last 20 Lines