Blender Git Loki

Update Grease Pencil anim channel side panel to show a warning when channel colors are disabled.

Merge remote-tracking branch 'origin/master' into temp-T82134-anim-group-colors-to-prefs

Grease Pencil UI code: use `row` for rows in the UI

Rename `col` to `row` when it's actually a row (and not a column).

No functional changes.

UI: Simplify some tool icon geometry

Removing interior vertices can remove some complexity from the final
exports. Also improved the topology slightly in some cases.

Cleanup: store results of function calls in const values.

Fix py-defined pointer properties wrong tag re ID ownership.

This fixes critical bug with liboverride when soe add-ons add some
RNA ID Pointer properties.

ID pointers should **never** have ownership of their ID when defined
from python.

(As a reminder, RNA properties owning their ID pointers are extremely
rare even from C code, only embedded IDs (root node trees, master
collections) and the shape keys snowflakes are concerned.)

Fix incorrect colors in grease pencil strength tool icon

Merge remote-tracking branch 'origin/master' into temp-T82156-parenting-and-constraints

Cleanup: Document output of `BKE_object_where_is_calc` and friends

Add a comment to the declaration of the `BKE_object_where_is_calc...()`
functions to explain where the result of the calculation is stored.

No functional changes.

Cleanup: Sanitise return value of `ED_object_parent_set()`

Consistently return `false` from `ED_object_parent_set()` when parenting
is not possible. Before, when parent and child were the same object, the
function would return `true` even though the parent-child relation was
not made.

Just returning `false` in the `parent == child` case would break the
parenting operator, as `false` stops its loop over all selected objects.
This tight coupling caused T82312. The loop now has its own check for
this, so that it properly continues, and the implementation of
`ED_object_parent_set()` is decoupled from its surrounding code.

No functional changes.

GPencil: Cleanup comments typo error

Fix build error on windows: Don't use designated initializers in C++

Fix T82156: Object with Copy Rotation Constraints translates when being parented

Avoid application of constraints when computing the parent-inverse
matrix.

Constraints are meant to be evaluated last; object transforms are
computed this order:

1. `parent->obmat` (the parent object's world matrix)
2. `ob->parentinv` (the object's parent-inverse matrix)
3. Object's loc/rot/scale
4. Object's constraint evaluation

When the constraints are used to compute the parent-inverse matrix,
their effect is moved from step 4 to step 2 in this list, potentially
rotating or scaling the object's local transform. This causes unwanted
movement as reported in T82156.

This behaviour (erroneously taking the constraints into account) has
been in Blender since the first commit, so historically I can only find
"Initial revision" as context.

Cleanup: Document output of `BKE_object_where_is_calc` and friends

Add a comment to the declaration of the `BKE_object_where_is_calc...()`
functions to explain where the result of the calculation is stored.

No functional changes.

Merge branch 'master' into geometry-nodes

Add comment about size of generated previews of filebrowser thumnails.

Starting progress on python interface

Fix crash opening some 2.4x .blend files with drivers or NLA editor

For example, outlinertest.blend from test240.zip.

Another attempt at fixing T81963: Random rare crashes in override code.

Adding another pass of ensuring valid up-to-date pose data in RNA
function itself...

LibOverride: Do not assert on missing operands in apply function.

This can happen after some changes in lib file and resync in user file
e.g..