Blender Git Loki

Fix T82519: Adding 2nd driver doesn't recalculate

Remove `return` from for-loop which blocked the recalculation of driven
values when it found the first driver.

Reviewed By: sybren, sergey

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

Geometry Nodes: simplify GeometrySet ownership handling

Previously, GeometrySets and GeometryComponents has reference
counters and could be shared. This commit changes it so that
only GeometryComponents are shared. A GeometrySet is a fairly
small type that is cheap to copy.

A lot of code simplifies when we can assume that GeometrySet
is cheap to copy.

Merge branch 'blender-v2.91-release'

Fix T81271: Fix crash in BLI_gzopen on Windows

Previously the return value of `ufopen` wasn't checked and if it failed,
`NULL` was passed into `fclose()` which resulted in a crash. This patch
avoids this by returning from `BLI_gzopen` when the file cannot be created.

Reviewed By: sebbas, iss

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

Remove multiple modes (always return the object to the initial position)

Merge branch 'master' into cycles_procedural_api

Merge branch 'blender-v2.91-release' into master

Fix T82460: Color Management Curves do not update when Image/UV Editor
is present

Caused by rB4212b6528afb.

'updateGLSLCurveMapping()' compares cacheIDs and in certain scenarios,
these are the same when they should not.

- whenever we had multiple viewports that are colormanaged with
curvemappings this worked right (cacheIDs were different)
- for example, this also worked right when the ImageEditor displays a
Render Result or a Compositor Viewer
- but it worked wrong when the Image Editor displays any other Image (or
no Image at all)
- it also worked right if there were multiple Image Editors [and one of
them displays a Render Result e.g]

Now why is this so?

For comparison, the curve mapping's pointer/address is used.

- update_glsl_display_processor frees the curve_mapping, see
BKE_curvemapping_free(global_glsl_state.curve_mapping)
- similar, update_glsl_display_processor creates a new curvemapping, see
BKE_curvemapping_copy(view_settings->curve_mapping)
- now for the situation that a viewport with curvemapping and a viewport
without curvemapping is present and you make changes to the curvemapping
the following happens:
-- curve_mapping_settings->cache_id is set once [to the memory address
of curvemapping before change]
-- change happens
-- viewport 1 frees curvemapping
-- viewport 2 duplicates using BKE_curvemapping_copy, but this one gets
the same address like before the change
-- this means we have different data on the same address with the same
cacheID...

Solution: to really make the cache ID unique we can combine the pointer
with its 'changed_timestamp' [which increases on every change].

Reviewers: jbakker

Maniphest Tasks: T82460

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

Enable modes according to the number of times the button is pressed

Transform: New feature to set a custom 'Snap With'

Ref T66484

Basically the idea is to use a modal keymap to change the current Base Point
defined by `Snap With` by one that can be chosen through snapping.

{F9170047}

I prefer to avoid using any of the modifier buttons (Shift, Alt, Ctrl) as
they can be used in the future to allow navigation during transform operations.

So the shortcut chosen in this patch is the {key B}.

Note:
This feature is not enabled if the scene snap option is Incremental or grid.

Maniphest Tasks: T66484

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

# Conflicts:
# source/tools

Merge remote-tracking branch 'origin/blender-v2.91-release'

Fix T82740: Drivers on movie clip datablock crash on file open

Add call to `BKE_animdata_blend_read_data()` after remapping the MovieClip
datablock address.

Geometry Nodes: Move GeometrySet classes to global namespace

Brecht and I decided to move the GeometrySet classes
to the global namespace for now. This way we can use
the same type in C and C++ and don't have to use
reinterpret_cast as often.

Cleanup: clang-tidy

Merge branch 'master' into geometry-nodes

Fix T81817: use-after-free when trying to open file from splash screen

The issues was that Blender was trying to refresh the
splash screen region. However, opening the file browser
closed the splash screen and freed the region.

The fix is to simply not refresh the region.

Has been approved in T81817.

Merge branch 'master' into xr-actions-D9124

Correct T82753 Fix from 979fe4f1be7d6fe65e52b3ab805ab061a92d1f03

Fix T82753: Crash using add-object tool when gizmos are disabled

Grease Pencil: adjust behavior of target object detection

First detect the other selected object, then check it can be written to.

Otherwise the target object could be the first one found when
looping over objects which is random from the user perspective.

Move the type check to the operator, which also checks the data isn't
library data which was being ignored.