Blender Git Commits

Fix missing document icons on macOS 10.4 and earlier

Re-adds a legacy document icon for macOS 10.14 Mojave that is
consistent with the system generated document icon on macOS 11
Big Sur. It uses the old-style document sheet icon, but includes the
file extension underneath the Blender icon (unlike the previous
legacy document icon that was removed in D10267).

Adds the missing description for the exported type identifier.
Finder now correctly displays ?Blender File? instead of ?data?
for Blender files.

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

GPencil: Hide Interpolate Only Selected in Draw Mode

This option is only valid in Edit mode. Also changed the space between options to improve UI.

Reviewed by: @mendio

GPencil: Move Automerge button position in topbar

This helps to keep animation buttons grouped.

Fix T86591: Tracking backwards is slower

The root of the issue was caused by the PredictMarkerPosition()
always returning false when tracking backwards. This was making
it so tracker always had to run brute initialization, which is
an expensive operation.

From own timing here:

- Tracking forward takes 0.667637 seconds
- Tracking backward used to take 2.591856 seconds
- Tracking backward now takes 0.827724 seconds

This is a very nice speedup, although the tracking backwards is
still somewhat slower. Will be investigated further as part of
a regular development.

Fix (unreported) memleak in collection/viewlayer code.

In collection/viewlayer synchronization code, in some cases, there are
extra unused view layer collections left in old list after all needed
ones have been moved to the new list.

Found while working on T86741.

Fix T86741: Remapping could create doublons of collections in hierarchy.

Code rebuilding/ensuring the sanity of the collection hierarchy was not
checking for a same collection being child of the same parent multiple
times.

This was already prevented to happen in code adding collections to other
collections, but not for the remapping case.

Fix: wrong use of assert in recent commit.

UI: Remove text for add and remove buttons in cryptomatte node

The text is unecessary and it's always cut off anyway.

Cleanup: remove unexposed nodes

Those nodes are leftovers from my work on particle nodes and are not needed currently.
They can be added back easily if they become necessary.

Geometry Nodes: make random float node more consistent with other nodes

Previously, different Random Float nodes would generate different values
depending on where they are in the node group hierarchy. This can be useful,
but should definitely not be the default behavior, because it is very inconsistent
with other nodes.

Functions: make multi functions smaller and cheaper to construct in many cases

Previously, the signature of a `MultiFunction` was always embedded into the function.
There are two issues with that. First, `MFSignature` is relatively large, because it contains
multiple strings and vectors. Secondly, constructing it can add overhead that should not
be necessary, because often the same signature can be reused.

The solution is to only keep a pointer to a signature in `MultiFunction` that is set during
construction. Child classes are responsible for making sure that the signature lives
long enough. In most cases, the signature is either embedded into the child class or
it is allocated statically (and is only created once).

Fix regression in bd7969159997b8802d54e79a002350a7fb97944d

The sequencer header needs to be drawn, even if the returned
variables aren't used.

LibraryOverrides: Initial Test Case.

Added a first test case for review. This will be the base for future test cases.

The current API is sufficient for what is expected for such a low level API.
One concern is that you need to trigger a save in order to update the library overrides
structure. Not expected from TD/Dev point of view.

Test cases are very important when implementing restrictive mode as it is a second evaluation mode that
has impact on the (current) permissive mode.

Reviewed By: Sebasti�n Barschkis, Bastien Montagne

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

Fix T86405: Always translate when in `en_US` locale too.

The impact of translations on UI performances is neglectable, and this
gives better handling of some odd cases where original language of an
add-on is not English.

Cleanup: add comment

See rBb9cd2f4531ca670c196b0b14b1359d0f375103c2 for more info.

Cleanup, LineArt: Fix MSVC compiler warnings.

Currently 3 of the printf lines use the format specifier "%lu" for data
of type size_t instead of "%zu". While this works, this creates compiler
warnings.

This diff fixes those warnings by using the correct format specifier.
Tested using MSVC, but should be correct on any compliant compiler.

Reviewed By: Sebastian Parborg

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

Clang Tidy: disable modernize-raw-string-literal again

This check does not work when gtests are enabled yet.
See rBb9cd2f4531ca670c196b0b14b1359d0f375103c2 for more details.

Fix UI (and code!) typos.

Cleanup: check the space type instead of the operators poll function

The poll function may fail for reasons other than the space type,
as this menu is selected based on space-type, check this instead.

RNA: rename recently added line-art properties

- Use `use_` prefix for boolean properties.
- Use `_all` instead of `_everything`,
following existing conventions and the properties UI text.
- Use `object_instances` instead of old term `dupli` / `duplication`.
- Use `clip_plane` instead of `clipping_boundaries`,
matching the RegionView3D property naming.