Blender Git Loki

Merge branch 'temp-review-asset-previews' into temp-review-asset-drag-drop

Merge branch 'temp-review-asset-create-ui' into temp-review-asset-previews

Merge branch 'temp-review-asset-metadata' into temp-review-asset-create-ui

Merge branch 'master' into temp-review-asset-metadata

Cleanup: Use enum for file selection type definitions

Makes it more clear that these belong together and allows using the enum type
rather than just `int`.

Cleanup: Correct an own earlier commit to use an existing utility function

Didn't know this function existed, better to use it then to avoid verbosity.

File Browser: Refactor access to the selection parameters struct

* Avoid direct access to `SpaceFile.params`, use a getter instead. This matters
because once the asset-browser changes are in, there will be an alternative
selection parameter object. The getter can return the correct one.
* Rename the function to ensure the parameters. The old name
`ED_fileselect_get_params()` wasn't a mere getter, it would create the
parameters if necessary. Now we have an actual getter, so better be clear.
* In some instances, I replaced the old "get" function with the new mere
getter. So the ensure logic is called less often. However, in these cases we
should be able to assume the selection parameters were created already as
part of the editor creation routine.

The term "active" in the new function names may seem a bit odd in the current
context, but that is a preparation for the Asset Browser merge as well. Like
said, there will be two file selection parameter objects in the space.

Fix some typos/mistakes

Cleanup: strip trailing spaces

add required data mask

update comments

change owner of evaluated pointcloud for more consistency

support geometry nodes modifier in edit mode and cleanup

don't make temporary mesh object, if the owner is a mesh itself

This way, doing pointer comparison to check if the mesh object is active works.

correct recursive dupli instancing

initial working version, does not work with edit mode yet

move DerivedMesh.c to c++

Cleanup: Use guarded allocator for data-block names returned from file reading

Direcly using the C library allocator functions is usually avoided in favor of
our guarded allocator. It's more useful when debugging.

Fix unexpected "/" path after normalizing empty directory path

Caused problems in the Asset Browser branch when passing an empty path. The
function shouldn't blindly add a slash but sanity-check the input parameters
first.

LibOverride: Add initial support for adding new NLA tracks.

Also makes NLA tracks and strips overridable.

User can either edit existing strips in existing NLA tracks (but not add or remove them), and/or add new NLA tracks after those comming from the linked data.

Most of the work was as usual checking operators and adding protections against illegal operations in override context.

Note that since we can only rely on indices to deal with local added tracks, we forbid any local track being before any linked/original track.

Maniphest Tasks: T72629

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