Differential D10239 Diff 33290 source/blender/blenkernel/intern/deform.c

Changeset View

Standalone View

source/blender/blenkernel/intern/deform.c

Context not available.
	}	}
	}	}

		void BKE_devert_normalize_gradient_paint(MDeformVert *dv,
		campbellbartonUnsubmitted Done Inline Actions Regarding naming `BKE_devert_normalize_gradient_paint` isn't very informative, as it doesn't let developers know whats different. Also, if this is used in other situations, the name is likely to become meaningless. campbellbarton: Regarding naming `BKE_devert_normalize_gradient_paint` isn't very informative, as it doesn't…
		MDeformWeight *dw,
		campbellbartonUnsubmitted Done Inline Actions No reason to pass both `dv` and `dw`, as you can access both with `dv`. campbellbarton: No reason to pass both `dv` and `dw`, as you can access both with `dv`.
		const int vgroup_tot,
		ankitmUnsubmitted Not Done Inline Actions Follow the naming style same as other variables in this module. `tot` is generally a prefix. ankitm: Follow the naming style same as other variables in this module. `tot` is generally a prefix.
		const int def_nr_lock)
		ankitmUnsubmitted Done Inline Actions def_nr_lock can be const too ankitm: def_nr_lock can be const too
		{
		if (dv->totweight == 0) {
		/do nothing/
		ankitmUnsubmitted Not Done Inline Actions Might want to early return here ? ankitm: Might want to early return here ?
		}

		if (dv->totweight == 1) {
		if (dw->def_nr < vgroup_tot && def_nr_lock != dw->def_nr) {
		dw->weight = 1;
		}
		ankitmUnsubmitted Done Inline Actions Early return here, and then remove the `else` (and the scope & indent it created). ankitm: Early return here, and then remove the `else` (and the scope & indent it created).
		}
		else {
		MDeformWeight *dw_lock = NULL;
		unsigned int i;
		ankitmUnsubmitted Done Inline Actions Keep the scope of `i` and other variables to the minimum. https://isocpp.github.io/CppCoreGuidelines/CppCoreGuidelines#es22-dont-declare-a-variable-until-you-have-a-value-to-initialize-it-with ankitm: Keep the scope of `i` and other variables to the minimum. https://isocpp.github.
		float tot_weight = 0.0f;
		float lock_iweight = 1.0f;
		ankitmUnsubmitted Done Inline Actions Is the first `i` in `lock_iweight` a typo? I'm not familiar with the code here, so ignore if I'm wrong here. ankitm: Is the first `i` in `lock_iweight` a typo? I'm not familiar with the code here, so ignore if…
		PratikPB2123AuthorUnsubmitted Done Inline Actions I keep code block related to calculation of vertex weight similar to other function (eg. see BKE_defvert_normalize_lock_single ,BKE_defvert_normalize_lock_map) . `lock_weight` or `lock_iweight` both are understandable . PratikPB2123: I keep code block related to calculation of vertex weight similar to other function (eg. see…

		for (i = dv->totweight, dw = dv->dw; i != 0; i--, dw++) {
		if ((dw->def_nr < vgroup_tot)) {
		if (dw->def_nr != def_nr_lock) {
		tot_weight += dw->weight;
		}
		else {
		dw_lock = dw;
		lock_iweight = (1.0f - dw_lock->weight);
		CLAMP(lock_iweight, 0.0f, 1.0f);
		}
		}
		}

		if (tot_weight > 0.0f) {
		/* Paranoid, should be 1.0 but in case of float error clamp anyway */
		ankitmUnsubmitted Done Inline Actions Comments should start with capital letter, and end with period. Same elsewhere. ankitm: Comments should start with capital letter, and end with period. Same elsewhere.
		PratikPB2123AuthorUnsubmitted Done Inline Actions see `BKE_defvert_normalize_lock_single` or `BKE_defvert_normalize_lock_map` PratikPB2123: see `BKE_defvert_normalize_lock_single` or `BKE_defvert_normalize_lock_map`

		float scalar = (1.0f / tot_weight) * lock_iweight;
		ankitmUnsubmitted Done Inline Actions `const float scalar` https://isocpp.github.io/CppCoreGuidelines/CppCoreGuidelines#con1-by-default-make-objects-immutable https://wiki.blender.org/wiki/Style_Guide/C_Cpp#Const ankitm: `const float scalar` https://isocpp.github.io/CppCoreGuidelines/CppCoreGuidelines#con1-by…
		for (i = dv->totweight, dw = dv->dw; i != 0; i--, dw++) {
		if (dw->def_nr < vgroup_tot) {
		if (dw != dw_lock) {
		dw->weight *= scalar;

		/* in case of division errors with very low weights */
		CLAMP(dw->weight, 0.0f, 1.0f);
		}
		}
		}
		}
		}
		}

	void BKE_defvert_flip(MDeformVert dvert, const int flip_map, const int flip_map_len)	void BKE_defvert_flip(MDeformVert dvert, const int flip_map, const int flip_map_len)
	{	{
	MDeformWeight *dw;	MDeformWeight *dw;
Context not available.