Blender Git Loki

Merge branch 'blender-v3.0-release'

Fix T93198: Frame Selected in greasepencil curve editing does not work

Was not taking into account curve points at all.

Maniphest Tasks: T93198

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

Cleanup: Removed unused code in curve mask creation.

Generating curve mask for 2d texture painting had some hard-coded
parameters that eventually weren't used in the algorithm (hardness and
rotation of the brush). This patch removes these parameters.

Cleanup: Make parameter const (BKE_brush_curve_strength_clamped).

Painting: migrated curve mask generation to CPP.

Curve mask generation is done during 2d texture painting. There are some
performance issues in this part of the code. Before addressing those we
move the code to CPP.

Revert "Fix (unreported) broken handling of constraints reordering with liboverride."

This reverts commit 6eaa69c66c98d291b80331330391664415f759a4.

Committed by nistake, sorry for the noise.

Cleanup: use simple data member instead of callback

This really doesn't have to be a callback currently, since it is always
the same `CPPType` for a socket type.

Cleanup: make naming more consistent

Merge branch 'blender-v3.0-release'

Fix broken NLA RNA code after own rBfa6a913ef19c.

Thanks to @scurest for noticing this mistake!

Fix (unreported) broken handling of constraints reordering with liboverride.

New drag&drop reordering code would call constraints reordering operator
with the generic context, and not the one from the panel's layout.
missing the "constraint" member which is mandatory for poll function to
properly deal with override vs. local constraints.

This commit fixes it by generating a temp bContextStore in the panel
re-ordering callback.

NOTE: this fix will have to be extended to modifiers (which happen to
work currently because they have an 'active' status), and gpencil
modifiers (which are also broken currently).

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

Fix T93256: Instances to points node broken after recent commit

When 97533eede444217 added the instance domain, it didn't change
the domain that instance attributes are read from in this node.

Functions: remove test for dynamic name

This was broken in rB6ee2abde82ef121cd6e927995053ac33afdbb438.

Functions: fix compile error in tests

Functions: use static string for parameter names

The idea behind this change is the same as in
rB6ee2abde82ef121cd6e927995053ac33afdbb438.

A `MultiFunction::debug_parameter_name` method could be
added separately when necessary.

Functions: use static names for multi-functions

Previously, the function names were stored in `std::string` and were often
created dynamically (especially when the function just output a constant).
This resulted in a lot of overhead.

Now the function name is just a `const char *` that should be statically
allocated. This is good enough for the majority of cases. If a multi-function
needs a more dynamic name, it can override the `MultiFunction::debug_name`
method.

In my test file with >400,000 simple math nodes, the execution time improves from
3s to 1s.

Cleanup: Move menu search template to C++

This allows the use of better/more readable data structures,
and will also make some refactors to the search button easier.
The build completed on the buildbot for all platforms.

Merge branch 'blender-v3.0-release'

Fix T92090: Eevee crash with Intel HD 4000 and macOS 10.15.7

A recent security update to macOS 10.15.7 causes crashes when using Eevee and
various other 3D viewport features. It appears that glGenerateMipmap is
broken, causing a crash whenever its commands are flushed/submitted to the GPU.

Ideally this would be fixed in a driver update, however it's unlikely this will
happen. Earlier macOS versions have been receiving security updates for 2 years,
and that window has just passed for 10.15. Further, computers with these GPUs
can't upgrade to a newer macOS version.

As a workaround, disable mipmaps on these GPUs, by setting the mipmap max level
to 0 and not calling glGenerateMipmaps. Effects like depth of field also use
mipmaps, but fill in the mip levels by other means. In those cases we keep the
mipmap level.

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

Cleanup: unset 'FILE_ENTRY_PREVIEW_LOADING' at the end

Although this function only runs on the main thread, it seems safer to
clear the flag only after setting the result.