Page MenuHome
Differential D10239 Diff 33550 source/blender/editors/sculpt_paint/paint_vertex_weight_ops.c

Changeset View

Standalone View

View Options

source/blender/editors/sculpt_paint/paint_vertex_weight_ops.c

Show First 20 Lines • Show All 564 Lines • ▼ Show 20 Lines
/* *** VGroups Gradient *** */ /* *** VGroups Gradient *** */
typedef struct WPGradient_vertStore { typedef struct WPGradient_vertStore {
float sco[2]; float sco[2];
float weight_orig; float weight_orig;
enum { enum {
VGRAD_STORE_NOP = 0, VGRAD_STORE_NOP = 0,
VGRAD_STORE_DW_EXIST = (1 << 0), VGRAD_STORE_DW_EXIST = (1 << 0),
VGRAD_STORE_IS_MODIFIED = (1 << 1)
} flag; } flag;
} WPGradient_vertStore; } WPGradient_vertStore;
typedef struct WPGradient_vertStoreBase { typedef struct WPGradient_vertStoreBase {
struct WPaintPrev wpp; struct WPaintPrev wpp;
WPGradient_vertStore elem[0]; WPGradient_vertStore elem[0];
} WPGradient_vertStoreBase; } WPGradient_vertStoreBase;
Show All 18 Linestypedef struct WPGradient_userData {
float weightpaint; float weightpaint;
} WPGradient_userData; } WPGradient_userData;
static void gradientVert_update(WPGradient_userData *grad_data, int index) static void gradientVert_update(WPGradient_userData *grad_data, int index)
{ {
Mesh *me = grad_data->me; Mesh *me = grad_data->me;
WPGradient_vertStore *vs = &grad_data->vert_cache->elem[index]; WPGradient_vertStore *vs = &grad_data->vert_cache->elem[index];
vs->flag &= ~VGRAD_STORE_IS_MODIFIED;
campbellbartonUnsubmitted
Not Done
Inline Actions

This can be moved under the VGRAD_STORE_DW_EXIST check below. Since that situation won't ever cause the vertices to be painted onto.

campbellbarton: This can be moved under the `VGRAD_STORE_DW_EXIST` check below. Since that situation won't ever…
/* Optionally restrict to assigned verices only. */ /* Optionally restrict to assigned verices only. */
if (grad_data->use_vgroup_restrict && ((vs->flag & VGRAD_STORE_DW_EXIST) == 0)) { if (grad_data->use_vgroup_restrict && ((vs->flag & VGRAD_STORE_DW_EXIST) == 0)) {
return; return;
} }
float alpha; float alpha;
if (grad_data->type == WPAINT_GRADIENT_TYPE_LINEAR) { if (grad_data->type == WPAINT_GRADIENT_TYPE_LINEAR) {
alpha = line_point_factor_v2(vs->sco, grad_data->sco_start, grad_data->sco_end); alpha = line_point_factor_v2(vs->sco, grad_data->sco_start, grad_data->sco_end);
Show All 30 Lines else {
else { else {
/* wasn't originally existing, remove */ /* wasn't originally existing, remove */
MDeformWeight *dw = BKE_defvert_find_index(dv, grad_data->def_nr); MDeformWeight *dw = BKE_defvert_find_index(dv, grad_data->def_nr);
if (dw) { if (dw) {
BKE_defvert_remove_group(dv, dw); BKE_defvert_remove_group(dv, dw);
} }
} }
} }
vs->flag |= VGRAD_STORE_IS_MODIFIED;
campbellbartonUnsubmitted
Done
Inline Actions

This should be moved into the alpha != 0.0 block above.

campbellbarton: This should be moved into the `alpha != 0.0` block above.
} }
static void gradientVertUpdate__mapFunc(void *userData, static void gradientVertUpdate__mapFunc(void *userData,
int index, int index,
const float UNUSED(co[3]), const float UNUSED(co[3]),
const float UNUSED(no_f[3]), const float UNUSED(no_f[3]),
const short UNUSED(no_s[3])) const short UNUSED(no_s[3]))
{ {
Show All 38 Lines if (ED_view3d_project_float_object(
return; return;
} }
MDeformVert *dv = &me->dvert[index]; MDeformVert *dv = &me->dvert[index];
const MDeformWeight *dw = BKE_defvert_find_index(dv, grad_data->def_nr); const MDeformWeight *dw = BKE_defvert_find_index(dv, grad_data->def_nr);
if (dw) { if (dw) {
vs->weight_orig = dw->weight; vs->weight_orig = dw->weight;
vs->flag = VGRAD_STORE_DW_EXIST; vs->flag = VGRAD_STORE_DW_EXIST;
} }
else { else {
vs->weight_orig = 0.0f; vs->weight_orig = 0.0f;
vs->flag = VGRAD_STORE_NOP; vs->flag = VGRAD_STORE_NOP;
} }
BLI_BITMAP_ENABLE(grad_data->vert_visit, index); BLI_BITMAP_ENABLE(grad_data->vert_visit, index);
gradientVert_update(grad_data, index); gradientVert_update(grad_data, index);
} }
▲ Show 20 Lines • Show All 42 Lines • ▼ Show 20 Linesstatic int paint_weight_gradient_exec(bContext *C, wmOperator *op)
WPGradient_vertStoreBase *vert_cache; WPGradient_vertStoreBase *vert_cache;
struct ARegion *region = CTX_wm_region(C); struct ARegion *region = CTX_wm_region(C);
Scene *scene = CTX_data_scene(C); Scene *scene = CTX_data_scene(C);
Object *ob = CTX_data_active_object(C); Object *ob = CTX_data_active_object(C);
Mesh *me = ob->data; Mesh *me = ob->data;
int x_start = RNA_int_get(op->ptr, "xstart"); int x_start = RNA_int_get(op->ptr, "xstart");
int y_start = RNA_int_get(op->ptr, "ystart"); int y_start = RNA_int_get(op->ptr, "ystart");
int x_end = RNA_int_get(op->ptr, "xend"); int x_end = RNA_int_get(op->ptr, "xend");
int y_end = RNA_int_get(op->ptr, "yend"); int y_end = RNA_int_get(op->ptr, "yend");
ankitmUnsubmitted
Done
Inline Actions

Probably a mistake to add an empty line

ankitm: Probably a mistake to add an empty line
const float sco_start[2] = {x_start, y_start}; const float sco_start[2] = {x_start, y_start};
const float sco_end[2] = {x_end, y_end}; const float sco_end[2] = {x_end, y_end};
const bool is_interactive = (gesture != NULL); const bool is_interactive = (gesture != NULL);
Depsgraph *depsgraph = CTX_data_ensure_evaluated_depsgraph(C); Depsgraph *depsgraph = CTX_data_ensure_evaluated_depsgraph(C);
WPGradient_userData data = {NULL}; WPGradient_userData data = {NULL};
▲ Show 20 Lines • Show All 69 Lines • ▼ Show 20 Lines if (data.is_init) {
data.vert_visit = NULL; data.vert_visit = NULL;
} }
else { else {
BKE_mesh_foreach_mapped_vert(me_eval, gradientVertUpdate__mapFunc, &data, MESH_FOREACH_NOP); BKE_mesh_foreach_mapped_vert(me_eval, gradientVertUpdate__mapFunc, &data, MESH_FOREACH_NOP);
} }
DEG_id_tag_update(&ob->id, ID_RECALC_GEOMETRY); DEG_id_tag_update(&ob->id, ID_RECALC_GEOMETRY);
WM_event_add_notifier(C, NC_OBJECT | ND_DRAW, ob); WM_event_add_notifier(C, NC_OBJECT | ND_DRAW, ob);
ankitmUnsubmitted
Done
Inline Actions

