Blender Git Loki

Cleanup: clang-format

Fix memory leaks in sculpt mode trimming tools

BKE_mesh_free() seems to not free the meshes correctly, so using BKE_id_free() instead.
The looptri array was also not freed.

Reviewed By: sergey

Differential Revision: https://developer.blender.org/D9426

Cleanup: transform.h comments, use doxy sections

Fix T81915: Draw Face Sets not working with deformed sculpt mesh

The draw face sets brush uses the poly center when used in meshes to increase
its precision when working in low poly geometry. For this to work with deformed
meshes, the deformed coordinates from the PBVH should be used instead.

Reviewed By: sergey

Maniphest Tasks: T81915

Differential Revision: https://developer.blender.org/D9424

UI: Tweaks to the Warning Icon

Warning Sign Alert Icon given a more rounded border.

Differential Revision: https://developer.blender.org/D9443

Reviewed by Pablo Vazquez

Cleanup: sort structs, files

Fix T80043: missing Cycles displacement update when relinking output sockets

Move "Camera Parent Lock" from preferences to Object Relations

"Camera Parent Lock" can be useful when rigging cameras, but it is not
intuitive, and has also generated a lot of confusion (bug reports).
This is because it breaks the fundamental parent <-> child relationship
conventions in Blender, and there is no indication that it's intended
without diving into the preferences.

This commit moves the setting to the object level, and exposes it in
the relations panel in the property editor. It is exposed for every
object type because any object type can be "View Locked" in the 3D view.
The property description is also updated to reflect this change and be
more specific without getting too long.

In the future this could become a more general feature of the transform
system, but for now it is limited to "Lock Camera to View".

Differential Revision: https://developer.blender.org/D9239

Fix T82164: Knife tool draws huge vertices after using bgl.glPointSize

Since it is possible to have multiple draw callbacks, (some of which
use bgl and others gpu), check and force the reset of the drawing status
at the end of each callback.

Differential Revision: https://developer.blender.org/D9476

Fix T82423: Replace duplicate name in keymap

The two entries `TFM_MODAL_AUTOCONSTRAINT` and
`TFM_MODAL_AUTOCONSTRAINTPLANE` had the same name
displayed in the UI. The latter is now includes
"plane" in it's name.

Reviewed By: mano-wii

Differential Revision: https://developer.blender.org/D9474

Cleanup: Reduce variable scope

Differential Revision: https://developer.blender.org/D9475

Fix T82423: Add modifier key back into keymap

Commit rBf5080c82dd915db6c7b9dd68a52aaaccf2600137
accidentally remove the Shift modifier key from
the `AUTOCONSTRAINPLANE` shortcut.

Differential Revision: https://developer.blender.org/D9480

Fix T82428: Cycles crashes when building volume meshes

The Volume Node did not have all of the sockets from its Mesh base class
which are now required due to the recent socket API change.

pre-merge commit

Fixed handling of edge seam flags in dyntopo, for some reason did the
code was not brought over from trimesh branch.

Added debug option in dyntopo to draw sculpt colors as solid cells
instead of interpolating (it's next to "draw smooth" in the dyntopo
settings panel).

Made vcol boundary tool avoid perfectly degenerate triangles.

Replace call to BMLog with origco CD layer.

Also fixed SCULPT_HIDE_FACE_SETS. It was 1<<16, which overlaps
SCULPT_DYNTOPO_DETAIL_MANUAL. Changed it to 1<<17. Someone should do
this in master too.

Merge remote-tracking branch 'origin' into temp_bmesh_multires

Fix redraw error for dyntopo smooth brush