Blender Git Loki

Merge remote-tracking branch 'origin/master' into temp-lineart-contained

Merge remote-tracking branch 'origin/master' into temp-lineart-contained

LineArt: Use array instead of array for ba->linked_edge

LineArt: Toggle duplicated edge for types.

Merge remote-tracking branch 'origin/master' into temp-lineart-contained

LineArt: Option for toggling chaining modes.

Allows switching between image-space and
geometry-space chainin.

Merge remote-tracking branch 'origin/master' into temp-lineart-contained

Fix T54339: Shapekey Max value doesn't clamp existing value

Update the "current value" of the Shape Key blend amount when value is
not within the min/max range. New function `rna_ShapeKey_update_minmax`
used to update and clamp the current value.

Reviewed By: mano-wii, lichtwerk, #animation_rigging, sybren

Maniphest Tasks: T54339

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

Fix T88180: Enable HQ normal workaround for RX 580X cards.

This needs to be backported to 2.83.

GPencil: Remove `B` key to create Blank Keyframe in Drawing mode

We have the `I` menu for that. This is part of the old code.

GPencil: Rename prop `dilate_pixels` to `dilate`

Better avoid units in name.

Fix T82155: Fade inactive geometry overlay not working with instanced geometry

This enables the overlay for instanced geometry.

After this change, objects that are an instance of the current active
object (which are also being modified in the current active mode) won't
fade, which is different from the previous behavior.

Reviewed By: fclem

Maniphest Tasks: T82155

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

Fix T87764: Mem-leak during viewport rendering.

When a scene uses cryptomatte the viewport rendering would lead to a
memory leak. The reason was that all image renders (viewport+final)
activated cryptomatte. But is only used for final rendering.

This patch only activates cryptomatte when doing final rendering.

Functions: Add materialize methods for generic mutable virtual array

Similar to how `GVArray_For_VArray` implements `materialize_impl` to
forward the work to its non-generic virtual array, we can do the same
thing for the mutable version, `GVMutableArray_For_VMutableArray`.

This commit should have no visible changes, since as far as I can tell
the only user of this class does not implement special materialize
methods anyway.

Cleanup: Use a helper function for repetitive code

Retrieving data from the component can be done in a separate function
to save some repetition.

Fix T88060: Expand freezing when deleting a Face Set with multiple loose parts

When checking if the mesh has only one Face Set only the current active
component for expand needs to be checked. Otherwhise other components
that won't be modified by Expand that contain other IDs will be taken
into account, making the Face Set deletion go into an infinite loop.

Reviewed By: JacquesLucke

Maniphest Tasks: T88060

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

Fix warning on Apple Clang, missing override specifier

Was giving a warning:
```
BKE_spline.hh:293:35: warning: 'interpolate_to_evaluated_points' overrides a
member function but is not marked 'override' [-Winconsistent-missing-override]
```

Fix T88145: Dynamic Paint initial color using vertex color not working as expected

Mistake in {rBe48c4d73d378}.

Was using the vertex index as a lookup for the loop color (instead of
the loop index).
(Issue was not present in original D1429 btw).

Maniphest Tasks: T88145

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

Cleanup: correct/clarify PlayAnim comments

Cleanup: unused variable warning