Remove "fix". Comment to concisely describe what the code below does is fine.

ankitm: Remove "fix". Comment to concisely describe what the code below does is fine.
if (is_interactive == false) { if (is_interactive == false) {
MEM_freeN(vert_cache); MEM_freeN(vert_cache);
} }
const int vgroup_tot = BLI_listbase_count(&ob->defbase);
ankitmUnsubmitted
Done
Inline Actions

Don't declare uninitialized variables.

ankitm: Don't declare uninitialized variables.
const int def_nr_lock = ob->actdef - 1;
const bool is_normalize = scene->toolsettings->auto_normalize;
const bool *vgroup_validmap = BKE_object_defgroup_validmap_get(ob, vgroup_tot);
campbellbartonUnsubmitted
Done
Inline Actions

This needs to be freed.

campbellbarton: This needs to be freed.
MDeformVert *dvert = me->dvert;
ankitmUnsubmitted
Done
Inline Actions

const bool is_normalize

ankitm: `const bool is_normalize`
//data.vert_visit = BLI_BITMAP_NEW(me->totvert, __func__);
campbellbartonUnsubmitted
Done
Inline Actions

This should be initialized from BKE_object_defgroup_validmap_get.

campbellbarton: This should be initialized from `BKE_object_defgroup_validmap_get`.
campbellbartonUnsubmitted
Not Done
Inline Actions

My mistake, that functions checks bones, this should be BKE_object_defgroup_lock_flags_get - to respect the groups lock icon.

campbellbarton: My mistake, that functions checks bones, this should be `BKE_object_defgroup_lock_flags_get`…
//BKE_mesh_foreach_mapped_vert(me_eval, gradientVertInit__mapFunc, &data, MESH_FOREACH_NOP);
if (is_normalize) {
for (int i = 0; i < me->totvert; i++) {
MDeformVert *dv = (dvert + i);
if (data.vert_cache->elem[i].flag & VGRAD_STORE_IS_MODIFIED) {
BKE_defvert_normalize_lock_single(dv, vgroup_validmap, vgroup_tot, def_nr_lock);
}
}
}
ankitmUnsubmitted
Done
Inline Actions

4 empty lines seem overdone here

ankitm: 4 empty lines seem overdone here
campbellbartonUnsubmitted
Done
Inline Actions

This entire block can be in an if (is_normalized) {...} check.

campbellbarton: This entire block can be in an `if (is_normalized) {...}` check.
// MEM_freeN(data.vert_visit);
campbellbartonUnsubmitted
Done
Inline Actions

This is unnecessary data.vert_visit is already created and freed above (line 840 without this patch applied).

campbellbarton: This is unnecessary `data.vert_visit` is already created and freed above (line 840 without this…
//data.vert_visit = NULL;
campbellbartonUnsubmitted
Done
Inline Actions

vc is only used to access scene use: Scene *scene = CTX_data_scene(C); instead.

campbellbarton: `vc` is only used to access `scene` use: `Scene *scene = CTX_data_scene(C);` instead.
return OPERATOR_FINISHED; return OPERATOR_FINISHED;
} }
static int paint_weight_gradient_invoke(bContext *C, wmOperator *op, const wmEvent *event) static int paint_weight_gradient_invoke(bContext *C, wmOperator *op, const wmEvent *event)
{ {
int ret; int ret;
if (ED_wpaint_ensure_data(C, op->reports, 0, NULL) == false) { if (ED_wpaint_ensure_data(C, op->reports, 0, NULL) == false) {
▲ Show 20 Lines • Show All 50 Lines • Show Last 20 Lines