Blender Git Loki

Fix T81953: Python error in UV Editor Overlay popup

Leftover from rBe05ce1ea2029, 'use_image_editor_legacy_drawing' was
removed.

Maniphest Tasks: T81953

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

Multires: Remove legacy subdivision code

Is no longer used, fully replaced with more powerful algorithm.

Multires: Cleanup, unused code

Multires: Remove legacy compatibility code

It was rather a huge chunk of code, which started to become
more harder to maintain with the transition to OpenSubdiv based
implementation. Because of this transition, the compatibility was
also rather on a poor side.

Remove compatibility support for pre-2.50.9 multires.

Ref T77107

Reviewed By: brecht, mont29

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

Compositor: Ensure keying node result is pre-multiplied

Historically the result of the keying node was violating alpha
pre-multiplication rules in Blender: it was simply overriding
the alpha channel of input.

This change makes it so keying node mixes alpha into the input,
which solves the following issues:

- The result is properly pre-multiplied, no need in separate
alpha-convert node anymore.

- Allows to more easily stack keying nodes.
This usecase was never really investigated, but since previously
alpha is always overwritten it was never possible to easily stack
nodes. Now it is at something to be tried.

Unfortunately, this breaks compatibility with existing files, where
alpha-convert node is to be manually removed.

From implementation side this is done as a dedicated operation since
there was no ready-to-use operation. Maybe in the future it might
be replaced with some sort of vector math node.

Reviewed By: brecht

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

GPU: Memory leak when scaling buffers

`imb_gpu_get_data` could reuse `data_rect` when it was already in used (double alloc).
making the first use leak. This was detected after enabling OpenGL Texture
Limit.

Reviewed By: Cl�ment Foucault

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

Merge branch 'blender-v2.91-release'

Fix T81949: Child Of Constraint can't assign inverse matrix

Caused by ad70d4b0956f5, assigning the matrix now
clears the flag that would reset it.

Cleanup: centralize BLF default functions in the header

Cleanup: split BLF default into own file

This avoids accidents using user-preferences in the main BLF API,
which could cause preferences to be used unintentionally
(such as stamping into renders or creating generated images).

As well as uses of BLF when preferences aren't loaded
such as animation playback.

Merge branch 'blender-v2.91-release'

Fix animation player initialization

Updates from 9d30fade3ea9b weren't applied to the animation player
causing an assert and missing call to IMB_init.

Merge branch 'blender-v2.91-release'

Merge branch 'blender-v2.91-release'

Cleanup: remove '_' prefix from used argument

Fix T81905: Active keyframe unavailable when handle selected

Selecting an F-Curve handle caused an assertion as well as treating
the key-frame as inactive.

Allow active the keyframe to be active when it's handle is selected,
as is done with bezier curves.

Cleanup: Use LISTBASE_FOREACH macro in curve code

These changes should result in more readable and undestandable code,
especially where while loops were use instead of for loops. They are
not comprehensive, and I skipped wherever the change was not obvious.

Merge branch 'blender-v2.91-release'

Fix T81939: crash calling bmesh.utils.vert_separate()

Missing NULL check in bmesh_kernel_vert_separate.

Merge branch 'blender-v2.91-release'